[release] Posts merging by rxu

Все расширения, созданные нашим сообществом для phpBB, как находящиеся в разработке, так и прошедшие валидацию на официальном сайте phpbb.com, будут анонсированы тут. Вся техническая поддержка по этим расширениям оказывается в этом форуме.
Правила форума
Местная Конституция | Шаблон запроса | Документация (phpBB3) | Мини [FAQ] по phpBB3.1.x/3.3.x | FAQ | Как задавать вопросы | Как устанавливать расширения

Ваш вопрос может быть удален без объяснения причин, если на него есть ответы по приведённым ссылкам (а вы рискуете получить предупреждение ;) ).
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14531
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 348 раз
Поблагодарили: 1556 раз

[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/.
Изображение

Перенесено из форума Бета-версии расширений для phpBB 3.1.x в форум Анонсы и поддержка расширений для phpBB 3.1.x 24.03.2015 21:09 модератором rxu

Аватара пользователя
Sheer
phpBB Guru
phpBB Guru
Сообщения: 11491
Зарегистрирован: 18.02.2007 19:01
Откуда: Калининград не Кенигсберг
Благодарил (а): 53 раза
Поблагодарили: 2579 раз

Re: [release] Posts merging by rxu

Сообщение Sheer » 05.02.2018 21:47

some0ne, удалите папку ext\rxu\PostsMerging\styles и не сушите моск!
Изображение
Общие ошибки новичков (07.11.2005) & Как задавать вопросы
Мини FAQ
Если ничто другое не помогает, прочтите, наконец, инструкцию!
"Никакая инструкция не может перечислить всех обязанностей должностного лица, предусмотреть все отдельные случаи и дать вперёд соответствующие указания, а поэтому господа инженеры должны проявить инициативу и, руководствуясь знаниями своей специальности и пользой дела, принять все усилия для оправдания своего назначения".
Циркуляр Морского технического комитета №15 от 29.11.1910 г.

some0ne
phpBB 1.2.1
Сообщения: 27
Зарегистрирован: 24.12.2017 18:58
Благодарил (а): 17 раз

Re: [release] Posts merging by rxu

Сообщение some0ne » 06.02.2018 13:45

rxu писал(а):
05.02.2018 18:10
Какого ответа? В полной форме или в расширении быстрого ответа?
в форме Быстрого ответа
Снимок экрана20180206134159.png
Отправлено спустя 2 минуты 42 секунды:
Sheer писал(а):
05.02.2018 21:47
some0ne, удалите папку ext\rxu\PostsMerging\styles и не сушите моск!
Почему и для чего, кроме мокрого мозга?

Аватара пользователя
Gubkin
phpBB 2.0.14
Сообщения: 999
Зарегистрирован: 11.07.2015 6:10
Благодарил (а): 209 раз
Поблагодарили: 97 раз

Re: [release] Posts merging by rxu

Сообщение Gubkin » 06.02.2018 14:09

some0ne писал(а):
06.02.2018 13:47
Почему и для чего, кроме мокрого мозга?
удалится вообще сама настройка в форме ответа, не будет никаких окон , галочек и предложений
Ещё один тупой вопрос и будете забанены. К гуру надо приходить подготовленными, а не как Вы. Вчера создали форум с парой постов, а сегодня уже 20 вопросов )))

Аватара пользователя
Pazh
Former team member
Сообщения: 2194
Зарегистрирован: 09.11.2009 17:46
Благодарил (а): 41 раз
Поблагодарили: 454 раза

Re: [release] Posts merging by rxu

Сообщение Pazh » 06.02.2018 14:24

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));
		}
форум ЖК Вестердам Помощь в ЛС/email только за WM или ЯД

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14531
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 348 раз
Поблагодарили: 1556 раз

Re: [release] Posts merging by rxu

Сообщение rxu » 06.02.2018 17:23

Pazh писал(а):
06.02.2018 14:24
Для исправления, в данном расширении в файле ext\rxu\PostsMerging\event\listener.php найти код:
Не надо там искать этот код. При обновлении расширения он там опять появится.
И если переменная шаблона не присвоена, она и так false.
Изображение

Аватара пользователя
Pazh
Former team member
Сообщения: 2194
Зарегистрирован: 09.11.2009 17:46
Благодарил (а): 41 раз
Поблагодарили: 454 раза

Re: [release] Posts merging by rxu

Сообщение Pazh » 06.02.2018 17:37

rxu писал(а):
06.02.2018 17:23
При обновлении расширения он там опять появится.
Если автор расширения не соизволит исправить код, то да, появится
rxu писал(а):
06.02.2018 17:23
И если переменная шаблона не присвоена, она и так 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);
		}
Дальше нужно разжовывать ответ на вопрос: Что будет, если установить право "Может игнорировать склейку" в "никогда"?
На гуру все аналогично...
форум ЖК Вестердам Помощь в ЛС/email только за WM или ЯД

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14531
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 348 раз
Поблагодарили: 1556 раз

Re: [release] Posts merging by rxu

Сообщение rxu » 06.02.2018 17:45

Pazh писал(а):
06.02.2018 17:37
этот же код вставляется расширением БО, но БЕЗ ПРОВЕРКИ ЭТИХ ЖЕ УСЛОВИЙ:
Это проблема экста БО, а не склейки.
Изображение

Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 4188
Зарегистрирован: 11.01.2005 14:29
Откуда: Питер
Благодарил (а): 109 раз
Поблагодарили: 443 раза

Re: [release] Posts merging by rxu

Сообщение Siava » 19.02.2018 20:42

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

Ситуация:

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

Раньше вроде такого не было или просто не замечал :roll:
Еще одно нарушение правил и будете забанены. © Mr. Anderson
https://siava.ru/forum/ (phpbb 2.0.x, 3.1.x 3.2.x)

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14531
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 348 раз
Поблагодарили: 1556 раз

Re: [release] Posts merging by rxu

Сообщение rxu » 19.02.2018 21:03

Siava писал(а):
19.02.2018 20:42
4 уведомления по email после каждого добавления.
В смысле, на 1 добавление - 4? Т.е. 16 в общей сложности?
Siava писал(а):
19.02.2018 20:42
Раньше вроде такого не было
Не было, раньше вообще никакие уведомления при склейке не отправлялись.
Изображение

Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 4188
Зарегистрирован: 11.01.2005 14:29
Откуда: Питер
Благодарил (а): 109 раз
Поблагодарили: 443 раза

Re: [release] Posts merging by rxu

Сообщение Siava » 19.02.2018 21:04

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

Отправлено спустя 4 минуты 14 секунд:
Чуть позже отключу все расширения, проверю.
Еще одно нарушение правил и будете забанены. © Mr. Anderson
https://siava.ru/forum/ (phpbb 2.0.x, 3.1.x 3.2.x)

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14531
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 348 раз
Поблагодарили: 1556 раз

Re: [release] Posts merging by rxu

Сообщение rxu » 19.02.2018 21:14

Siava писал(а):
19.02.2018 21:08
всего 4
Ну тогда вроде логично. Фактически 4 ответа же было. Если бы склейки не было, всё случилось бы точно также :)
Изображение

Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 4188
Зарегистрирован: 11.01.2005 14:29
Откуда: Питер
Благодарил (а): 109 раз
Поблагодарили: 443 раза

Re: [release] Posts merging by rxu

Сообщение Siava » 19.02.2018 21:18

rxu, :shock: а мне казалось, что всегда приходило только одно уведомление - первое, неважно сколько после комментариев добавилось.. :)
Еще одно нарушение правил и будете забанены. © Mr. Anderson
https://siava.ru/forum/ (phpbb 2.0.x, 3.1.x 3.2.x)

Аватара пользователя
Татьяна5
Поддержка
Поддержка
Сообщения: 9776
Зарегистрирован: 08.08.2011 2:02
Благодарил (а): 175 раз
Поблагодарили: 2719 раз

Re: [release] Posts merging by rxu

Сообщение Татьяна5 » 19.02.2018 22:03

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

Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 4188
Зарегистрирован: 11.01.2005 14:29
Откуда: Питер
Благодарил (а): 109 раз
Поблагодарили: 443 раза

Re: [release] Posts merging by rxu

Сообщение Siava » 13.03.2018 12:16

rxu, Re: Предложения и замечания
Есть идея как исправить парсер при склейке сообщений? :roll:
Еще одно нарушение правил и будете забанены. © Mr. Anderson
https://siava.ru/forum/ (phpbb 2.0.x, 3.1.x 3.2.x)

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14531
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 348 раз
Поблагодарили: 1556 раз

Re: [release] Posts merging by rxu

Сообщение rxu » 13.03.2018 16:58

Пока нет.
Изображение

Вернуться в «Анонсы и поддержка расширений для phpBB»