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

[Sheer]

some0ne, удалите папку ext\rxu\PostsMerging\styles и не сушите моск!

[some0ne]

Какого ответа? В полной форме или в расширении быстрого ответа?

в форме Быстрого ответа

Снимок экрана20180206134159.png

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

some0ne, удалите папку ext\rxu\PostsMerging\styles и не сушите моск!

Почему и для чего, кроме мокрого мозга?

[Gubkin]

Почему и для чего, кроме мокрого мозга?

удалится вообще сама настройка в форме ответа, не будет никаких окон , галочек и предложений

[Pazh]

some0ne, нашел. Галочка действительно не убирается не взирая на настройку прав доступа - это косяк совмещения данного расширения с этим [3.1][3.2][release] Быстрый ответ: перезагрузка (QuickReply Reloaded)
В БО есть такой код:

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

			// Always show the checkbox if PostsMerging extension is installed.
			$this->user->add_lang_ext('rxu/PostsMerging', 'posts_merging');
			$template_variables += array('POSTS_MERGING_OPTION' => true);

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

Для исправления, в данном расширении в файле ext\rxu\PostsMerging\event\listener.php найти код:

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

		{
			$this->user->add_lang_ext('rxu/PostsMerging', 'posts_merging');
			$this->template->assign_vars(array('POSTS_MERGING_OPTION' => true));
		}

Добавить после:

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

		else
		{
			$this->template->assign_vars(array('POSTS_MERGING_OPTION' => false));
		}

[rxu]

Для исправления, в данном расширении в файле ext\rxu\PostsMerging\event\listener.php найти код:

Не надо там искать этот код. При обновлении расширения он там опять появится.
И если переменная шаблона не присвоена, она и так false.

[Pazh]

При обновлении расширения он там опять появится.

Если автор расширения не соизволит исправить код, то да, появится

И если переменная шаблона не присвоена, она и так false.

в этом и проблема - кроме Вашего расширения с кучей условий:

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

		if ($this->auth->acl_get('u_postsmerging') && $this->auth->acl_get('u_postsmerging_ignore')
			&& $this->merge_interval && $this->user->data['is_registered'] && (!$mode || in_array($mode, array('reply', 'quote')))
			&& (time() - (int) $post_data['topic_last_post_time']) < $this->merge_interval
			&& !$this->helper->excluded_from_merge(array('forum_id' => $forum_id, 'topic_id' => $topic_id))
			&& $post_data['topic_last_poster_id'] == $this->user->data['user_id']
			&& $this->auth->acl_get('f_noapprove', $forum_id)
		)
		{
			$this->user->add_lang_ext('rxu/PostsMerging', 'posts_merging');
			$this->template->assign_vars(array('POSTS_MERGING_OPTION' => true));
		}

этот же код вставляется расширением БО, но БЕЗ ПРОВЕРКИ ЭТИХ ЖЕ УСЛОВИЙ:

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

		if (
			$this->phpbb_extension_manager->is_enabled('rxu/PostsMerging') &&
			$this->user->data['is_registered'] &&
			$this->config['merge_interval']
		)
		{
			// Always show the checkbox if PostsMerging extension is installed.
			$this->user->add_lang_ext('rxu/PostsMerging', 'posts_merging');
			$template_variables += array('POSTS_MERGING_OPTION' => true);
		}

Дальше нужно разжовывать ответ на вопрос: Что будет, если установить право "Может игнорировать склейку" в "никогда"?
На гуру все аналогично...

[rxu]

этот же код вставляется расширением БО, но БЕЗ ПРОВЕРКИ ЭТИХ ЖЕ УСЛОВИЙ:

Это проблема экста БО, а не склейки.

[Siava]

rxu, phpbb 3.2.1 + PostsMerging 2.1.0 (последний с гитхаба).

Ситуация:

Я подписан на одну тему.
Другой пользователь пишет в неё. Через 3 минуты добавляет сообщение. Ещё через 6 минут добавляет сообщение. Ещё через 12 минут добавляет...
В итоге я получаю 4 уведомления по email после каждого добавления.

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

[rxu]

4 уведомления по email после каждого добавления.

В смысле, на 1 добавление - 4? Т.е. 16 в общей сложности?

Раньше вроде такого не было

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

[Siava]

rxu, не.. всего 4. То есть после каждого добавления приходило новое уведомление об ответе.

Отправлено спустя 4 минуты 14 секунд:
Чуть позже отключу все расширения, проверю.

[rxu]

всего 4

Ну тогда вроде логично. Фактически 4 ответа же было. Если бы склейки не было, всё случилось бы точно также

[Siava]

rxu, а мне казалось, что всегда приходило только одно уведомление - первое, неважно сколько после комментариев добавилось..

[Татьяна5]

Да, одно уведомление обычно приходит (без расширений), до момента пока не просмотришь тему/форум

[Siava]

rxu, Re: Предложения и замечания
Есть идея как исправить парсер при склейке сообщений?

[rxu]

Пока нет.