Уважаемые пользователи!
C 7 ноября 2020 года phpBB Group прекратила выпуск обновлений и завершила дальнейшее развитие phpBB версии 3.2.
С 1 августа 2024 года phpBB Group прекращает поддержку phpBB 3.2 на официальном сайте.
Сайт официальной русской поддержки phpBB Guru продолжит поддержку phpBB 3.2 до 31 декабря 2024 года.
С учетом этого, настоятельно рекомендуется обновить конференции до версии 3.3.

[release] Posts merging by rxu

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

Ваш вопрос может быть удален без объяснения причин, если на него есть ответы по приведённым ссылкам (а вы рискуете получить предупреждение ;) ).
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 16313
Стаж: 17 лет 10 месяцев
Откуда: Красноярск
Благодарил (а): 533 раза
Поблагодарили: 2119 раз
Контактная информация:

[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/.
Вложения
posts_merging-develop-3.2.zip
(106.14 КБ) 43 скачивания
Изображение
Перенесено из форума Бета-версии расширений для phpBB 3.1.x в форум Анонсы и поддержка расширений для phpBB 3.1.x 24.03.2015 21:09 модератором rxu

Аватара пользователя
Sheer
Former team member
Сообщения: 12113
Стаж: 17 лет 1 месяц
Откуда: Калининград не Кенигсберг
Благодарил (а): 54 раза
Поблагодарили: 2752 раза
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение Sheer »

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

Re: [release] Posts merging by rxu

Сообщение some0ne »

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

Re: [release] Posts merging by rxu

Сообщение Gubkin »

some0ne писал(а): 06.02.2018 13:47Почему и для чего, кроме мокрого мозга?
удалится вообще сама настройка в форме ответа, не будет никаких окон , галочек и предложений
Ещё один тупой вопрос и будете забанены. К гуру надо приходить подготовленными, а не как Вы. Вчера создали форум с парой постов, а сегодня уже 20 вопросов )))
Аватара пользователя
Pazh
Former team member
Сообщения: 2317
Стаж: 14 лет 4 месяца
Благодарил (а): 43 раза
Поблагодарили: 506 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение 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));
		}
Помощь в ЛС/email только за WM или ЮMoney
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 16313
Стаж: 17 лет 10 месяцев
Откуда: Красноярск
Благодарил (а): 533 раза
Поблагодарили: 2119 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение rxu »

Pazh писал(а): 06.02.2018 14:24Для исправления, в данном расширении в файле ext\rxu\PostsMerging\event\listener.php найти код:
Не надо там искать этот код. При обновлении расширения он там опять появится.
И если переменная шаблона не присвоена, она и так false.
Изображение
Аватара пользователя
Pazh
Former team member
Сообщения: 2317
Стаж: 14 лет 4 месяца
Благодарил (а): 43 раза
Поблагодарили: 506 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение Pazh »

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 или ЮMoney
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 16313
Стаж: 17 лет 10 месяцев
Откуда: Красноярск
Благодарил (а): 533 раза
Поблагодарили: 2119 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение rxu »

Pazh писал(а): 06.02.2018 17:37этот же код вставляется расширением БО, но БЕЗ ПРОВЕРКИ ЭТИХ ЖЕ УСЛОВИЙ:
Это проблема экста БО, а не склейки.
Изображение
Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 5270
Стаж: 19 лет 2 месяца
Откуда: Питер
Благодарил (а): 188 раз
Поблагодарили: 800 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение Siava »

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

Ситуация:

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

Раньше вроде такого не было или просто не замечал :roll:
Еще одно нарушение правил и будете забанены. © Mr. Anderson
Ты очистил кеш? © Sheer
https://siava.ru (phpbb 2.0.x 3.5.x)
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 16313
Стаж: 17 лет 10 месяцев
Откуда: Красноярск
Благодарил (а): 533 раза
Поблагодарили: 2119 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение rxu »

Siava писал(а): 19.02.2018 20:424 уведомления по email после каждого добавления.
В смысле, на 1 добавление - 4? Т.е. 16 в общей сложности?
Siava писал(а): 19.02.2018 20:42Раньше вроде такого не было
Не было, раньше вообще никакие уведомления при склейке не отправлялись.
Изображение
Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 5270
Стаж: 19 лет 2 месяца
Откуда: Питер
Благодарил (а): 188 раз
Поблагодарили: 800 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение Siava »

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

Отправлено спустя 4 минуты 14 секунд:
Чуть позже отключу все расширения, проверю.
Еще одно нарушение правил и будете забанены. © Mr. Anderson
Ты очистил кеш? © Sheer
https://siava.ru (phpbb 2.0.x 3.5.x)
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 16313
Стаж: 17 лет 10 месяцев
Откуда: Красноярск
Благодарил (а): 533 раза
Поблагодарили: 2119 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение rxu »

Siava писал(а): 19.02.2018 21:08всего 4
Ну тогда вроде логично. Фактически 4 ответа же было. Если бы склейки не было, всё случилось бы точно также :)
Изображение
Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 5270
Стаж: 19 лет 2 месяца
Откуда: Питер
Благодарил (а): 188 раз
Поблагодарили: 800 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение Siava »

rxu, :shock: а мне казалось, что всегда приходило только одно уведомление - первое, неважно сколько после комментариев добавилось.. :)
Еще одно нарушение правил и будете забанены. © Mr. Anderson
Ты очистил кеш? © Sheer
https://siava.ru (phpbb 2.0.x 3.5.x)
Аватара пользователя
Татьяна5
Поддержка
Поддержка
Сообщения: 12372
Стаж: 12 лет 7 месяцев
Благодарил (а): 223 раза
Поблагодарили: 3517 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение Татьяна5 »

Да, одно уведомление обычно приходит (без расширений), до момента пока не просмотришь тему/форум
Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 5270
Стаж: 19 лет 2 месяца
Откуда: Питер
Благодарил (а): 188 раз
Поблагодарили: 800 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение Siava »

rxu, Re: Предложения и замечания
Есть идея как исправить парсер при склейке сообщений? :roll:
Еще одно нарушение правил и будете забанены. © Mr. Anderson
Ты очистил кеш? © Sheer
https://siava.ru (phpbb 2.0.x 3.5.x)
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 16313
Стаж: 17 лет 10 месяцев
Откуда: Красноярск
Благодарил (а): 533 раза
Поблагодарили: 2119 раз
Контактная информация:

Re: [release] Posts merging by rxu

Сообщение rxu »

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

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