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

[c61]

rxu, "Duplicate entry..." для config_text - известная фича, мы её обсуждали давным-давно, то ли при появлении empennage ( [BETA] empennage: добавление контента в низ/верх страниц ), то ли около того... Достаточно проверить, изменился ли текст, и если не изменился, то и не выдавать в базу. См. соотв. код в empennage, extender'е и возможно ещё в каких-то моих расширениях...

[rxu]

Достаточно проверить, изменился ли текст, и если не изменился, то и не выдавать в базу.

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

[c61]

Нет, если есть изменения - ошибка не проявляется, а если нет изменений - достаточно не вызывать set_array, вот кусок из empennage_module.php, жалоб после исправлений не поступало:

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

			foreach ( $empennage_config_text_values as $key => $value)
			{
				$new_value = $request->variable(array('config',$key),'',true);
				if ( (string) $new_value !== (string) $value )
				{
					$value = $empennage_config_text_values[$key] = $new_value;
					$config_text->set_array(array(
						$key => $value,
					));
				}
			}

[rxu]

В принципе, вариант. Можно попробовать.

[Dutch]

А не проще ли вместо проверок и INSERT просто обновлять информацию через UPDATE?

[Dutch]

Стабильная версия 2.0.2 не совместима с быстрым ответом от Татьяны (с аяксом) — тоесть сообщение отправляется, но не отображается отправленное.

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

[LavIgor]

Dutch, так галочку как раз и нужно ставить, чтобы сообщение не склеивалось.

[Dutch]

Dutch, так галочку как раз и нужно ставить, чтобы сообщение не склеивалось.

Тут дело не галочке, а в том, что в быстром ответе Татьяны не отображается ответ, хоть и склеен он (то что нужно!), а в дев-версии для пользователей расширение не работает.

[LavIgor]

Какая версия движка и какая версия расширения БО?

[Dutch]

Какая версия движка и какая версия расширения БО?

PHPBB 3.1.6
Версии расширений были 2 — первая 2.0.5 и 2.0.3-dev (я писал об этом выше)

[LavIgor]

2.0.5 и 2.0.3-dev

какая версия расширения БО?

БО - быстрый ответ.

[Dutch]

БО - быстрый ответ.

А чем отличается от этого?

Стабильная версия 2.0.2 не совместима с быстрым ответом от Татьяны (с аяксом) — тоесть сообщение отправляется, но не отображается отправленное.

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

Тут и версия сразу видна и суть проблемы!

[LavIgor]

Тем, что Вы указали версию расширения склейки.
А есть ещё расширение для быстрого ответа, которое сравнительно недавно обновилось.

[Dutch]

Расширение от Татьяны последней версии

[rxu]

Dutch, обновитесь на это расширение обычным способом (оно является продолжением предыдущего) [beta] Быстрый ответ: перезагрузка (QuickReply Reloaded)