Jump to content
Sign in to follow this  
chukcha

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

Recommended Posts

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

 

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

 

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

 

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

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

 

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

 

 

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

 

Share this post


Link to post
Share on other sites

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

 

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

 

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

 

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

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

 

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

 

 

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

 

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

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

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

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

 

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

Share this post


Link to post
Share on other sites

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

Share this post


Link to post
Share on other sites

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

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

 

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

Share this post


Link to post
Share on other sites
Шаблон не должен делать никаких вычисление и должен содержать минимум логики,

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

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

 

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

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

Share this post


Link to post
Share on other sites
Шаблон не должен делать никаких вычисление и должен содержать минимум логики,
Тут очень спорный вопрос....Логика, вычисления

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

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

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

 

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

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

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

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

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

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

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

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

Share this post


Link to post
Share on other sites

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

 

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

 

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

 

Т.е.

 

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

 

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

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

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
You are posting as a guest. If you have an account, please sign in.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this  

  • Recently Browsing   0 members

    No registered users viewing this page.

×

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.