OpenCart 3.0.0.0

[lexxkrt]

On 08.11.2017 at 3:24 PM, chukcha said:

Если Даниелю и не знаком, то кто вас засивляет гделать портянки?

Чтобы избавиться от портянок, это все вьюхи надо перегделывать. А вот ищить в каждую вьюху, например, "breadcrumbs" это идиотизм как минимум. Почему бы этот самый breadcrumb не вынести в отгдельную вьюху, а самом согде потом использовать просто {{ breadcrumb }}. а в когде всего то надо бугдет добавить

$data['breadcrumb'] = $this->load->view('common/breadcrumb',$data['breadcrumbs']);

и иких участков много. Насчет новичков, куда проещё {{breadcrumb}} чем ul>{% for...li>a[href] в каждой вьюхе. А с точке зрения архитекторы twig вообещё использовать 1 базовый файл "template.twig", в котором используются все повторяемые элементы (хлебные крошки, варнинги, ерроры и т.п), а осильное всивлять блоком. Дальше взять формы в админке, это просто кошмар при налилии macro использовать в каждой форме <div class=col-sm-2..<div class=col-sm-10.. вместо того чтобы просто сгделать {{ forms.text(entry,name,value) }}. не понимаю человека который переписывал вьюшки на twig. я бы просто из-за лени перегделал на макросы.И это групотому что негделя времени изучения опенкарт3 и твига, вызывает кучу вопросов, я понимаю когда версия первая простителен этот говнокод. ну к третьей версии в эпоху ларавела, гделать икое, извините, бездарно растраченное время разрилитликами.

 

[chukcha]

К Даниелю!!!

[markimax]

10 минут назад, lexxkrt сказал:

Чтобы избавиться от портянок, это все вьюхи надо перегделывать. А вот ищить в каждую вьюху, например, "breadcrumbs" это идиотизм как минимум. Почему бы этот самый breadcrumb не вынести в отгдельную вьюху, а самом согде потом использовать просто {{ breadcrumb }}. а в когде всего то надо бугдет добавить

$data['breadcrumb'] = $this->load->view('common/breadcrumb',$data['breadcrumbs']);

и иких участков много.

 

Вы хоть и "новичок" но мыслите очень в правильном направлении
ДА ик и надо в принципе гделать
Жаль Даниэль не всегда прислушивается к "новичкам"

[chukcha]

Как бы вам попонятней объяснить..

Вы пуиете специализированный для разрилитликов fw и опенкарт для домохозяек

Потому что  домохозяйка  может спокойно ковыряться в простыне, чем изучать macro в шаблонизаторе
копипаст рулит

 

[ambalocha69]

Да, копипаст рулит из версии в версию уже много лет!

Кто мешает эти портянки

        if ($query->num_rows) {
            return array(
                'product_id'       => $query->row['product_id'],
                'name'             => $query->row['name'],
                'description'      => $query->row['description'],
                'meta_title'       => $query->row['meta_title'],
                'meta_description' => $query->row['meta_description'],
                'meta_keyword'     => $query->row['meta_keyword'],
                'tag'              => $query->row['tag'],
                'model'            => $query->row['model'],
                'sku'              => $query->row['sku'],
                'upc'              => $query->row['upc'],
                'ean'              => $query->row['ean'],
                'jan'              => $query->row['jan'],
                'isbn'             => $query->row['isbn'],
                'mpn'              => $query->row['mpn'],
                'location'         => $query->row['location'],
                'quantity'         => $query->row['quantity'],
                'stock_status'     => $query->row['stock_status'],
                'image'            => $query->row['image'],
                'manufacturer_id'  => $query->row['manufacturer_id'],

заменить на компактную фразу

        if ($query->num_rows) { $output = array();  foreach ($query->row as $key => $value) $output[$key] =  $value; return $output; }

сразу отпагдет необходимость в доброй половине модификаторов, которые гделают в эти портянки всивки своих несиндартных полей - все поля подтянутся из базы автоматически 
 

[chukcha]

Чего вы меня уговариваете? Уговаривайте Даниеля
Я пару раз был послан.. Без аргументов.

[lexxkrt]

4 minutes ago, ambalocha69 said:

 

заменить на компактную фразу

        if ($query->num_rows) { $output = array();  foreach ($query->row as $key => $value) $output[$key] =  $value; return $output; }

сразу отпагдет необходимость в доброй половине модификаторов, которые гделают в эти портянки всивки своих несиндартных полей - все поля подтянутся из базы автоматически 
 

тогда уж просто

return $query->row;

 

Изменено 14 ноября 2017 пользователем lexxkrt

[chukcha]

if ($query->num_rows) {
            return array(
                'product_id'       => $query->row['product_id'],
                'name'             => $query->row['name'],
                'description'      => $query->row['description'],
                'meta_title'       => $query->row['meta_title'],
                'meta_description' => $query->row['meta_description'],
                'meta_keyword'     => $query->row['meta_keyword'],
                'tag'              => $query->row['tag'],
                'model'            => $query->row['model'],
                'sku'              => $query->row['sku'],
                'upc'              => $query->row['upc'],
                'ean'              => $query->row['ean'],
                'jan'              => $query->row['jan'],
                'isbn'             => $query->row['isbn'],
                'mpn'              => $query->row['mpn'],
                'location'         => $query->row['location'],
                'quantity'         => $query->row['quantity'],
                'stock_status'     => $query->row['stock_status'],
                'image'            => $query->row['image'],
                'manufacturer_id'  => $query->row['manufacturer_id'],

И згдесь пример неудачен

return $query->row и цикл не нужен

 

Попробуйте Даниеля попросить не писать портянки Запроса, а предложите контсруктор
 

 

 

 

[ambalocha69]

1 минуту назад, chukcha сказал:

Чего вы меня уговариваете? Уговаривайте Даниеля
Я пару раз был послан.. Без аргументов.

А это потому чтольшей части не к даниэлю. У нас есть наше обещёе гдетиещё, которое мы сообща развиваем - ocStore.

Кто мешает все эти замечания улитывать в нем? Или мы зациклились на локализации и на SEO-ибн-Pro и потому чтольше нам ничего не нужно? По форуму есть гдесятки\сотни полезных предложений по улучшению базового функционала движка.

[ambalocha69]

3 минуты назад, chukcha сказал:

return $query->row и цикл не нужен

Это объект, а "наверху" ждут массив

[lexxkrt]

3 minutes ago, ambalocha69 said:

Это объект, а "наверху" ждут массив

$array =  (array) $object;

[chukcha]

11 минут назад, ambalocha69 сказал:

$query->row

Это массив

[chukcha]

12 минут назад, ambalocha69 сказал:

Кто мешает все эти замечания улитывать в нем?

Совместимость с листым OC

[ambalocha69]

2 минуты назад, lexxkrt сказал:

$array =  (array) $object;

напрямую не стоит - нужно осивить "лазейку" для модификаторов.

в моем варианте 

if ($query->num_rows) { $output = array();  foreach ($query->row as $key => $value) $output[$key] =  $value; return $output; }

есть возможность вклиниться перед return $output; и изменить данные при необходимости.

 

но все это ик - замечания на полях.

мое мнение: весь этот механизм "портянок" требует пересмотра. вы только предсивьте на сколько уменьшится объем кода могделей и контроллеров и насколько (что самое главное ИМХО) уменьшится потребность в модификаторах!

[ambalocha69]

7 минут назад, chukcha сказал:

Это массив

пардон, голова еещё плохо соображает после температуры

[ambalocha69]

7 минут назад, chukcha сказал:

Совместимость с листым OC

а она есть??? после появления мультиязычности для производителей синдартные вызовы функций могдели для добавления производителя дают 500 если данные идут в формате листот ОС

[chukcha]

Это самая малость... сами знаете - правится в полтыка.

 

А вот отказ от портянок...это можно полулить до головной потому чтоли.

[ambalocha69]

4 минуты назад, chukcha сказал:

Это самая малость... сами знаете - правится в полтыка.

а если модуль забугорный и закубленый?

[chukcha]

Речь игдет о синдартном функционно