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

[rxu]

'test'

Отправлено спустя 10 секунд:
"test"

Отправлено спустя 11 секунд:
'test2'

Отправлено спустя 14 секунд:
Что я не так делаю?

Отправлено спустя 35 секунд:
'test'

Отправлено спустя 15 секунд:

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

'test'

Отправлено спустя 27 секунд:
т.е. проблема только в кастомном ббкоде icode?

[Татьяна5]

Тест
'текст в кавычках'
До склейки.

Отправлено спустя 20 секунд:
Тест
'текст в кавычках'
После склейки.

Отправлено спустя 54 секунды:
Ничего не преобразовалось

Дубль 2 - в теге b
'текст в кавычках'

Отправлено спустя 1 минуту 3 секунды:
Ничего

В теге ref
'текст в кавычках'

Отправлено спустя 37 секунд:
Да, проблема в кастомных ббкодах (icode, ref)

[Perfecthus]

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

Снимок.PNG

Установлены следующие расширения

Снимок0.PNG

Подскажите, в чём подвох?

[LavIgor]

Perfecthus, склейка не использует Ajax. Его использует расширение QuickReply.
У Вас ошибка 500. Что говорят логи на сервере?

[Perfecthus]

Что говорят логи на сервере?

Надо заглянуть :)

[romeo_piter]

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

Отправлено спустя 1 минуту 14 секунд:

А в некоторых добавляется название темы:

Отправлено спустя 1 минуту 14 секунд:
Название темы

[LavIgor]

добавляется название темы

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

[romeo_piter]

Это можно как-то отключить?

[Perfecthus]

Что говорят логи на сервере?

Логи говорят об этом:

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

[Fri Sep 04 18:13:49 2015] [error] [client 42.28.16.11] PHP Fatal error:  Using $this when not in object context in /www/domain.ru/ext/rxu/PostsMerging/event/listener.php on line 187, referer: http://www.precision-machines.ru/viewtopic.php?f=48&p=2326

Строка 187 в listener.php function ($matches) { return $this->user->lang($matches[1]); },
Как-то так... И чего ему не нравится?

[LavIgor]

Perfecthus, у нас принято читать тему или пользоваться поиском перед отправкой ответа.
Re: [release] Posts merging by rxu (Пост LavIgor #467485)
В моём сообщении даже ссылка на правку есть, которую нужно просто временно отменить.

[Perfecthus]

В моём сообщении даже ссылка на правку есть, которую нужно просто временно отменить

Извиняюсь, не заметил, такое бывает, так что не стоит ругаться ;)

Отправлено спустя 13 минут 13 секунд:

Т.е. есть несовместимость с PHP 5.3.3

У меня PHP Version 5.3.29-1~dotdeb.0
Внёс правки с github расширение заработало. Спасибо.

[Dutch]

После сохранения настроек выдает ошибку:

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

SQL ERROR [ mysqli ]

Duplicate entry 'posts_merging_separator_text' for key 'PRIMARY' [1062]

SQL

INSERT INTO phpbb_config_text (config_name, config_value) VALUES ('posts_merging_separator_text', '{L_MERGE_SEPARATOR}')

Правда в БД настройки все же сохраняются.

В чем может быть трабл и как его решить? Может в тонкой настройки мускула?

[rxu]

После сохранения настроек выдает ошибку

Версия MySQL 5.6 или старше? Таблицы InnoDB?

[Dutch]

Версия MySQL 5.6 или старше? Таблицы InnoDB?

5.3.12-MariaDB, а таблицы в InnoDB.

[rxu]

Для теста нужно хакнуть \phpbb\config\db_text.php, а именно попробовать убрать или закомментировать 2 строки вида $this->db->sql_transaction в функции public function set_array(array $map).