Обновить

Комментарии 7

> global $modx;

как минимум вот это претендует на то, что код не тестировался…

> $selectQuery = $modx->db->select('*', $tblName, 'contentid='. $docID. ' AND tmplvarid='. $tplRow['id']);

вот это я так понял превратится в select * from — что тоже не очень хорошо и относительно медленно…

надо кодом еще работать и работать, а сама идея вроде как не плохая.
Код тестировался. Работает на живом сайте, функция взята из modxcms.com/extras/package/?package=81 чтобы свою не писать.

вы немного, наверное, меня не поняли. я подразумевал, кто код не тестировался это значит что не были написаны блочные тесты, регрессионные тесты, интеграционные и тд и тп…

написать такие тесты довольно сложно при использовании глобальных переменных, не используйте их, сделайте лучше параметров к функции
НЛО прилетело и опубликовало эту надпись здесь
даже для небольшого количества не нужно писать плохого кода. использование global является признаком плохого кода
НЛО прилетело и опубликовало эту надпись здесь
Обновил код.
+ исправлена ошибка с тем, что если раньше в хранилище баннеры были в неопубликованном виде, MODx выдавал эксепшн
+ включена поддержка swf файлов
Зарегистрируйтесь на Хабре, чтобы оставить комментарий

Публикации