Tickets 1.9.1-pl

Внезапно вышло обновление Tickets!

Так как я давно не делал про него анонсов, тот вот некоторые интересные изменения последних версий:
— Тег kbd добавлен в список разрешенных по умолчанию.
— Исправлена модель компонента для работы в PHP 7.2
— Запись TicketTotal корректно удаляется вместе с Ticket
— Календарь в админке слушает настройку manager_week_start
— Евгений Борисов кое-где закрыл возможные SQL инъекции
— Обновление пункта меню в админке с id на namespace, чтобы не было возмущений в MODX 2.7 про древний modAction.

Ну и самое главное — в новой версии можно писать комментарии прямо из админки. Все благодарности за это уходят пользователю GitHub tolanych.

 

Теперь вы можете общаться со своими посетителями прямо из админки. Конечно, там нет редактора и специальных настроек, но это всё равно гораздо лучше, чем было.

Обновляемся, чистим кэш и проверяем.
Василий Наумкин
30 ноября 2018, 07:30
3
429
+17

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

Sem
Sem
30 ноября 2018, 08:34
0
Приятно видеть обновления компонентов, которые очень редко обновляются.
Жаль, что баг с галочкой очистить кэш у разделов с тикетами по-прежнему жив — yadi.sk/i/09s8dKOh3McKKu
А так автору спасибо, компонент постоянный желанный гость в большинстве сайтов.
    Василий Наумкин
    30 ноября 2018, 08:51
    0
    Не вижу я этого бага на тестовом сайте.
    http://s16541.h4.modhost.pro/manager/?a=resource/update&id=2&parent=0&context_key=web
    s16541
    0tacIkVJ8Z7Q
    
    Может, его и нет, а это ты сам, своим плагином так делаешь?
      Sem
      Sem
      30 ноября 2018, 09:40
      +1
      Странно проверяю на Вашем тестовом сайте — проблема есть
      1)Делаю так — yadi.sk/i/-CfnE_YNXbwjdQ
      2)Обновляют страницу браузера — yadi.sk/i/i5AVXu29_fXfLw
        Василий Наумкин
        30 ноября 2018, 10:28
        0
        А, ну то есть это после перезагрузки страницы только, а не сразу после сохранения?

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

        Честно говоря, впервые слышу, что это проблема. Кому мешает — пусть шлёт PR в репозиторий, обсудим.
          Sem
          Sem
          30 ноября 2018, 10:37
          0
          На счёт реализации очистки кэша это понятно, просто когда наводишь на чекбокс там написано — «Если эта опция отмечена, MODX будет очищать кэш при сохранение ресурса. Таким образом, ваши посетители не увидят старой версии ресурса.»
          Воспринимается как — один раз установи и дальше система будет очищать кэш сама при сохранении, а на деле нужно каждый раз включать чекбокс если хочется очистить кэш.
            Василий Наумкин
            30 ноября 2018, 10:40
            0
            Это ж стандартный чекбокс MODX и надпись — стандартная.

            Честно скажу, мне с этим разбираться лень. Если кто пришлёт исправление на GitHub, то обработаю. Вот как добавление комментариев из админки прислали, например.
              Sem
              Sem
              30 ноября 2018, 10:44
              0
              ок, это уже хорошо.
              Попробую в комментариях этой заметки пустить клич, для тех, кто умеет делать эти PR
Sem
Sem
30 ноября 2018, 10:48
0
Уважаемые представители сообщества!
Обращаюсь к тем, кто умеет делать PR и разбирается в MODX и компонентах.
Василий готов принять исправление небольшого, как мне кажется «бага», связанного с очисткой кэша у разделов с тикетами (Подробнее можно почитать в комментариях).
Если у кого-то есть немного времени и желания улучшить компонент, просьба помочь решить проблему.
Евгений Лазарев
30 ноября 2018, 16:27
0
А вопрос такой. Работаю ли события в Тикетах?
Пробовал OnTicketThreadOpen — ноль внимания.
Евгений Лазарев
30 ноября 2018, 17:01
0
Не совсем понял но может подскажете. Насколько я понял это событие на момент открытия Тикета. Определенный тикет привязан к get страницею Соответственно и его создание идет в тот момент когда на нее зайдут. Я пытаюсь записать в тикет при его создание подписанных на него пользователей.
    Василий Наумкин
    30 ноября 2018, 17:04
    0
    Внезапно — нет, ты совсем ничего не понял.

    Ветку комментариев можно закрыть для комментирования, а потом открыть. Вот для второго и есть событие OnTicketThreadOpen, которое срабатывает только в админке, при открытии ветки для комментирования.

    И эта, учись отвечать на коммент, а не в корень писать.
      Евгений Лазарев
      30 ноября 2018, 17:07
      0
      Понял. Сможете посоветовать что сделать можно в моей ситуации? На момент именно создания тикета.
        Василий Наумкин
        30 ноября 2018, 17:10
        0
        Работать с событиями на создание тикета (то есть ресурса). Мне кажется, про такое уже писали не раз, и даже здесь.
          Евгений Лазарев
          30 ноября 2018, 17:12
          0
          Писали. Но как я уже говорил ресурса такового нет. К примеру есть страница а у нее порядка 30 get страниц. И каждая из них создает ticket лишь в момент захода на нее. Палка о двух концах.
            Василий Наумкин
            30 ноября 2018, 17:16
            0
            Ведь невозможно самостоятельно создать!

            Задача невыполнима, увы.
snyadanak
02 декабря 2018, 00:25
0
У меня одного после обновления пропали политики доступа? Или так должно быть

    Василий Наумкин
    02 декабря 2018, 06:31
    +1
    Нет, не у тебя одного.

    Поправил, обновляйся.
      Это сообщение было удалено
        Это сообщение было удалено
      Это сообщение было удалено
      Это сообщение было удалено
        Василий Наумкин
        03 декабря 2018, 11:51
        0
        Скажи пожалуйста, зачем ты сюда пишешь про свои ошибки, в которых нет ни слова про мои дополнения?

        Если обновился на MODX 2.7 — пиши issues на Github, сюда флудить не нужно.
          Wassi Wassinen
          03 декабря 2018, 14:18
          0
          Прости, с недосыпу паника проскочила :)) Василий, с офисом после обновления всё хорошо? У меня кнопка logout перестала работать. Использую твой стандартный чанк с fenom.
            Василий Наумкин
            03 декабря 2018, 14:21
            0
            По Office вопросы нужно задавать в поддержке.
              Wassi Wassinen
              03 декабря 2018, 14:29
              0
              Вась, ну по старой дружбе. :)) **угощает вкусным соком**
Пётр
04 декабря 2018, 20:32
0
Такой же баг с «Очисткой кеша». (заметил, что на других ресурсов также)

Только у меня еще в добавок не сохраняет настройки дочерних ресурсов и также рейтинга.

Из-за этого не могу вывести шаблон, изменить url, отключить Jevix. С чем связано — известно?
Алексей
13 декабря 2018, 11:13
0
После перехода на php7.2 если в админке изменить автора тикета вручную, то в БД сохраняется ноль вместо id.
Думал, что версия старая, обновился, то же самое. Посмотрите у себя.
    Василий Наумкин
    13 декабря 2018, 11:42
    0
    Проверил на bezumkin.ru — всё ок.

    Таблицы ресурсов вообще не от Tickets, это стандартные таблицы MODX и сохраняет данные в них тоже он.
      Алексей
      13 декабря 2018, 11:53
      0
      Спасибо, буду искать причину.
      Алексей
      13 декабря 2018, 12:19
      0
      Ещё на старой версии Tickets (действительно очень старой) вместо id после сохранения в админке в поле createdby прописался 0. После обновления автор нормально записывается, но в поле editedby.
      Поправил в БД createdby и кажется все хорошо.
Алексей
13 декабря 2018, 14:43
0
Ещё момент, который не отследил. Или после обновления php или обновления Tickets в одном разделе не выводится количество просмотров / результат голосования и остальные цифры по каждому тикету.
[[!pdoPage:default=`[[%donotfind]]`?
    &element=`getTickets`
    &context=`[[*context_key]]`
    &parents=`10`
    &user=`[[!+userid]]`
    &page=`[[!+pageid]]`
    &pageLimit=`7`
    &limit=`10` 
    &checkPermissions=`list`
    &tplWrapper=`@INLINE <div class="tickets-list">
	[[+output]]
</div>`
]]
[[!+page.nav]]
Особенность в том, что это раздел — часть профиля пользователя, работающий через плагин на PageNotFound (аналогичный modx.pro/users/allex/topics выводу заметок здесь).
Если зайти в любой из выведенных тикетов и вернуться обратно к их списку, то у этого тикета все цифры проставляются. Я и документацию перерыл и с гитхабом посравнивал, идей нет.
ps
[[!+userid]] и [[!+pageid] правильно передаются
    Алексей
    13 декабря 2018, 16:49
    0
    Так, кто будет обновлять древние версии Tickets читайте сюда.
    Надо делать пересчет рейтингов раз core/components/tickets/cron/remove_votes.php и два core/components/tickets/cron/rebuild_rating.php
    После этого всё стало выводиться.