Масштабирование изображений (tools->resize)

[chukcha]

Сейчас масштабирование изображений происходит в контроллере.

 

Но с точки зрения MVC, даже опенкартовского, это не совсем верно.

 

Для одного шаблона нужно один размер, для другого - другой.

 

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

Настройки шаблона - для каждого шаблона свой "модуль"  в админку.

 

Тогда можно и масштабировать в контроллере.

 

 

зы.. чувствую, что так все и останется. одношаблонным.

 

[markimax]

Сейчас масштабирование изображений происходит в контроллере.

 

Но с точки зрения MVC, даже опенкартовского, это не совсем верно.

 

Для одного шаблона нужно один размер, для другого - другой.

 

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

Настройки шаблона - для каждого шаблона свой "модуль"  в админку.

 

Тогда можно и масштабировать в контроллере.

 

 

зы.. чувствую, что так все и останется. одношаблонным.

 

Совершенно верно заметили.

Я в своем модуле исправил это недоразумение

У меня в модуле настройки resize есть как в категории, так и в виджетах (модуль с настройками в модуле)

Точно также как есть и подключаемые пользовательские шаблоны.

 

Это более правильная реализация MVC, где контроллер не должен знать о выводе (view).

[snastik]

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

[sv2109]

MVC здесь не причем, с точки зрения MVC все сделано правильно - масштабирование должно происходить в модели, где оно собственно и происходит (ModelToolImage->resize())

Контроллер отвечает за логку - дает указание модели какое изображение до какого размера нужно масштабировать после чего отдает готовые результат шаблону. 
Шаблон не должен делать никаких вычисление и должен содержать минимум логики, он должен выводить готовый результат, что он и делает - выводит готовое масштабированное изображение, переданное ему контроллером. 

 

Проблема как я понял в настройках, передачи настроек в контроллер не из стандартных настроек опенкарта, а чтобы для каждой темы были свои настройки для размера изображений? Ну так это проблема самого опенкарта, а не MVC. 

[chukcha]

Шаблон не должен делать никаких вычисление и должен содержать минимум логики,

Тут очень спорный вопрос....Логика, вычисления

Логика и вычисления должны быть понятны "верстальщику". Вот на таком уровне.

 

А я не говорю о проблеме опенкартовcкой MVC,  я сказал, , что по сути, настройки, параметры темы, должны быть в настройках, параметрах темы.

Но так как это не исправится никогда :) , то тема - чтоб поговорить.

[sv2109]

Шаблон не должен делать никаких вычисление и должен содержать минимум логики,

Тут очень спорный вопрос....Логика, вычисления

Логика и вычисления должны быть понятны "верстальщику". Вот на таком уровне.

Никаких "спорных вопросов": модель должна отвечать за вычисления, контролер за логику, шаблон за отображение готовых результатов.

В шаблоне может присутствовать логика, но очень простая например условные операторы if, циклы foreach итд. Никакой сложной логики и тем более никаких вычислений в шаблоне быть не должно, для этого есть контроллер и модель. Это MVC.

 

А я не говорю о проблеме опенкартовcкой MVC,  я сказал, , что по сути, настройки, параметры темы, должны быть в настройках, параметрах темы.

Но так как это не исправится никогда :) , то тема - чтоб поговорить.

В своем первом после вы написали:

"Сейчас масштабирование изображений происходит в контроллере.

Но с точки зрения MVC, даже опенкартовского, это не совсем верно."

То есть вы как раз таки и говорите о "проблеме опенкартовского MVC"

Я же вам говорю, что никакой проблемы c MVC здесь нет, есть проблема с настройками для темы, но это вообще никак не относится к MVC.

Точная формулировка проблемы это первый шаг для ее решения. Если же вы в постановке проблемы путаетесь в основных понятиях то так вы эту проблему не решите.

[chukcha]

Масштабирование изображения - это логика шаблона.

 

Потому и должно принадлежать шаблону.

 

Если не самому шаблонизатору, то настройкам шаблона

 

Т.е.

 

init шаблона, должен переписать системные дефолтные" настройки"

 

Но init'а у шаблона нет.

Вот и получается, что - одношаблонная система.