Posts merging by rxu

Обсуждение новой системы расширения функциональности phpBB 3.3, разработки новых расширений.
Дмитрий Николаевич
Сообщения: 54
Зарегистрирован: 22 июл 2014, 09:46
Откуда: из мамы
Благодарил (а): 13 раз
Поблагодарили: 7 раз
Контактная информация:

Posts merging by rxu

Сообщение Дмитрий Николаевич »

Про это и говорим. "Добавление" же. А если посмотреть в сторону adv_posts_merging_2.1.2 (для phpbb2), там используется именно "Добавлено"

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

$lang['Merge_separator'] = "\r\n\r\n[color=green][size=9]Добавлено спустя%s%s%s%s:[/size][/color]\r\n\r\n";
Ну и Invision-ы вроде как "добавляют".

Вобщем вам решать, вы ж автор этих строк в EN локали

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

'MERGE_SEPARATOR'		=> "\n\n[size=85][color=green]Added in %s %s %s:[/color][/size]\n",

Аватара пользователя
rxu
Сообщения: 1116
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Благодарил (а): 84 раза
Поблагодарили: 108 раз
Контактная информация:

Posts merging by rxu

Сообщение rxu »

ОК, подумаю. Возможно, в этом есть смысл.

Аватара пользователя
rxu
Сообщения: 1116
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Благодарил (а): 84 раза
Поблагодарили: 108 раз
Контактная информация:

Posts merging by rxu

Сообщение rxu »

Код расширения обновлен.
Во фронтенде два основных изменения:
- убрано окно подтверждения отправки сообщения;
- добавлен контроль лимита количества вложений. Теперь, если сумма числа вложений в основном и добавляемом сообщении превысит лимит, склейки не произойдет (за исключением администраторов и модераторов).

Аватара пользователя
Alecto
Сообщения: 571
Зарегистрирован: 05 авг 2014, 21:41
Благодарил (а): 31 раз
Поблагодарили: 29 раз

Posts merging by rxu

Сообщение Alecto »

обновился. при входе в тему белый экран

Аватара пользователя
rxu
Сообщения: 1116
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Благодарил (а): 84 раза
Поблагодарили: 108 раз
Контактная информация:

Posts merging by rxu

Сообщение rxu »

См. лог ошибок. Попробуйте снова скачать и заменить файл event/listener.php, он исправлен буквально только что.

Аватара пользователя
Alecto
Сообщения: 571
Зарегистрирован: 05 авг 2014, 21:41
Благодарил (а): 31 раз
Поблагодарили: 29 раз

Posts merging by rxu

Сообщение Alecto »

обновился.
белый экран пропал, склейка ббкода выполнилась правильно.
спасибо.

Аватара пользователя
HAMMER663
Сообщения: 123
Зарегистрирован: 22 янв 2014, 09:23
Откуда: Москва
Github repo: https://github.com/HAMMER663
Благодарил (а): 11 раз
Поблагодарили: 17 раз

Posts merging by rxu

Сообщение HAMMER663 »

Если в QuickReply Включить ajax-отправку сообщений: НЕТ.
То при отправке сообщений вываливается

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

Fatal error: Uncaught exception 'Symfony\Component\DependencyInjection\Exception\InvalidArgumentException' with message 'The service definition "quote" does not exist.' in Z:\home\test1.ru\www\vendor\symfony\dependency-injection\Symfony\Component\DependencyInjection\ContainerBuilder.php:871 Stack trace: #0 Z:\home\test1.ru\www\vendor\symfony\dependency-injection\Symfony\Component\DependencyInjection\ContainerBuilder.php(478): Symfony\Component\DependencyInjection\ContainerBuilder->getDefinition('quote') #1 Z:\home\test1.ru\www\phpbb\notification\manager.php(874): Symfony\Component\DependencyInjection\ContainerBuilder->get('quote') #2 Z:\home\test1.ru\www\phpbb\notification\manager.php(854): phpbb\notification\manager->load_object('quote') #3 Z:\home\test1.ru\www\phpbb\notification\manager.php(345): phpbb\notification\manager->get_item_type_class('quote') #4 Z:\home\test1.ru\www\phpbb\notification\manager.php(339): phpbb\notification\manager->add_notifications('quote', Array, Array) #5 Z:\home\test1.ru\www\ext\rxu\PostsMer in Z:\home\test1.ru\www\vendor\symfony\dependency-injection\Symfony\Component\DependencyInjection\ContainerBuilder.php on line 871

Аватара пользователя
rxu
Сообщения: 1116
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Благодарил (а): 84 раза
Поблагодарили: 108 раз
Контактная информация:

Posts merging by rxu

Сообщение rxu »

HAMMER663, а в полной форме ответа так же?

Отправлено спустя 3 минуты 22 секунды:
Всё понятно, исправлено.

Аватара пользователя
HAMMER663
Сообщения: 123
Зарегистрирован: 22 янв 2014, 09:23
Откуда: Москва
Github repo: https://github.com/HAMMER663
Благодарил (а): 11 раз
Поблагодарили: 17 раз

Posts merging by rxu

Сообщение HAMMER663 »

Да. Теперь все работает ;)

Аватара пользователя
Alecto
Сообщения: 571
Зарегистрирован: 05 авг 2014, 21:41
Благодарил (а): 31 раз
Поблагодарили: 29 раз

Posts merging by rxu

Сообщение Alecto »

в финале работает.

Закрыто