[3.3] [release] Posts merging by rxu

[rxu]

Расширение прошло валидацию на phpbb.com: https://www.phpbb.com/customise/db/exte ... tsmerging/

Исходный код: https://github.com/rxu/posts_merging
Копировать в папку /ext/rxu/postsmerging/.

[Emperor1]

LavIgor, скачал, включил, но сообщения не склеиваются почему-то, в настройках расширения всё по дефолту стоит, галка "Не склеивать с предыдущим сообщением" отключена. Может ещё какие-то настройки есть?

[Dook]

Emperor1,
права доступа:
Может использовать склеивание сообщений
Может игнорировать склеивание сообщений

rxu,
подключение Prime Notify сможет решить проблему? или только правка ядра?

[rxu]

подключение Prime Notify сможет решить проблему?

Без понятия, никогда не использовал.

[Dook]

rxu, а в сторону какой функции в ядре копать?

[rxu]

Dook, копать придется много чего, и хелпер, и слушатель.
Вкратце, расширение берет из базы предыдущий пост, декодирует, добавляет к нему новый текст, записывает обратно в базу, обновляя поисковый индекс и время создания сообщения.
Обновить идентификатор вряд ли получится, ибо он автоинкрементный.
Следовательно, вам придется менять логику работы: брать старый пост, добавлять к нему новый, постить новый пост и при этом удалять старый. В расширении вообще нет кода для последних двух операций.

[Dook]

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

Отправлено спустя 2 минуты 47 секунд:
rxu, в связи с чем в коде отсутствует эта строка?

Код: Выделить всё

$time[] = ($interval->d) ? $this->user->lang('D_MDAY', $interval->d) : null; 

Какая-то несовместимость с ajax версией quickreply?

[rxu]

в связи с чем в коде отсутствует эта строка?

Чисто из волюнтаристских соображений. Изначально не планировалось, что найдутся пользователи, устанавливающие исчисляемый днями, месяцами и годами интервал склейки.

[Dook]

Более элегантное решение, с минимальными изменениями в исходном коде расширения, через удаление старых уведомлений

[rxu]

Dook, еще более элегантно было бы отправить пулл реквест в репо

[Dook]

rxu, отправил, хотя интерфейс гитхаба далеко не элегантный)

[LavIgor]

интерфейс гитхаба далеко не элегантный

Не соглашусь. Глючит, правда, последнее время, но выглядит прекрасно.

А вот вставлять своё имя в каждую правку, даже если это всего 2 строки - это точно неэлегантно.
Даже для мода это было бы некрасиво.

[Dook]

вставлял я чисто для себя, и вставляю так всегда, чтобы при изменениях исходного текста с минимальными телодвижениями перенести в новую версию. пользоваться ими и тем более эстетически восхищаться никого не заставляю. на гитхаб выложил по "просьбе" rxu.

[LavIgor]

Dook, тогда другое дело.

[rxu]

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

[Dook]

а правильно ли будет удалять уведомления без ведома пользователя

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

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

Собственно они и должны быть непрочитанными, на то они и уведомления, чтобы их читать.