ocmod - зло. мнение

[freelancer]

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

[AlexDW]

есть еще засада с ocmod в ocStore 23023

а именно - с очередностью применения модификаторов по дате их добавления, а не по имени

 

тот кто это учудил - либо тупо не просчитал последствия,

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

 

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

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

 

при сортировке же по дате - полный писец, т.к. при переустановке модулей клиентом вся очередность лечит к черту

 

автор

$results = $this->model_extension_modification->getModifications(array('sort'=>'date_added', 'order'=>'ASC'));

если вдруг внезапно напала икота - знай, что кто-то в очередной раз сражается с твоей гениальностью!!!

[freelancer]

@AlexDW по дате как раз логично

плохо что нет инструмента управления очередью, да.

[chukcha]

22 минуты назад, freelancer сказал:

по дате как раз логично

и где логика?

логично по коду (code), а не по имени
Имя может быть быть в utf8, анужно выполнить в начале
 

[spectre]

3 часа назад, freelancer сказал:

@AlexDW по дате как раз логично

плохо что нет инструмента управления очередью, да.

 

Набросал такой модик на скорую руку

 

sort_order_mod_23.ocmod.xml

 

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

выглядит не очень но работает) ставим цифирки нажимаем рефреш

 

ВАЖНО

надо добавить sort_order в модель, у меня ее не получилось модом поменять

 

admin/model/extension/modification.php

чтоб получилось

 

		$sort_data = array(
			'sort_order',
			'name',
			'author',
			'version',
			'status',
			'date_added'
		);

 

[chukcha]

Не прокатит, т.е. это можно, грубо самостоятельно на проекте,

Но такие вещи надо только в ядро

[spectre]

Я не собираюсь это продавать, решил с коллегами полезным делом поделиться, может кому пригодится, только на этой неделе раза 3 нужен был такой инструмент

можно ж разок отсортировать как надо и потом не трогать или вообще в базе сорт ордер править, это ж удобнее чем дату менять

[chukcha]

дата - это в ocstore
в оригинале по имени.

как вариант на своих проектах менять в ядре.

[chukcha]

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

Я не собираюсь это продавать,

Я не говорил про продажу, я про применение.
 

[spectre]

ну имя тоже не сильно круто как по мне

потом появляются _______mod1, __a_mod_24, z_mod_last и тп

[chukcha]

Только что, spectre сказал:

ну имя тоже не сильно круто как по мне

Согласен
По коду - было бы самый раз

ну, не ___
а яМойКод

яяМойКод

[AlexDW]

8 часов назад, freelancer сказал:

@AlexDW по дате как раз логично

 

10 часов назад, AlexDW сказал:

при сортировке же по дате - полный писец, т.к. при переустановке модулей клиентом вся очередность лечит к черту

 

 

4 часа назад, chukcha сказал:

По коду - было бы самый раз

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

при одинаковых кодах этого просто не случится

 

да и название всегда на виду, а код еще отдельно нужно будет смотреть

[chukcha]

28 минут назад, AlexDW сказал:

да и название всегда на виду, а код еще отдельно нужно будет смотреть

согласен, уже нарвался сегодня, на собсвенный модуль и... Ваш

[AlexDW]

сваял такое

 

@chukcha , удалось решить проблему с изменением model/extension/modification.php ?

если да - то как, если не секрет? хотелось бы более толковое решение чем здесь

[chukcha]

На тот момент - нет..
Нужно было срочно... и.. я ручками в код прописал нужное..
Давно это было.. Работает 2 года, чего его трогать?