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

[VADR]

Наверное уже обсуждалось, но не нашёл...
Чистая установка phpbb 3.1.6 (качал отсюда: https://www.phpbb.com/downloads/), Post Merging версии 2.0.2 (качал отсюда: https://www.phpbb.com/customise/db/exte ... tsmerging/). Больше никаких расширений нет, даже языкового пакета. Специально делал установку "с нуля", чтобы одну проблему проверить.
После включения расширения post merging при написании любого поста в существующей теме (т.е. пост не первый) получаем такую ошибку:

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

[phpBB Debug] PHP Warning: in file [ROOT]/phpbb/search/fulltext_native.php on line 107: Missing argument 8 for phpbb/search/fulltext_native::__construct(), called in [ROOT]/ext/rxu/PostsMerging/core/helper.php on line 261 and defined
[phpBB Debug] PHP Warning: in file [ROOT]/includes/functions.php on line 2416: Cannot modify header information - headers already sent by (output started at [ROOT]/includes/functions.php:3903)

update:
при этом, если вернуться в тему, изменения оказываются сохранёнными

[Sumanai]

VADR, используйте версию с репозитория:
https://github.com/rxu/posts_merging/archive/master.zip

[LavIgor]

Наверное уже обсуждалось

Re: [release] Posts merging by rxu (Пост rxu #463162)

[VADR]

Спасибо, установил 2.0.3-dev из репозитория - заработало.

Теперь, собственно, проблема, из-за которой я и делал всю эту установку с нуля (дабы исключить возможные влияния других расширений). При вставке сообщений из текста удаляются обратные слеши (такие вот: "\"). Причём не только те, которые в "дописываемом" сообщении, но и те, что были выше. Если в тексте был двойной обратный слеш "\\") - остаётся один.

Отправлено спустя 1 минуту 49 секунд:
проверяю обратный слеш прямо здесь: \ тест

Отправлено спустя 28 секунд:
а здесь почему-то такой проблемы нет :(

[LavIgor]

VADR, в расширении есть проблемка с кавычками, которая пока не имеет решения: 'текст в кавычках'.
Возможно, это как-то связано.

Отправлено спустя 36 секунд:
Двойной слеш: \\
Двойной слеш перед кавычкой: \\' \\'.

Отправлено спустя 49 секунд:
Как итог: проблема со слешами не наблюдается.

[VADR]

Как итог: проблема со слешами не наблюдается.

Тут я уже проверил - вижу, что проблемы нет. Откуда берётся - непонятно. Я ведь, собственно, из-за этого тестовую площадку и переделывал: только чистый phpbb 3.1.6, из расширений - только это. А слеши обратные мне таки нужны для другого расширения...

[KEMnEP]

в расширении есть проблемка с кавычками

Ане с icode случаем эта проблема?

Отправлено спустя 1 минуту :
ибо "текст в кавычках" и "текст в кавычках"

Отправлено спустя 56 секунд:
о, терь и " даже в icode нормально отрабатывает

[VADR]

Что интересно, у меня на тестовой площадке проблемы с кавычками нет...

[LavIgor]

KEMnEP, проблема только с одинарными кавычками и только с Custom BBCodes, одним из которых на данной конференции является icode.

[VADR]

KEMnEP, проблема только с одинарными кавычками и только с Custom BBCodes, одним из которых на данной конференции является icode.

Фишка в том, что у меня нет Custom BBCodes, и icode тоже нет, соответственно. А проблема с поеданием обратных слешей - есть.

[VADR]

Таки поборол проблему со слешами. Вроде бы :). Возможно, и с кавычками это поможет (хотя на моей тестилке такая проблема не наблюдалась). в файле rxu/PostsMerging/core/helper.php в 190-й строке добавляем addslashes. Было:

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

$text = html_entity_decode($message_parser->message,  ENT_COMPAT, 'UTF-8'); 

Получаем:

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

$text = addslashes(html_entity_decode($message_parser->message,  ENT_COMPAT, 'UTF-8')); 

[Sheer]

addslashes не пропускает валидатор Travis

[LavIgor]

Sheer, это не Travis, а EPV ругается.
Но мега-валидаторы из Extensions Team не то что не противятся этому, а даже заставляют меня вместо штатной правильной функции движка (которая делает то же самое, но рекурсивно) использовать addslashes.
С объяснением, что "you don't need it recursively".

[VADR]

addslashes не пропускает валидатор Travis

Sheer, это не Travis, а EPV ругается.
Но мега-валидаторы из Extensions Team не то что не противятся этому, а даже заставляют меня вместо штатной правильной функции движка (которая делает то же самое, но рекурсивно) использовать addslashes.
С объяснением, что "you don't need it recursively".

А что не так с функцией addslashes, что она влияет на валидность кода? Или не влияет? :)

[rxu]

А что не так с функцией addslashes,

Есть теоретические проблемы с безопасностью, пример http://stackoverflow.com/questions/8609 ... addslashes