[sinneren]

UMI.CMS Commerce
Версия системы   2.9.1
Ревизия   2215

Тип данных "новость" в "новостная лента".
Устанавлиаю Дату публикации, начала и окончания активности. Но материал в лучшем случае не скрывается, в худшем - даже не появляется. Активность включалась на RUS версии, на ENG даже это не работает.
В чём может быть причина. Утверждают, что ранее это работало. С тех пор обновлений не было и не могло быть ввиду окончания срока лицензии.

Настройки макроса: %news lastlist(/actions/,npc_index_actions,3,1,0,1)%

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

UPD: До этого контроль актуальности не был включен, уверяли, что работало и так. С ним на публикацию не отправляет, но иногда снимает,а иногда нет.
UPD2:Крон стоит на 59мин. Менял на шустрее, не вышло. Главное, чтоб включал новости.

[admin]

механизм простой.

1. по крону запускается event

Код: [Выделить]

$onCronActivateNews = new umiEventListener("cron", "news", "cronActivateNews");

проверьте его в файле /classes/modules/news/events.php

2. данный event выполняет функцию в файле /classes/modules/news/class.php

Код: [Выделить]

public function cronActivateNews() {
		$pages = new selector('pages');
		$pages->types('hierarchy-type')->name('news', 'item');
		$pages->where('is_active')->notequals(true);
		$pages->where('begin_time')->eqless(time());
		foreach ($pages as $page){
			$page->setIsActive(true);
			$page->commit();
		}
	}

Её тоже проверьте

P.S. попробуй запустить вручную cron события через http://site.ru/cron.php

Возможно, есть какой-то еще event, который возвращает fatal errore и не дает выполниться тому, что идет после него. То есть, посмотри что перед твоим event'ом и попробуй отключить эти event (просто найди их в файлах системы и закомментируй). Если при ручном запуске http://site.ru/cron.php станут активные новости? которые были выключены, но имели begin_time меньше текущего времени, значит что-то с запуском cron

[sinneren]

1.

Код: [Выделить]

	
$onCronNewsRead = new umiEventListener("cron", "news", "feedsImportListener");
$onCronActivateNews = new umiEventListener("cron", "news", "cronActivateNews");
new umiEventListener("cron", "news", "cronDeactivateNews");

2.Немного отличается

Код: [Выделить]

public function cronActivateNews() {
		$pages = new selector('pages');
		$pages->types('hierarchy-type')->name('news', 'item');
		$pages->where('is_active')->notequals(true);
		$pages->where('begin_time')->eqless(time());
		foreach ($pages as $page){
			
			
			if(strtotime($page->getValue("end_time")) <= time()){
				$page->setIsActive(false);
			} else {
				$page->setIsActive(true);
			}
			
			$page->commit();
		}
	}
	
	public function cronDeactivateNews(){
		$pages = new selector('pages');
		$pages->types('hierarchy-type')->name('news', 'item');
		$pages->where('end_time')->eqless(time());
	
		foreach ($pages as $page){
			$page->setValue("test",date("h:i:s"));
			$page->setIsActive(false);
			$page->commit();
		}
	}

[sinneren]

Ответ ручного запуска крон. Новость не появилась.
Executed event handlers:
   1. news::cronDeactivateNews (umiEventPoint), priority = 5, not critial
   2. news::feedsImportListener (umiEventPoint), priority = 5, not critial
   3. news::cronActivateNews (umiEventPoint), priority = 5, not critial
   4. news::cronDeactivateNews (umiEventPoint), priority = 5, not critial
   5. dispatches::onAutosendDispathes (umiEventPoint), priority = 5, not critial
   6. emarket::onCronSyncCurrency (umiEventPoint), priority = 5, not critial
   7. config::runGarbageCollector (umiEventPoint), priority = 5, not critial

[sinneren]

Опробовал твой код активации, с режимом контроля публикаций и без. Запускал вручную. Эффекта ноль.

[admin]

попробуй отрубить cron

1. news::cronDeactivateNews (umiEventPoint), priority = 5, not critial
   2. news::feedsImportListener (umiEventPoint), priority = 5, not critial

если они не глушат (то есть не ломают что-то), то смотри дело остается только в макросе) 

P.S. еще у тебя почему-то 2 раза запускается cronDeactivateNews , причем второй после cronActivateNews , может он его перекрывает))

[sinneren]

А где-то есть еще файл, где пишутся кастомные обработчики, вроде как у битрикса init.php?
В файле class.php один раз объявлена cronDeactivateNews и нигде и ничто к ней не ссылается там.

Код: [Выделить]

$pages = new selector('pages');
		$pages->types('hierarchy-type')->name('news', 'item');
		$pages->where('end_time')->eqless(time());
	
		foreach ($pages as $page){
			$page->setValue("test",date("h:i:s"));
			$page->setIsActive(false);
			$page->commit();
		}

Можно полный код стандартного файла получить?

[admin]

у меня нет макроса cronDeactivateNews  и нет eventlistner который вызывал бы такой макрос (версия 2.9.x)

попробуйте отключить строчку

Код: [Выделить]

new umiEventListener("cron", "news", "cronDeactivateNews");

в файлу events.php и снова запустить cron.php

P.S. Это конечно вряд ли, но проверьте у той новости, включение которой вы ожидаете, содержимое полей begin_time и end_time. Не противоречат ли они друг другу? Если можете, приведите сюда их значение.

[sinneren]

Executed event handlers:
   1. news::cronDeactivateNews (umiEventPoint), priority = 5, not critial
   2. news::feedsImportListener (umiEventPoint), priority = 5, not critial
   3. news::cronActivateNews (umiEventPoint), priority = 5, not critial
   4. dispatches::onAutosendDispathes (umiEventPoint), priority = 5, not critial
   5. emarket::onCronSyncCurrency (umiEventPoint), priority = 5, not critial
   6. config::runGarbageCollector (umiEventPoint), priority = 5, not critial

Код: [Выделить]

<?php
	$onCronNewsRead = new umiEventListener("cron", "news", "feedsImportListener");
	$onCronActivateNews = new umiEventListener("cron", "news", "cronActivateNews");
	//new umiEventListener("cron", "news", "cronDeactivateNews");
?>

Дата публикации - 2014-09-02 15:38:00
Дата начала активности - 2014-09-02 15:46:00
Дата окончания активности - 2014-09-02 15:49:00

[sinneren]

Добавление новости, может что тут

Код: [Выделить]

<?php
public function addNewsItem() {
		$hierarchy = umiHierarchy::getInstance();
		$hierarchyTypes = umiHierarchyTypesCollection::getInstance();
		$objectTypes = umiObjectTypesCollection::getInstance();
		$cmsController = cmsController::getInstance();

		$parent_id = (int) getRequest('param0');
		$object_type_id = (int) getRequest('param1');
		$title = htmlspecialchars(trim(getRequest('title')));

		$parentElement = $hierarchy->getElement($parent_id);
		$tpl_id		= $parentElement->getTplId();
		$domain_id	= $parentElement->getDomainId();
		$lang_id	= $parentElement->getLangId();

		$hierarchy_type_id = $hierarchyTypes->getTypeByName("news", "item")->getId();
		if(!$object_type_id) {
			$object_type_id = $objectTypes->getBaseType("news", "item");
		}

		$object_type = $objectTypes->getType($object_type_id);
		if($object_type->getHierarchyTypeId() != $hierarchy_type_id) {
			$this->errorNewMessage("Object type and hierarchy type doesn't match");
			$this->errorPanic();
		}

		$element_id = $hierarchy->addElement($parent_id, $hierarchy_type_id, $title, $title, $object_type_id, $domain_id, $lang_id, $tpl_id);

		$users = $cmsController->getModule("users");
		if($users instanceof def_module) {
			$users->setDefaultPermissions($element_id);
		}

		$element = $hierarchy->getElement($element_id, true);

		$element->setIsActive(true);
		$element->setIsVisible(false);
		$element->setName($title);

		$element->commit();
		$parentElement->setUpdateTime(time());
		$parentElement->commit();

		if($is_xslt) {
			return Array("node:result" => "ok");
		} else {
			$this->redirect($referer_url);
		}
	}
?>

[sinneren]

есть еще custom_events.php с
new umiEventListener("cron", "news", "cronDeactivateNews");
может потому дважды в отчете крона идет

[admin]

Добавление новости, может что тут

Код: [Выделить]

<?php
public function addNewsItem() {
		$hierarchy = umiHierarchy::getInstance();
		$hierarchyTypes = umiHierarchyTypesCollection::getInstance();
		$objectTypes = umiObjectTypesCollection::getInstance();
		$cmsController = cmsController::getInstance();

		$parent_id = (int) getRequest('param0');
		$object_type_id = (int) getRequest('param1');
		$title = htmlspecialchars(trim(getRequest('title')));

		$parentElement = $hierarchy->getElement($parent_id);
		$tpl_id		= $parentElement->getTplId();
		$domain_id	= $parentElement->getDomainId();
		$lang_id	= $parentElement->getLangId();

		$hierarchy_type_id = $hierarchyTypes->getTypeByName("news", "item")->getId();
		if(!$object_type_id) {
			$object_type_id = $objectTypes->getBaseType("news", "item");
		}

		$object_type = $objectTypes->getType($object_type_id);
		if($object_type->getHierarchyTypeId() != $hierarchy_type_id) {
			$this->errorNewMessage("Object type and hierarchy type doesn't match");
			$this->errorPanic();
		}

		$element_id = $hierarchy->addElement($parent_id, $hierarchy_type_id, $title, $title, $object_type_id, $domain_id, $lang_id, $tpl_id);

		$users = $cmsController->getModule("users");
		if($users instanceof def_module) {
			$users->setDefaultPermissions($element_id);
		}

		$element = $hierarchy->getElement($element_id, true);

		$element->setIsActive(true);
		$element->setIsVisible(false);
		$element->setName($title);

		$element->commit();
		$parentElement->setUpdateTime(time());
		$parentElement->commit();

		if($is_xslt) {
			return Array("node:result" => "ok");
		} else {
			$this->redirect($referer_url);
		}
	}
?>

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

Executed event handlers:
   1. news::cronDeactivateNews (umiEventPoint), priority = 5, not critial
   2. news::feedsImportListener (umiEventPoint), priority = 5, not critial
   3. news::cronActivateNews (umiEventPoint), priority = 5, not critial
   4. dispatches::onAutosendDispathes (umiEventPoint), priority = 5, not critial
   5. emarket::onCronSyncCurrency (umiEventPoint), priority = 5, not critial
   6. config::runGarbageCollector (umiEventPoint), priority = 5, not critial

Код: [Выделить]

<?php
	$onCronNewsRead = new umiEventListener("cron", "news", "feedsImportListener");
	$onCronActivateNews = new umiEventListener("cron", "news", "cronActivateNews");
	//new umiEventListener("cron", "news", "cronDeactivateNews");
?>

Дата публикации - 2014-09-02 15:38:00
Дата начала активности - 2014-09-02 15:46:00
Дата окончания активности - 2014-09-02 15:49:00

я правильно понял, что вы вначале заккоментировали event в файле events.php (код которого вы привели 2 блоком)
потом вы запустили cron.php (код которого вы привели 1 блоком)

при этом строчка

1. news::cronDeactivateNews (umiEventPoint), priority = 5, not critial

показывает что у вас где-то еще срабатывает event деактивации новости.

Если яправильно понял ваше сообщение, то поищите по другим модулям в файлах events.php или custom_event.php

Второй момент, у вас поставлен очень маленький диапазон времени между begin и end time, а что есть time() в функции активации новости, возвращает время сервера, у которого другая time_zone (особенно если хостинг за раничный). Проверьте, что возвращает вам php функция time(). Может стоить скорректировать begin_time?

[sinneren]

Так... по порядку. Код добавления я привел просто кстати, чтоб посмотерть может там какая лишняя обработка.

Сначала я закомментировал umiEventListener,потом вручную запустил крон.пхп.

Строчка news::cronDeactivateNews исчезла,когда закомментировал строку в файле custom_event.php

[sinneren]

Текущее время сервера 16:14:54 (1409660094). Реальное совпадает.

[admin]

просто ради интереса, запустите udata://news/cronActivateNews на вашем сайте. чтобы без всякого крона убедиться работает ли активация новости, а потом уже копать в сторону крона или events