Comments 7
> global $modx;
как минимум вот это претендует на то, что код не тестировался…
> $selectQuery = $modx->db->select('*', $tblName, 'contentid='. $docID. ' AND tmplvarid='. $tplRow['id']);
вот это я так понял превратится в select * from — что тоже не очень хорошо и относительно медленно…
надо кодом еще работать и работать, а сама идея вроде как не плохая.
как минимум вот это претендует на то, что код не тестировался…
> $selectQuery = $modx->db->select('*', $tblName, 'contentid='. $docID. ' AND tmplvarid='. $tplRow['id']);
вот это я так понял превратится в select * from — что тоже не очень хорошо и относительно медленно…
надо кодом еще работать и работать, а сама идея вроде как не плохая.
0
Код тестировался. Работает на живом сайте, функция взята из modxcms.com/extras/package/?package=81 чтобы свою не писать.
+1
вы немного, наверное, меня не поняли. я подразумевал, кто код не тестировался это значит что не были написаны блочные тесты, регрессионные тесты, интеграционные и тд и тп…
написать такие тесты довольно сложно при использовании глобальных переменных, не используйте их, сделайте лучше параметров к функции
написать такие тесты довольно сложно при использовании глобальных переменных, не используйте их, сделайте лучше параметров к функции
0
Обновил код.
+ исправлена ошибка с тем, что если раньше в хранилище баннеры были в неопубликованном виде, MODx выдавал эксепшн
+ включена поддержка swf файлов
+ исправлена ошибка с тем, что если раньше в хранилище баннеры были в неопубликованном виде, MODx выдавал эксепшн
+ включена поддержка swf файлов
0
Sign up to leave a comment.
Делаем собственную баннерную крутилку