[release] Posts merging by rxu

Сообщение **rxu** » 09.01.2015 15:03

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

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

VADR · Сообщение **VADR** » 15.11.2015 23:39

Наверное уже обсуждалось, но не нашёл...
Чистая установка 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 · Сообщение **Sumanai** » 15.11.2015 23:56

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

LavIgor · Сообщение **LavIgor** » 15.11.2015 23:57

VADR писал(а): Наверное уже обсуждалось

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

VADR · Сообщение **VADR** » 16.11.2015 12:25

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

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

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

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

LavIgor · Сообщение **LavIgor** » 16.11.2015 17:18

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

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

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

VADR · Сообщение **VADR** » 16.11.2015 23:02

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

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

KEMnEP · Сообщение **KEMnEP** » 17.11.2015 7:57

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

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

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

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

VADR · Сообщение **VADR** » 17.11.2015 8:20

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

LavIgor · Сообщение **LavIgor** » 17.11.2015 15:41

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

VADR · Сообщение **VADR** » 18.11.2015 16:41

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

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

VADR · Сообщение **VADR** » 25.11.2015 23:31

Таки поборол проблему со слешами. Вроде бы :). Возможно, и с кавычками это поможет (хотя на моей тестилке такая проблема не наблюдалась). в файле 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** » 25.11.2015 23:55

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

LavIgor · Сообщение **LavIgor** » 26.11.2015 0:20

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

VADR · Сообщение **VADR** » 27.11.2015 12:10

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

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

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

Сообщение **rxu** » 27.11.2015 12:44

VADR писал(а): А что не так с функцией addslashes,

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