Страница 22 из 28
[release] Posts merging by rxu
Добавлено: 09.01.2015 15:03
rxu
Расширение прошло валидацию на phpbb.com:
https://www.phpbb.com/customise/db/exte ... tsmerging/
Исходный код:
https://github.com/rxu/posts_merging
Копировать в папку
/ext/rxu/postsmerging/.
Re: [release] Posts merging by rxu
Добавлено: 05.02.2018 21:47
Sheer
some0ne, удалите папку ext\rxu\PostsMerging\styles и не сушите моск!
Re: [release] Posts merging by rxu
Добавлено: 06.02.2018 13:45
some0ne
rxu писал(а): ↑05.02.2018 18:10Какого ответа? В полной форме или в расширении быстрого ответа?
в форме Быстрого ответа
Снимок экрана20180206134159.png
Отправлено спустя 2 минуты 42 секунды:
Sheer писал(а): ↑05.02.2018 21:47some0ne, удалите папку
ext\rxu\PostsMerging\styles и не сушите моск!
Почему и для чего, кроме мокрого мозга?
Re: [release] Posts merging by rxu
Добавлено: 06.02.2018 14:09
Gubkin
some0ne писал(а): ↑06.02.2018 13:47Почему и для чего, кроме мокрого мозга?
удалится вообще сама настройка в форме ответа, не будет никаких окон , галочек и предложений
Re: [release] Posts merging by rxu
Добавлено: 06.02.2018 14:24
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));
}
Re: [release] Posts merging by rxu
Добавлено: 06.02.2018 17:23
rxu
Pazh писал(а): ↑06.02.2018 14:24Для исправления, в данном расширении в файле ext\rxu\PostsMerging\event\listener.php найти код:
Не надо там искать этот код. При обновлении расширения он там опять появится.
И если переменная шаблона не присвоена, она и так false.
Re: [release] Posts merging by rxu
Добавлено: 06.02.2018 17:37
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);
}
Дальше нужно разжовывать ответ на вопрос: Что будет, если установить право "Может игнорировать склейку" в "никогда"?
На гуру все аналогично...
Re: [release] Posts merging by rxu
Добавлено: 06.02.2018 17:45
rxu
Pazh писал(а): ↑06.02.2018 17:37этот же код вставляется расширением БО, но БЕЗ ПРОВЕРКИ ЭТИХ ЖЕ УСЛОВИЙ:
Это проблема экста БО, а не склейки.
Re: [release] Posts merging by rxu
Добавлено: 19.02.2018 20:42
Siava
rxu, phpbb 3.2.1 + PostsMerging 2.1.0 (последний с гитхаба).
Ситуация:
Я подписан на одну тему.
Другой пользователь пишет в неё. Через 3 минуты добавляет сообщение. Ещё через 6 минут добавляет сообщение. Ещё через 12 минут добавляет...
В итоге я получаю 4 уведомления по email после каждого добавления.
Раньше вроде такого не было или просто не замечал
Re: [release] Posts merging by rxu
Добавлено: 19.02.2018 21:03
rxu
Siava писал(а): ↑19.02.2018 20:424 уведомления по email после каждого добавления.
В смысле, на 1 добавление - 4? Т.е. 16 в общей сложности?
Siava писал(а): ↑19.02.2018 20:42Раньше вроде такого не было
Не было, раньше вообще никакие уведомления при склейке не отправлялись.
Re: [release] Posts merging by rxu
Добавлено: 19.02.2018 21:04
Siava
rxu, не.. всего 4. То есть после каждого добавления приходило новое уведомление об ответе.
Отправлено спустя 4 минуты 14 секунд:
Чуть позже отключу все расширения, проверю.
Re: [release] Posts merging by rxu
Добавлено: 19.02.2018 21:14
rxu
Siava писал(а): ↑19.02.2018 21:08всего 4
Ну тогда вроде логично. Фактически 4 ответа же было. Если бы склейки не было, всё случилось бы точно также
Re: [release] Posts merging by rxu
Добавлено: 19.02.2018 21:18
Siava
rxu,
а мне казалось, что всегда приходило только одно уведомление - первое, неважно сколько после комментариев добавилось..
Re: [release] Posts merging by rxu
Добавлено: 19.02.2018 22:03
Татьяна5
Да, одно уведомление обычно приходит (без расширений), до момента пока не просмотришь тему/форум
Re: [release] Posts merging by rxu
Добавлено: 13.03.2018 12:16
Siava
rxu,
Re: Предложения и замечания
Есть идея как исправить парсер при склейке сообщений?
Re: [release] Posts merging by rxu
Добавлено: 13.03.2018 16:58
rxu
Пока нет.