Внимание!
Не все расширения для phpBB 3.2 совместимы с phpBB 3.3, главным образом из-за неверного синтаксиса в определениях сервисов (отсутствия обрамляющих кавычек - '...').
Перед обновлением необходимо убедиться в совместимости всех расширений.
Рекомендуется предварительно тестировать обновление на копии конференции (локально или на сервере).

[release] Posts merging by rxu

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

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

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

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

Аватара пользователя
Sheer
phpBB Guru
phpBB Guru
Сообщения: 11569
Стаж: 13 лет 4 месяца
Откуда: Калининград не Кенигсберг
Благодарил (а): 54 раза
Поблагодарили: 2611 раз

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
Стаж: 2 года 6 месяцев
Благодарил (а): 17 раз

Re: [release] Posts merging by rxu

Сообщение some0ne »

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.15
Сообщения: 1021
Стаж: 4 года 11 месяцев
Благодарил (а): 214 раз
Поблагодарили: 101 раз

Re: [release] Posts merging by rxu

Сообщение Gubkin »

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

Аватара пользователя
Pazh
Former team member
Сообщения: 2194
Стаж: 10 лет 8 месяцев
Благодарил (а): 41 раз
Поблагодарили: 455 раз

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 или ЯД

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 15132
Стаж: 14 лет 1 месяц
Откуда: Красноярск
Благодарил (а): 388 раз
Поблагодарили: 1737 раз

Re: [release] Posts merging by rxu

Сообщение rxu »

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

Аватара пользователя
Pazh
Former team member
Сообщения: 2194
Стаж: 10 лет 8 месяцев
Благодарил (а): 41 раз
Поблагодарили: 455 раз

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 или ЯД

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 15132
Стаж: 14 лет 1 месяц
Откуда: Красноярск
Благодарил (а): 388 раз
Поблагодарили: 1737 раз

Re: [release] Posts merging by rxu

Сообщение rxu »

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

Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 4438
Стаж: 15 лет 5 месяцев
Откуда: Питер
Благодарил (а): 127 раз
Поблагодарили: 521 раз

Re: [release] Posts merging by rxu

Сообщение Siava »

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.5.x)

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 15132
Стаж: 14 лет 1 месяц
Откуда: Красноярск
Благодарил (а): 388 раз
Поблагодарили: 1737 раз

Re: [release] Posts merging by rxu

Сообщение rxu »

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

Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 4438
Стаж: 15 лет 5 месяцев
Откуда: Питер
Благодарил (а): 127 раз
Поблагодарили: 521 раз

Re: [release] Posts merging by rxu

Сообщение Siava »

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

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

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 15132
Стаж: 14 лет 1 месяц
Откуда: Красноярск
Благодарил (а): 388 раз
Поблагодарили: 1737 раз

Re: [release] Posts merging by rxu

Сообщение rxu »

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

Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 4438
Стаж: 15 лет 5 месяцев
Откуда: Питер
Благодарил (а): 127 раз
Поблагодарили: 521 раз

Re: [release] Posts merging by rxu

Сообщение Siava »

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

Аватара пользователя
Татьяна5
Поддержка
Поддержка
Сообщения: 10114
Стаж: 8 лет 11 месяцев
Благодарил (а): 178 раз
Поблагодарили: 2816 раз

Re: [release] Posts merging by rxu

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

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

Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 4438
Стаж: 15 лет 5 месяцев
Откуда: Питер
Благодарил (а): 127 раз
Поблагодарили: 521 раз

Re: [release] Posts merging by rxu

Сообщение Siava »

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

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 15132
Стаж: 14 лет 1 месяц
Откуда: Красноярск
Благодарил (а): 388 раз
Поблагодарили: 1737 раз

Re: [release] Posts merging by rxu

Сообщение rxu »

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

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