Pull to refresh

Comments 7

> global $modx;

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

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

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

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

написать такие тесты довольно сложно при использовании глобальных переменных, не используйте их, сделайте лучше параметров к функции
Смысл? ModX большое количество посетителей/показов не выдержит, а для небольшого количества сильная оптимизация — излишество.
даже для небольшого количества не нужно писать плохого кода. использование global является признаком плохого кода
С этим я согласен. Комментарий относился к тестированию.
Обновил код.
+ исправлена ошибка с тем, что если раньше в хранилище баннеры были в неопубликованном виде, MODx выдавал эксепшн
+ включена поддержка swf файлов
Sign up to leave a comment.

Articles