Делаем собственную баннерную крутилку

[Joka]

> global $modx;

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

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

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

надо кодом еще работать и работать, а сама идея вроде как не плохая.

[vodka_ru]

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

[Joka]

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

написать такие тесты довольно сложно при использовании глобальных переменных, не используйте их, сделайте лучше параметров к функции

[Joka]

даже для небольшого количества не нужно писать плохого кода. использование global является признаком плохого кода

[vodka_ru]

Обновил код.
+ исправлена ошибка с тем, что если раньше в хранилище баннеры были в неопубликованном виде, MODx выдавал эксепшн
+ включена поддержка swf файлов