унифицируем thumb как в опенкарт 4

имхо в опенкарт 3 это делать поздно, а в 4 рано 

. 

если унифицировать это только вью и то, потерять совместимость примерно со всем что модифицирует вывод 

Можна набагато простіше для шаблонів, щоб не тягати "туди-сюди" код шаблонів

{% include 'default/template/product/thumb.twig' %} 

//$loader = new \Twig\Loader\ArrayLoader(array($filename . '.twig' => $code));
$loader1 = new \Twig_Loader_Array(array($filename . '.twig' => $code));
$loader2 = new \Twig_Loader_Filesystem(array(DIR_TEMPLATE)); // to find further includes
$loader = new \Twig_Loader_Chain(array($loader1, $loader2));

Взагалі на "сітку" виводу отих усіх контролерів можна було зробити один контролер (так саме як і модулів виводу товарів)
І не писати простирадло (зараз код Даніеля виглядає як неандертальський в контролерах), а ось щось такого плану

        public function index($settings) {
            return $this
                ->start(settings)
                ->load_language()
                ->header()
                ->breadcrumb()
                ->main()
                ->positions()
                ->output();
        }

...        
private function header() {
	$this->document->setTitle($this->language->get('heading_title'));
	if (method_exists($this->document, 'setRobots')) {
		$this->document->setRobots('noindex,follow');
	}
	return $this;
}
...

А потім подіями як завгодно обробляти

// У файлі установки
$this->model_setting_event->addEvent('your_event_name', 'catalog/controller/your_controller/header/after', 'extension/event/your_event/afterHeader');

// У контролері події
public function afterHeader(&$route, &$data, &$output) {
    // Ваш код, який буде виконано після методу header()
}

там проблема в зовсім іншому

 

				$product_data = [
					'product_id'  => $result['product_id'],
					'name'        => $result['name'],
					'description' => $description,
					'thumb'       => $image,
					'price'       => $price,
					'special'     => $special,
					'tax'         => $tax,
					'minimum'     => $result['minimum'] > 0 ? $result['minimum'] : 1,
					'rating'      => $result['rating'],
					'href'        => $this->url->link('product/product', 'language=' . $this->config->get('config_language') . '&product_id=' . $result['product_id'] . $url)
				];

				$data['products'][] = $this->load->controller('product/thumb', $product_data);

Ідея передавати product_id  у контролер, а в ньому вже getProduct, краще ніж у моделі getProducts
Або зовсім не $product_data, а краще увесь $result без усяких  попередніх змін, типу цього

				$description = trim(strip_tags(html_entity_decode($result['description'], ENT_QUOTES, 'UTF-8')));

				if (oc_strlen($description) > $this->config->get('config_product_description_length')) {
					$description = oc_substr($description, 0, $this->config->get('config_product_description_length')) . '..';
				}

				if (is_file(DIR_IMAGE . html_entity_decode($result['image'], ENT_QUOTES, 'UTF-8'))) {
					$image = $this->model_tool_image->resize(html_entity_decode($result['image'], ENT_QUOTES, 'UTF-8'), $this->config->get('config_image_product_width'), $this->config->get('config_image_product_height'));
				} else {
					$image = $this->model_tool_image->resize('placeholder.png', $this->config->get('config_image_product_width'), $this->config->get('config_image_product_height'));
				}

				if ($this->customer->isLogged() || !$this->config->get('config_customer_price')) {
					$price = $this->currency->format($this->tax->calculate($result['price'], $result['tax_class_id'], $this->config->get('config_tax')), $this->session->data['currency']);
				} else {
					$price = false;
				}

				if ((float)$result['special']) {
					$special = $this->currency->format($this->tax->calculate($result['special'], $result['tax_class_id'], $this->config->get('config_tax')), $this->session->data['currency']);
				} else {
					$special = false;
				}

				if ($this->config->get('config_tax')) {
					$tax = $this->currency->format((float)$result['special'] ? $result['special'] : $result['price'], $this->session->data['currency']);
				} else {
					$tax = false;
				}

Я писав про це на гіті, аде цар - сам по собі.

В 13.11.2024 в 11:48, chukcha сказав:

там проблема в зовсім іншому

 

Ідея передавати product_id  у контролер, а в ньому вже getProduct, краще ніж у моделі getProducts
Або зовсім не $product_data, а краще увесь $result без усяких  попередніх змін, типу цього

 

Та там у Даніеля до фіга таких "нюансів" м'яко кажучи )))
Що таке уніфікація, стандартизація, відсутність фрагментації та відсутність диференціації мабуть він не чув.
Але ідеального в світі нічого не буває... як я кажу - "Навіть на сонці є плями, але воно нас гріє та дає життя"

В 13.11.2024 в 11:49, chukcha сказав:

Я писав про це на гіті, аде цар - сам по собі.

Ти вірно писав, але мабуть до нього не дійшло )))
 

В 13.11.2024 в 11:52, markimax сказав:

Та там у Даніеля до фіга таких "нюансів" м'яко кажучи )))
Що таке уніфікація, стандартизація, відсутність фрагментації та відсутність диференціації мабуть він не чув.
Але ідеального в світі нічого не буває... як я кажу - "Навіть на сонці є плями, але воно нас гріє та дає життя"

а не вiдiйдемо ми тодi від MVC?

13.11.2024 в 14:24, Sha сказал:

а не вiдiйдемо ми тодi від MVC?

ні разу
MVC це не послідовність
до речі у ОС - С - M - V

У OC може бути
С (c-m-v) - M (v) - V

13.11.2024 в 14:24, Sha сказал:

а не вiдiйдемо ми тодi від MVC?

мне кажется наоборот. унификация это основа mvc)

14.11.2024 в 13:53, kJlukOo сказал:

унификация это основа

та нi уніфікацію ні хто не чіпає)

я про спосіб реалізації та ієрархію. Ось якраз про це

13.11.2024 в 14:33, chukcha сказал:

С (c-m-v) - M (v) - V

наскільки це правильно, таке розгалуження...

14.11.2024 в 21:39, Sha сказал:

наскільки це правильно, таке розгалуження...

а що в цьому не так?
наприклад breadcrumbs

а також

header
footer
...