Tickets introtext и Jevix

Я так и не понял баг это или фича. Tickets сохраняет content, обработанный Jevix, в introtext. Делает это даже когда Jevix отключен. Чтоб избавиться от этого нужно отредактировать 2 файла: create.class.php и update.class.php по адресу /core/components/tickets/controllers/ticket/.

Я заменил:

// Define introtext
$introtext = $this->getProperty('introtext');
if (empty($introtext)) {
$introtext = $this->object->getIntroText($this->getProperty('content'), false);
}
if (empty($properties['disable_jevix'])) {
$introtext = $this->object->Jevix($introtext);
}

На:

// Define introtext
$introtext = $this->getProperty('introtext');
// if (empty($introtext)) {
// $introtext = $this->object->getIntroText($this->getProperty('content'), false);
// }
if (empty($properties['disable_jevix'])) {
$content2intro = $this->object->getIntroText($this->getProperty('content'), false);
$introtext = $this->object->Jevix($content2intro);
}

Теперь content в introtext сохраняется только если Jevix включен. Вроде как так и должно быть. Нет?

Денис Дыранов

01 февраля 2016, 09:06

4 311

Комментарии: 21

Сергей Шлоков

01 февраля 2016, 12:34

Ну если заходить с позиции «Для настоящих пацанов Страдивари делал барабаны», то сойдёт. Возможно придется ещё много чего переделывать, ибо проблем из-за Jevix море. Даже если его отключить, письма всё-равно отправляются, комментарии пишутся, просмотры считаются, рейтинги не отключаются и т.д.
Какая связь между introtext и Jevix?

Денис Дыранов

01 февраля 2016, 12:54

Не надо преувеличивать. Связь описана.

Если включена обработка Jevix и introtext пуст, то в него попадает обработанный content для вывода на страницах сайта в списке тикета.

Проблема в том, что content туда попадает, даже если jevix выключен. В результате не работает такая конструкция при отображении списка тикетов:

[[+introtext:isnot=``:then=`
[[+introtext]]
<a href="[[+link]]">Читать дальше</a>
`:else=`[[+content]]`]]

Сергей Шлоков

01 февраля 2016, 13:09

Не надо преувеличивать. Связь описана.

Где это можно прочитать?

Зачем в списке тикетов выводить контент тикета?

Денис Дыранов

01 февраля 2016, 13:11

Здесь: github.com/bezumkin/Tickets/issues/96

Если статья короткая, я хочу выводить её полностью без «Читать дальше». Кат тоже не катит, потому что краткое описание статьи может отличаться от её начала.

Сергей Шлоков

01 февраля 2016, 13:27

Автор пишет про пустой introtext и cut.

Не используй тег cut и пропиши свой introtext.

Лично я в этом

Если включена обработка Jevix и introtext пуст, то в него попадает обработанный content для вывода на страницах сайта в списке тикета.

не вижу связи. Если Jevix включен, то попадет обработанный контент. Если отключен, то необработанный. Что и логично.

Выход.

Не используй тег cut и пропиши свой introtext.

Денис Дыранов

01 февраля 2016, 13:31

Что логичного в дублировании контента? Вот в ветке ниже я об этом же.

Сергей Шлоков

06 февраля 2016, 15:08

В чём проблема-то с этим дублированием? Не по феншую или mySql теперь такие записи в выдаче через SELECT понижает?
Изначально сниппеты getResources и pdoResources (да и некоторые другие) не подгружают поле content без специального указания (&includeContent=`1`), чтобы не раздувать память.

Пётр Молчанов

01 февраля 2016, 12:36

так и задумано

Денис Дыранов

01 февраля 2016, 13:04

Не уверен github.com/bezumkin/Tickets/issues/64

Пётр Молчанов

01 февраля 2016, 13:11

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

Денис Дыранов

01 февраля 2016, 13:14

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

Пётр Молчанов

01 февраля 2016, 13:20

нет. интротекст — это краткая инфа статьи, контент — полная инфа статьи.
на примере новостей: главная новостей, где выводится список всех новостей, у каждой новости выводится интротекст, а при подробном просмотре — уже контент.
так, вроде, во всех CMS сделано

Денис Дыранов

01 февраля 2016, 13:27

Спасибо, кэп! :-) Но проблема в том, что мне в краткое описание статьи, попадает её полное содержание. И в БД у меня каждая статья дублируется. Вопрос: зачем?

Пётр Молчанов

01 февраля 2016, 13:28

предполагается, что ты будешь заполнять интротекст

Денис Дыранов

01 февраля 2016, 13:38

…и что мне плевать на дубли в БД.

Я думаю, предполагалось, что нужно где-то хранить результат работы джевикса. И интротекст для этого вполне подошел. Но если у джевикса нет результатов работы, соответственно и нет смысла хранить контент 2 раза.

Пётр Молчанов

01 февраля 2016, 13:56

я вот заполняю интротекст каждый раз и у меня всё норм. что мешает делать также? в админке это поле даже так и называется «Аннотация (введение)»

Денис Дыранов

01 февраля 2016, 14:20

Интротекст — чаще всего используется как подобие ката. В большинстве случаев кат — зло. Если у меня статья не на 10 экранов с кучей фотографий, кат мне не нужен. Потому как чтение блога превращается в чтение огрызков статей. Плюс открытие полных версий в куче табов. Однако, в ряде случаев он необходим.

Если я хочу отобразить статью полностью, то нет смысла забивать интротекст.

Конечно, можно придумать кучу решений этой проблемы. Отображать ссылку на полную версию, если интротекст=контенту или если в интротексте лежит какой-то текст-триггер, но это тоже костыли. И вообще, есть вероятность, что вся эта проблема — просто следствие недоработки.

Пётр Молчанов

01 февраля 2016, 14:44

ок, у каждого своя ситуация) как вариант решения без влезания в код дополнения, могу предложить на событие OnDocFormSave делать проверку, либо ставить интротекст пустым принудительно

Денис Дыранов

01 февраля 2016, 14:53

Спасибо за ещё один вариант. Конечно, влезать в код дополнения — плохая идея. Но я надеялся привлечь этим Василия к обсуждению. Похоже, зря :-)

Денис Дыранов

01 февраля 2016, 16:24

Похоже что всё-таки на OnDocFormRender. У меня это, вроде как, работает, может ещё кому пригодится:

<?php
switch ($modx->event->name) {
    case 'OnDocFormRender';
	$content = $resource->getContent();
	$introtext = $resource->get('introtext');
	if ($introtext = $content) {
      		$resource->set('introtext','');
		$resource->save();
   	}
    break;
}

Илья Уткин

14 марта 2016, 17:06

Немного дополнил плагин, так как не всегда $introtext = $content

if ($modx->event->name == "OnDocFormSave") {
    $introtext = $resource->get('introtext');
    $generateIntrotext = $resource->getIntroText($resource->get('content'), false);
    if (empty($resource->getProperty('disable_jevix'))) {
        $generateIntrotext = $resource->Jevix($generateIntrotext);
    }
    if ($introtext == $generateIntrotext) {
        $resource->set('introtext', '');
    $resource->save();
    }
}

Авторизуйтесь или зарегистрируйтесь, чтобы оставлять комментарии.

Tickets introtext и Jevix

Комментарии: 21

Здесь упомянуты