Как должен выглядеть хук?

Истинно православный хук. Какой он?

#1 30 апреля 2016 в 18:41

Почитал про инстантправославие. Проникся идеей выноса всех изменений в отдельные файлы и вызов их по хуку.
Теперь хочу выяснить, как это должно выглядеть с точки зрения создателей CMS.
Допустим, возникла потребность в создании хитрого датасета, который никак средствами стандартной модели не сделать. Или потребовалось в определенном типе контента выводить комментарии в другом порядке и в другом шаблоне.
Как в таком случае активировать вышеупомянутый хук?
Просто в системный файл вписывать условие, что если if(something){$this->runExternalHook(filehook, $parametrs -> $tarametrs) ?
Файл с хуком располагается в месте filehook и в нем содержится класс (или не содержится?) и функция, которую активирует данный хук.
Как все это правильно должно выглядеть?

#2 30 апреля 2016 в 18:56

Теперь хочу выяснить, как это должно выглядеть с точки зрения создателей CMS.
HiAndy

Нужно в первую очередь прочитать инстант-православную документацию.

Допустим, возникла потребность в создании хитрого датасета, который никак средствами стандартной модели не сделать.
HiAndy

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

Если в движке нужного вам хука не заложено, то необходимо:

1. Проанализировать, куда наиболее функционально вставить вызов хука, что ему передать и как его назвать максимально универсально, не прицепляясь к вашей идее.
2. Добавить вызов хука в код последовательной или параллельной схемой вызова (см. документацию)
3. Написать нам с предложением внести хук в движок или же прислать пулреквест на гитхабе.

Если по документации что-то непонятно — спрашивайте уточняющие вопросы.

🛠 docs.instantcms.ru

#3 30 апреля 2016 в 19:37

1. Проверить, нет ли уже в движке хука на нужную вам правку функционала.
Fuze

Нет, этого нет и быть не может. Это кроме меня вряд ли кому пригодится.
Для обеспечения спецолимпиады на сайте потребовалось создать в таблице рейтинга новую колонку votedate (время голосования).
После этого создал датасеты "лучшие за день" и "лучшие за вчерашний день". И важно, что именно не дата публикации, а сумма рейтинга проголосовавших за отведенное время имеет значение…
Собственно все работает, но пришлось добавить новую функцию в модель контента:

Спойлер

      public function getBest($ctype_name, $dataset)
	  {
		 if ($dataset == 'bestweek') {if (date("H")<20)
        {
        $dfrom = date("Y-m-d H:i:s",mktime(20, 0, 0, date("m")  , date("d")-9, date("Y")));
        $dto =  date("Y-m-d H:i:s",mktime(19, 59, 59, date("m")  , date("d"), date("Y")));
        }
        else
        {
        $dfrom = date("Y-m-d H:i:s",mktime(20, 0, 0, date("m")  , date("d")-7, date("Y")));
        $dto =  date("Y-m-d H:i:s",mktime(19, 59, 59, date("m")  , date("d")+1, date("Y")));
        } 
 
		  $sql = "SELECT target_id, SUM(score) as points FROM cms_rating_log WHERE target_subject = 'anec' AND votedate BETWEEN '$dfrom' AND NOW() 
		   GROUP BY target_id ORDER BY points DESC LIMIT 0, 15";
		 } else {
		 if (date("H")<20)
{
$dfrom = date("Y-m-d H:i:s",mktime(20, 0, 0, date("m")  , date("d")-3, date("Y")));
 
$dto =  date("Y-m-d H:i:s",mktime(19, 59, 59, date("m")  , date("d")-1, date("Y")));
}
else
{
$dfrom = date("Y-m-d H:i:s",mktime(20, 0, 0, date("m")  , date("d")-1, date("Y")));
$dto =  date("Y-m-d H:i:s",mktime(19, 59, 59, date("m")  , date("d"), date("Y")));
}
$sql="Select  target_id, sum(score) as points From cms_rating_log  Where votedate Between '$dfrom' And '$dto' Group by target_id Order by points DESC LIMIT 15";
	  }
	    $items = array();
		$arr = $this->db->query($sql);
		while ($joke = mysqli_fetch_assoc($arr)){
		global $points;
		$id = $joke['target_id'];
        $points = $joke['points'];		
		$table_name = $this->table_prefix . $ctype_name;
 
        $this->select('u.nickname', 'user_nickname');
 
        $this->join('cms_users', 'u', 'u.id = i.user_id');
 
        $this->useCache("content.item.{$ctype_name}");
 
        $item = $this->getItemByField($table_name, 'id', $id, function($item, $model){
            global $points;
            $item['user'] = array(
                'id' => $item['user_id'],
                'nickname' => $item['user_nickname']
            );
			$item['rating'] = $points;
 
 
            return $item;
 
        }, 'id');
        $items[] = $item;
		unset ($points);
    }
 
    return $items;
 
        }
 
}

И условие во фронтенд:

Спойлер

		if ($dataset == 'bestyesterday' || $dataset == 'besttoday'){
		$items = $this->model->getBest($ctype['name'], $dataset);
		$total = 15;} else {	
        $total = $this->model->getContentItemsCount($ctype['name']);
        $items = $this->model->getContentItems($ctype['name']);
		}

И вот мне бы хотелось всю эту красоту из системный файлов вынести.

Сегодня в 17:37

#4 30 апреля 2016 в 19:55

HiAndy, у вас во фронтенде стоит уже два хука перед самыми вашими правками

 
		list($ctype, $this->model) = cmsEventsManager::hook('content_list_filter', array($ctype, $this->model));
		list($ctype, $this->model) = cmsEventsManager::hook("content_{$ctype['name']}_list_filter", array($ctype, $this->model));

Создайте файл этого хука, пропишите его в манифесте и делайте с моделью что хотите.
в начале файла можете прописать ваше условие:

 
<?php
 
class onВашкомпонентContentListFilter extends cmsAction {
 
	public function run($data){
 
list($ctype, $model) = $data;
 
if ($dataset == 'bestyesterday' || $dataset == 'besttoday'){ //  $dataset можно получить из реквеста
 
$model->orderBy('the_best_of_the_best'); // или что-то типа того
}
return array($ctype, $model);

Нет, этого нет и быть не может. Это кроме меня вряд ли кому пригодится.
HiAndy

Fuze имел ввиду что добавить строчку типа этой cmsEventsManager::hook('content_list_filter', array($ctype, $this->model)); в нужном месте, а не всю вашу писанину из под первого спойлера.

#5 30 апреля 2016 в 23:37

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

Но зачем мне тогда городить огород с хуками, если с моделью делать что хочу?
Я и так просто написал условие во фронтенде и допфункцию в модели. У меня и так всё работает.

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

#6 30 апреля 2016 в 23:51

если с моделью делать что хочу?
HiAndy

Вы не поняли. В хуке и выполняйте свои действия. Хук реализуется от контроллера. Т.е. чтобы сделать хук, нужен контроллер, его обрабатывающий. А к контроллеру модель своя (если нужна).
Соответственно в своей модели вы реализуете метод getBest, а в хуке дополните уже существующие записи своими данными.

($joke = mysqli_fetch_assoc($arr)){
HiAndy

Так делать не нужно. Используйте методы класса БД или модели.

Вообще в целом ваш код будет выполняться долго и весь подход в корне неверный — все перемешано, новый список записей получаете по одной, еще и перезаписывая то, что уже получено. Вам нужно использовать хук content_ВАШТ_ТИП_КОНТЕНТА_before_list, в нем для переданных id получить недостающие данные через конструкцию IN mysql и полученные данные записать в массив.

global лучше не использовать, сделать так:

        $item = $this->getItemByField($table_name, 'id', $id, function($item, $model) use($points){
            $item['user'] = array(
..........

🛠 docs.instantcms.ru

#7 1 мая 2016 в 00:01

Спасибо, теперь почти понял.

Так делать не нужно. Используйте методы класса БД или модели.
Fuze

Это тяжелое наследие темного прошлого. Эта функция в данный момент работает на сайте под InstantCMS 1.9. Пять лет работает.

global лучше не использовать, сделать так:
Fuze

Завтра попробую. Еще раз спасибо за помощь!

Приходят ошибки на почту

Как в компонент принять параме

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

Как должен выглядеть хук?

Похожие темы

подменю в хуке меню

[ЕСТЬ РЕШЕНИЕ] как получить id открытого профиля в своём хуке ?

Азбука вебмастера

Мобильный шаблон, каким он должен быть?

Магазин цифровых товаров, каким должен быть?

Функционал простого интернет магазина. Каким он должен быть?

InstantCMS Team

О проекте

Поддержка

Дополнения