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

[ciiz1]

Не подскажите как в генерируемом Отправлено спустя секунды убрать.

[rxu]

Не понял вопроса, что где убрать. Что будет генерироваться, если прошло меньше минуты?

[ciiz1]

Вы понимаете сделал я раньше как то (без логических операций), причём языками не владею (интуитивно что то поменял), обновил расширение, сейчас найти не могу.

[Andex]

Столкнулся с такой ситуацией

Стоит 3.3.1
Расширение 2.1.0

Все склеивается, все ок.

Однако, когда производится поиск и находится данная мессага (в настройках обрезка сообщения до 300 символов), она странно парсится и выводится в следующем виде, например:

<r><B><s>[b]</s>Andex<e>[/b]</e></B><br/> привет, давно не виделись<E>:D</E> <br/> <br/> <B><s>[b]</s>odessitmws<e>[/b]</e></B>

В общем, какая-то каша с тегами и т.п.
Если обрезку в результатах поиска убрать - все ок.
Сама мессга в топике тоже вывоится нормально

Не сталкивались с таким?
Куда смотреть?

Отправлено спустя 1 минуту 54 секунды:
А, да, и еще момент забыл.
Если после склейки результирующий пост пересоранить (редактированием) - в поиске тоже становится все ок

Отправлено спустя 1 минуту 20 секунд:
О! этот мой пост как раз пример. На этом форуме такая же ерунда...

search.php?author_id=5521&sr=posts

[Татьяна5]

На этом форуме текст без ббкодов отобразился
Баг повторить не смогла (на 3.3.3 и последней версии расширения)

[Andex]

Хм, разбираюсь дальше.
Отпишусь по результатам

[Pazh]

Татьяна5,

image.png

Этот баг есть и связан не столько с конкретным расширением, а с обрезкой постов - часто встречающаяся проблема еще с 3-ки - перед обрезанием поста нужно удалить из него все bbcode и html сущности

[Siava]

Об этом я ранее уже писал тут. Не помню точно когда появилось.. где-то после обновления с 3.1 до 3.2. Но сейчас не проявляется. В старых постах попадается (в поиске) такое:

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

<r>Если хотите или нужно пересаживать, то цветонос лучше удалить. И укоренить. А пересаживать по обстоятельствам. В принципе, во-второй или третьей декаде февраля вполне уже можно.<br/> <br/> <SIZE size="85"><s>[size=85]</s><COLOR color="green"><s>[color=green]</s>Отправлено спустя 1 минуту 17 секун...

P.S.
поправка, ещё на phpbb 3.1 такое было.

[Pazh]

Но сейчас не проявляется

на скрине выше пост, написанный вчера

[Siava]

Pazh, ну так на скрине тот же текст, что и в посте.

[Pazh]

Вот пример с другого форума поста БЕЗ склеивания, но в выдаче в поиске он выглядит аналогично (ссылки затерты, но суть проблемы видна):

image.png

[Andex]

Об этом я ранее уже писал тут. Не помню точно когда появилось.. где-то после обновления с 3.1 до 3.2. Но сейчас не проявляется. В старых постах попадается (в поиске) такое:

я не одинок оказывается... правда я как раз только эту проблему обнаружил.
А у вас именно при склейке такое проявлялось или независимо?

Вот пример с другого форума поста БЕЗ склеивания

У меня почему-то это проявляется именно при склейке...
Причем, что самое странное, если этот пост отредактировать - то проблема исчезает.

Не понимаю куда копать, ковырять PostsMerging или search.php
Да и вообще непонятна причина проблемы, слишком избирательно как-то происходит... Не ясна логика...

Отправлено спустя 29 минут 7 секунд:
Важное уточнение

В общем, опытным путем я установил, что при склейке у поста пропадет bbcode_uid
Как только его нет и есть обрезка — получаем эту кашу с тегами.

Отправлено спустя 47 минут 9 секунд:
А "пропадает" bbcode_uid вот в этом месте listener.php:

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

$warn_msg = generate_text_for_storage($merge_post_data['post_text'], $merge_post_data['bbcode_uid'], $merge_post_data['bbcode_bitfield'], $options, $merge_post_data['enable_bbcode'], $merge_post_data['enable_magic_url'], $merge_post_data['enable_smilies']);

Попробовал предварительно сохранить значение во временной переменной, а потом переназначить обратно - проблема ушла.

Как-то так:

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

//Сохраняем bbcode_uid, т.к. в generate_text_for_storage он куда-то пропадает, а ниже возвращаем его обратно
$save_bbcode_uid =  $merge_post_data['bbcode_uid'];
			
$warn_msg = generate_text_for_storage($merge_post_data['post_text'], $merge_post_data['bbcode_uid'], $merge_post_data['bbcode_bitfield'], $options, $merge_post_data['enable_bbcode'], $merge_post_data['enable_magic_url'], $merge_post_data['enable_smilies']);

//возвращаем bbcode_uid
$merge_post_data['bbcode_uid'] = $save_bbcode_uid;

Однако не уверен в правильности такого решения, хотелось бы услышать комментарий rxu.

[Татьяна5]

На 3.1 было когда-то, на 3.3 не наблюдается

на скрине выше пост, написанный вчера

Там корректно отображён сам текст сообщения

[Andex]

Слушайте, ну я не могу это повторить на phpbbguru

Но повторяю на голом форуме 3.3.3 с одним данным расширением.
Попробует кто?

[Татьяна5]

Andex, расширение когда и откуда скачано? Точно последней версии со всеми правками?