[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/.

[LONER]

DeathMan, Повторюсь, если сразу не понятно.
Правила, пункт: 2.5
[quote]Сообщения о диагностических ошибках приводите полностью и только в виде текста, обрамленного BBCode

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

[/b].[/quote]

[DeathMan]

LONER, Да я уже отредактировал свое сообщение

[LBeaver]

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

[phpBB Debug] PHP Warning: in file [ROOT]/ext/rxu/PostsMerging/event/listener.php on line 176: preg_replace(): The /e modifier is no longer supported, use preg_replace_callback instead
[phpBB Debug] PHP Warning: in file [ROOT]/includes/functions.php on line 1835: Cannot modify header information - headers already sent by (output started at [ROOT]/includes/functions.php:3257)

При отправке двух сообщений с вложениями (phpBB 3.2, PHP 7.0).

[shvager]

Также интересует: будет доработка под 3,2? Подтверждаю ("Кушает" предыдущее сообщение...)

[Sumanai]

PHP 7.

Это. Так же расширение несовместимо с MySQL 5.7.

[rxu]

При отправке двух сообщений с вложениями (phpBB 3.2, PHP 7.0).

Исправлено в репо.

расширение несовместимо с MySQL 5.7.

В каком именно месте?

[Siava]

В каком именно месте?

Вероятно вот в этом: https://github.com/rxu/posts_merging/pull/34/files

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

In aggregated query without GROUP BY, expression #1 of SELECT list contains nonaggregated column '***.phpbb_attachments.attach_id'; this is incompatible with sql_mode=only_full_group_by [1140]

[rxu]

Вероятно вот в этом

Тоже, видимо, исправил.

[misterleks]

Установлено последняя версия из репозитория.
Удаляется текст предыдущего сообщения, если оно было с вложением и последующее с вложением. Вложения суммируются, а текст исчезает. Я так понимаю это исправляли?

[rxu]

Вложения суммируются, а текст исчезает. Я так понимаю это исправляли?

Не уверен.

Отправлено спустя 1 минуту 20 секунд:
Возможно, вот это https://github.com/rxu/posts_merging/co ... efa8ae4a4b

[misterleks]

rxu, это исправление у меня стоит. Но текст все равно исчезает, если второе сообщение с картинкой.

[cianid]

Поставил версию с гитхаба, все ОК. Сообщения не затираются, как с вложениями и без.

[misterleks]

Перекачал. Установил заново. Все Ok. Значит, у меня была не последняя версия.

[Vlad__]

Поставил версию с гитхаба, все ОК. Сообщения не затираются, как с вложениями и без.

Какую версию именно? На 3.2 затираются вложения.

[rxu]

На 3.2 затираются вложения.

Ничего там не затирается.