chukcha

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

Рекомендуемые сообщения

chukcha    822

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

 

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

 

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

 

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

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

 

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

 

 

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

 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
markimax    1 379

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

 

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

 

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

 

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

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

 

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

 

 

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

 

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

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

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

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

 

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
snastik    736

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
sv2109    420

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

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

 

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
chukcha    822
Шаблон не должен делать никаких вычисление и должен содержать минимум логики,

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

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

 

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
sv2109    420
Шаблон не должен делать никаких вычисление и должен содержать минимум логики,
Тут очень спорный вопрос....Логика, вычисления

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

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

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

 

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

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

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

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

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

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

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
chukcha    822

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

 

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

 

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

 

Т.е.

 

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

 

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Для публикации сообщений создайте учётную запись или авторизуйтесь

Вы должны быть пользователем, чтобы оставить комментарий

Создать учетную запись

Зарегистрируйте новую учётную запись в нашем сообществе. Это очень просто!

Регистрация нового пользователя

Войти

Уже есть аккаунт? Войти в систему.

Войти


  • Последние посетители   0 пользователей онлайн

    Ни одного зарегистрированного пользователя не просматривает данную страницу