Вопрос по установке модификаторов через задницу

[fromtheroof]

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

А теперь вводная часть вопроса:

Есть модуль, исправляющий стандартную логику фильтров движка.

Это FixFilter OC2x от AlexDW. Всё прекрасно ставится и логика фиксится.

Но! При этом слетают изменения, которые я своими кривыми ручками до этого сделал напрямую в файлах.

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

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

Открываю в редакторе файл fix_filter.ocmod.xml и вижу, что он в файле catalog/model/catalog/product.php ищет строку

$sql .= " AND pf.filter_id IN (" . implode(',', $implode) . ")";

и вставляет после неё код

// FixFilter
 ...бла бла бла...
// FixFilter END

А теперь вопрос:

В файле product.php искомая строка находится два раза!
Мне нужно после каждого случая вставлять код "бла бла бла" или только после первого или только после второго?
А?

Змінено 1 липня 2018 користувачем fromtheroof

[johser]

Если атрибут index не стоит, то после каждого. Ну или найти и посмотреть измененный файл в папке system/storage/modification

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

Змінено 1 липня 2018 користувачем johser

[AlexDW]

  В 01.07.2018 в 07:32, fromtheroof сказав:

В файле product.php искомая строка находится два раза!
Мне нужно после каждого случая вставлять код "бла бла бла" или только после первого или только после второго?

Expand  

 

если внимательно глянуть модификатор, то видно что условие <search> не содержит index

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

конкретно в данном случае - дважды

[fanatic]

del

p.s   опередили

 

[SiteMix]

  В 01.07.2018 в 07:32, fromtheroof сказав:

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

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

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

Expand  

 

Правильней, всё же, воспользоваться советом johser

 

  В 01.07.2018 в 07:55, johser сказав:

написать свой модификатор, чтобы каждый раз не переписывать код, при установке модификатора, затрагивающего код Ваших файлов.

Expand  

 

Найти свои правки можно сравнив оригинальный файл шаблона с нынешним файлом (до внедрения кода для фикса фильтра).

[fromtheroof]

  В 01.07.2018 в 07:55, johser сказав:

проще написать свой модификатор

Expand  

Это резонно. Но проблема в том, что я уже не помню какие файлы допиливал напильником. И даже не помню всех задач, которые решал.

 

  В 01.07.2018 в 07:57, AlexDW сказав:

конкретно в данном случае - дважды

Expand  

спасибо за ответ от автора модуля

 

  В 01.07.2018 в 09:48, SiteMix сказав:

Найти свои правки можно сравнив оригинальный файл шаблона с нынешним файлом

Expand  

Спасибо. Займусь на досуге.

 

  В 01.07.2018 в 08:03, fanatic сказав:

p.s   опередили

Expand  

спасибо тебе, тоже

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