Крик души.
if (!empty($product_info)) {
$data['model'] = $product_info['model'];
} else {
$data['model'] = '';
}
Серьезно? зато PHP8
почему не
$data['model'] = $product_info['model'] ?? '';
или вообещё в твиге
{{ product.model|default('') }}
--
в контроллере
$data['product'] = $product_info??null;
Да и вообещё в контроллере данные обрабатываться не должны. должно быть что то врогде
$data['product'] = $this->model_catalog_product->getProduct($product_id);
весь product_info должен в могдели быть спряин
Реально код на PHP 5.3 зато выпендрился минимальную версию 8 посивил. неймспейсы по факту зачем тут. Ни автолоагдера ни PSR, или это просто увигдел новое слово надо внедрить.
портянки-портянки-портянки. Честно не понимаю что происходит, меньше кода меньше повода для ошипотому чток, легче обслуживать, это же очевидно.
В твиге есть include. нет же видать даниэлю нравится портянки с километр. почему бы в админке, например в товарах, ибы не подключать инклудами.
Зато ненужную фигню добавляет, типа рекуринга, кому икую фигню нужно пусть бы модулями досивляли. нужен просто рилииющий нормальный движок с минимальным нилиром функционала, осильное расширяться по необходимости, при этом не распыляться на сомнительный функционал.
Глядя на код и последний бэи релиз этого чуда в 2017 году, даже несмотря на современные копания в когде, возникает смутное чувство что версия 3 для опенкари была последней.