Модуль производителей

OCappLab

OCappLab

< coder />
#1
Предлагаю в сборку добавить модуль с выводом производителей.
Если поддержите, тогда перепишу без префиксов, какие я создавал для распространения.
За 10-ть лет работы с OpenCart еще ни разу не видел, чтоб подобный модуль добавлялся в какую-то сборку, как по мне очень удобный модуль.
 

Вкладення

S

SlaSoft

Maestro
#2
Код:
$data['header_title'] = $setting['module_description'][$this->config->get('config_language_id')]['title'] ? $setting['module_description'][$this->config->get('config_language_id')]['title'] : $this->language->get('header_title');
Здесь спорное решение


Код:
        $data['header_title'] = (isset($setting['module_description'][$this->config->get('config_language_id')]['title']) && $setting['module_description'][$this->config->get('config_language_id')]['title']) ? $setting['module_description'][$this->config->get('config_language_id')]['title'] : $this->language->get('header_title');
Т.е. Если язык появился после добавления модуля... то будет ошибка
 
OP
OCappLab

OCappLab

< coder />
Thread Starter #3
Подкорректировал.

Сейчас переписываю без индексов для файлов. И перезалью архив
 
S

SlaSoft

Maestro
#4
Языковый индекс - это правильно..
Файлы - увы..
Идеология тройки - переход в облака, и ограничение доступа пользователей к файлам.
 
S

SlaSoft

Maestro
#6
а... ну зачем же

я свое не собираюсь удалять.. меньше конфликтов

Я вот даже стокнулся с конфликтом Названия модуля

мой и чужой имеют (имели) одинаковое название)
И мой модуль не попадал в списки..
 
OP
OCappLab

OCappLab

< coder />
Thread Starter #8
Подправил проверку.
+ добавил еще проверку для цикла, а то вспомнилось, была на каком-то хостинге, что foreach не захотел пустой массив обрабатывать
 

Вкладення

Симпатії: Baco
ALarik

ALarik

expert
#9
Отправил на интеграцию @Baco для гитхаб, исправил пути (если можно, впредь не надо создавать дополнительные подкаталоги, я попрошу что бы OCappLab - добавили в список соавторов сборки, но для удобности, оставляйте модули - в каталоге модулей, языковые переменные - в каталоге языковых переменных модулей, и въюшки - соответственно, не надо дополнительно стоздавать ждя этого каталог ocapplab) + добавил TPL файл.
 
Baco

Baco

architect
#10
Код не тестировал, просмотрел - все гуд, заливаю на гит, по предложению указания своего бренда в сборке, поддержу Артура, не надо делать этого в сборке, это создает дополнительную файловую структуру, прошу придерживаться текущей, не нарушая, если уж нет логического каталога - можно оговорить, так - все в стандартные каталоги прошу инклудить.
 
OP
OCappLab

OCappLab

< coder />
Thread Starter #11
Ок. Я его вылаживал в готовом виде, тот что был. Просто как предложение.
В будующем буду структуру сохранять станадртного OpenCart
 
Симпатії: Baco
Зверху