Forumlar / National / Russian / Модули и плагины / Новый хук для pages

cot_page_import

Macik
#41898 2016-07-06 23:25

Есть несколько вопросов и замечаний...

  1. Хочу услышать подробней про сценарий использования, что будет вызываться по хуку (имею в виду алгоритм той части что будет вызвана в конкретно вашем случае). Потому, как изменять импортированные данные или добавлять дополнительные в `$rpage` можно и во вне функции.
  2. Слово `query ` здесь в названии хука лишнее, и используется в случаях когда хук призван изменять значения строк составляющих будущий SQL запрос. Логичнее назвать его `page.import.main` по аналогии с другими частями. Или если четко следовать букве правил именования, то `page.functions.import.main`
  3. Мое мнение таково, что расширение функций хуками не очень хорошая практика (не смотря на то, что много где используется в нашем движке). Поясню — функция как программный блок и часть API  должна быть максимально понятной, атомарной, детерминированной. Если мы внедряем внутрь хук мы теряем эти свойства. Плюс несколько усложняем отладку.

 

 

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F