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
modx.pro
2
1 798
+17

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

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

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

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

            Честно скажу, мне с этим разбираться лень. Если кто пришлёт исправление на GitHub, то обработаю. Вот как добавление комментариев из админки прислали, например.
              Семён Кудрявцев
              Семён Кудрявцев
              30 ноября 2018, 10:44
              0
              ок, это уже хорошо.
              Попробую в комментариях этой заметки пустить клич, для тех, кто умеет делать эти PR
    Семён Кудрявцев
    Семён Кудрявцев
    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
        snyadanak
        02 декабря 2018, 00:25
        0
        У меня одного после обновления пропали политики доступа? Или так должно быть

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

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

                  Если обновился на MODX 2.7 — пиши issues на Github, сюда флудить не нужно.
                    Wassi Wassinen
                    Wassi Wassinen
                    03 декабря 2018, 14:18
                    0
                    Прости, с недосыпу паника проскочила :)) Василий, с офисом после обновления всё хорошо? У меня кнопка logout перестала работать. Использую твой стандартный чанк с fenom.
                      Василий Наумкин
                      Василий Наумкин
                      03 декабря 2018, 14:21
                      0
                      По Office вопросы нужно задавать в поддержке.
                        Wassi Wassinen
                        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 и кажется все хорошо.
                    Wassi Wassinen
                    Wassi Wassinen
                    15 декабря 2018, 12:25
                    0
                    Вася, привет. Скажи, на modx.pro есть тикеты с просмотрами более 260.000? Если есть, то как они себя ведут при загрузке? У меня на сайте такие тикеты грузятся всё медленнее и медленнее. Сначала не мог понять в чем дело, но потом «методом научного тыка» пришёл к тому, что все исправляется если отключить вызов сниппета TicketMeta. Это может быть как-то связано с кол-вом просмотров и их отображением?

                    Заранее благодарен за ответ.
                      Василий Наумкин
                      Василий Наумкин
                      15 декабря 2018, 15:18
                      0
                      Наверное есть, но после обновления на modx.pro Tickets не используется — тут полностью свой код, но на MODX.

                      А так, да, TicketsMeta подсчитывает всякое и может тормозить. Дебаж, и шли исправления в репо.
                        Wassi Wassinen
                        Wassi Wassinen
                        16 декабря 2018, 17:26
                        0
                        Попробую найти кого-нибудь, кто сможет исправить. ПР отправлять на гит?
                          snyadanak
                          snyadanak
                          16 декабря 2018, 17:48
                          0
                          А удаление своих же тикетов на фронте не планируется добавить? Вроде и pr отправили с той же просьбой
                          Алексей
                          Алексей
                          19 декабря 2018, 14:58
                          0
                          Василий, подскажите можно ли убрать подсчет количества просмотров так, чтобы можно было спокойно обновляться потом?
                          Просто убрать вывод из чанка вряд ли уберет сам процесс подсчета?
                            tolanych
                            tolanych
                            19 декабря 2018, 22:31
                            0
                            Да, сам процесс подсчета к чанку не относится.

                            Полумеры — не считать просмотры гостями — системная настройка tickets.count_guests

                            Так чтобы совсем отключить — это уже кочевряжить код нужно, в плагине Tickets событие OnWebPageComplete вызывает подсчет просмотров

                            UPD

                            хотя можете попробовать в настройках плагина (вкладка системные события) просто снять галочку OnWebPageComplete
                              Алексей
                              Алексей
                              19 декабря 2018, 22:37
                              0
                              Жаль, вывод количества просмотров мало на каких сайтах даёт что-то полезное пользователю, многие от этого уходят. Не хватает «галочки» отключить подсчет.
                    Алексей
                    Алексей
                    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
                      После этого всё стало выводиться.
                      Сергей Шлоков
                      Сергей Шлоков
                      24 декабря 2018, 21:33
                      0
                      а вы научили компонент нормально удаляться?
                        Александр Мельник
                        Александр Мельник
                        06 февраля 2019, 16:41
                        0
                        Господа.
                        При установке Tickets на MODX 2.7 ошибка при установке
                        Could not create table `sometext_tickets_threads` SQL: CREATE TABLE `sometext_tickets_threads` (`id` INTEGER unsigned NOT NULL AUTO_INCREMENT, `resource` INT(10) unsigned NOT NULL DEFAULT '0', `name` VARCHAR(255) NOT NULL DEFAULT '', `subscribers` TEXT NOT NULL, `createdon` DATETIME NULL, `createdby` INTEGER(10) unsigned NOT NULL DEFAULT '0', `closed` TINYINT(1) unsigned NOT NULL DEFAULT '0', `deleted` TINYINT(1) unsigned NOT NULL DEFAULT '0', `deletedon` DATETIME NULL, `deletedby` INTEGER(10) unsigned NOT NULL DEFAULT '0', `comment_last` INT(10) unsigned NOT NULL DEFAULT '0', `comment_time` DATETIME NULL, `comments` INT(10) unsigned NULL DEFAULT '0', `properties` TEXT NULL, PRIMARY KEY (`id`), UNIQUE KEY `name` (`name`), INDEX `resource` (`resource`), INDEX `comment_last` (`comment_last`), INDEX `comments` (`comments`), INDEX `closed` (`closed`)) ENGINE=InnoDB ERROR: Array ( [0] => 42000 [1] => 1071 [2] => Specified key was too long; max key length is 767 bytes )
                        На сервере mysql 5.6, поднять выше не могу. Насколько я понимаю, этой ошибки не возникло бы на 5,7 но как же быть…
                          Авторизуйтесь или зарегистрируйтесь, чтобы оставлять комментарии.
                          47