egor23, я честно скажу, что считаю тебя бестолковым форумны троллем который ничего не понимает, а просто подгавкивает, но чтобы не быть голословным я отвечу на озвученные претензии:
1. По поводу SQL запроса в контроллере: там один запрос в контроллере, который был туда добавлен чтобы не вытягивать из базы лишнюю информацию через стандартную модель и не создавать тем самым дополнительную нагрузку на базу. Использование SQL запросов в контроллере никак не ограничено стандартнами кода Opencart и SQL запросы есть в контроллерах ядра. Создавать отдельный файл модели для одного запроса я считаю друным тоном: это усложняет дебаг, усложняет удаление модуля, а, главное, создает дополнительные сложности для проверки кода на наличие вреноносного кода (это то, что нужно делать всегда перед установкой бесплатных модулей).
2. По поводу HTML в контроллере - в этом модуле нет предсталвения т.е. нет HTML. Нет в контроллере и нет вообще нигде. Твоему другу под ником netruxa что-то привидилось и он написал глупость. Ты её повторил.
3. Javascript в контроллере работает ничем не хуже, чем Javascript в отдельном файле, упрощает обслуживание кода, упрощает удаление модуля, дает дополнительный контроль над расширением через ивенты и исключает проблемы из-за некорректного chmod на js-файл. К слову, Даниэль уже анонсировал, что единственным методом модификаций в ближайших обновлениях будут Ивенты поэтому я думаю к ближайшие пару лет можно ожидать, что весь Javascript ядра перейдет в контроллеры.
В обещм, Егор, иди учи матчасть, с тобой говорить очевидно не о чем.
А вот тебя, stickpro, я бы послушал. Ты очевидно позер и хмало, но вдруг ты правда что-то увидел. Тебе есть что сказать?