Posts merging by rxu

Обсуждение новой системы расширения функциональности phpBB 3.3, разработки новых расширений.
Аватара пользователя
Alecto
Сообщения: 571
Зарегистрирован: 05 авг 2014, 21:41
Благодарил (а): 31 раз
Поблагодарили: 29 раз

Posts merging by rxu

Сообщение Alecto »

собственный тег IMW

Отправлено спустя 1 минуту 21 секунду:
вот пример темы http://motobratva.com/viewtopic.php?f=7&t=4117

Аватара пользователя
Татьяна5
Сообщения: 570
Зарегистрирован: 02 фев 2014, 15:05
Благодарил (а): 33 раза
Поблагодарили: 80 раз

Posts merging by rxu

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

rxu писал(а):QR_BBPOST надо провести кучу проверок на то, можно ли это сделать в принципе, для того, чтобы не вводить в заблуждение пользователя
На 3.0 работало без проверки
Опция, на самом деле, очень удобная

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28
Благодарил (а): 36 раз
Поблагодарили: 97 раз
Контактная информация:

Posts merging by rxu

Сообщение Алг »

rxu писал(а):На самом деле, не всё так просто, как кажется.
Чтобы показать опцию "Не склеивать с предыдущим", надо провести кучу проверок на то, можно ли это сделать в принципе, для того, чтобы не вводить в заблуждение пользователя. Это лишний SQL запрос и дополнительная нагрузка на страницу.
Надо подумать, имеет ли смысл городить этот огород ради одной галки на странице.
А мне нужно, чтобы была такая опция на уровне форума. Тогда на форумы без склейки будет меньшая нагрузка

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

Posts merging by rxu

Сообщение rxu »

Татьяна5 писал(а):QR_BBPOST На 3.0 работало без проверки
Это неправильно. В таком случае галка будет появляться независимо от того, чье последнее сообщение в теме и прошел ли период склейки, например.

Отправлено спустя 26 секунд:
Алг писал(а):QR_BBPOST А мне нужно, чтобы была такая опция на уровне форума.
Она есть на уровне и тем, и форумов. См. админку ;)

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28
Благодарил (а): 36 раз
Поблагодарили: 97 раз
Контактная информация:

Posts merging by 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 »

ура!!!!!!!!!!!!!!!!!!!
щас затестируем )))

Отправлено спустя 15 минут 59 секунд:
на первый взгляд вроде работает.
но, есть одно НО... у одного из расклеяных постов появилась благодарность от другого пользователя
сами понимаете, что на тесте ДРУГИХ пользователей НЕТ, а я ее не ставил, да и время какое-то стремное.

Изображение

проделывал те же или похожие действия несколько раз в других темах - не повторил.

Аватара пользователя
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 »

нажал обновление счетчиков - вывалилась ошибка

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

phpBB Debug] PHP Notice: in file [ROOT]/ext/gfksx/ThanksForPosts/acp/acp_thanks_refresh_module.php on line 106: Use of undefined constant THANKS_TABLE - assumed 'THANKS_TABLE'
Вернуться на главную страницу	
Общая ошибка
SQL ERROR [ mysqli ]

Table 'motobra2_mb.THANKS_TABLE' doesn't exist [1146]

SQL

DELETE FROM THANKS_TABLE WHERE post_id IN ('25808', '31191', '56592', '32855', '33866', '36753', '36753', '26848', '26848', '36770', '36753', '37137', '37287', '37525', '37602', '36571', '36570', '38758', '38915', '37602', '44258', '39840', '41119', '41443', '25749', '41690', '41789', '41789', '41793', '41803', '41911', '42106', '42106', '42111', '42251', '42251', '42251', '42251', '42251', '42251', '42251', '23864', '42251', '42106', '43144', '43315', '37287', '43946', '43946', '43946', '44185', '44188', '44191', '44188', '44194', '44194', '44194', '44194', '44194', '44258', '44239', '44258', '44258', '44258', '44320', '44320', '44258', '44601', '44725', '45052', '26722', '46310', '46310', '46310', '46310', '46254', '46884', '46884', '47116', '47115', '47116', '47115', '45725', '47115', '46884', '47115', '47116', '47115', '47116', '47545', '47487', '47761', '48039', '48275', '48270', '48275', '48275', '48275', '48270', '48275', '48275', '46310', '41793', '46355', '48951', '48951', '49099', '49099', '49158', '49099', '48951', '49350', '49350', '49350', '49276', '49350', '49417', '49417', '49417', '49428', '49384', '48979', '49609', '49586', '49692', '49942', '49948', '50097', '50096', '50097', '50121', '51043', '51043', '51253', '51253', '51321', '51401', '51619', '51984', '51984', '52277', '52336', '52356', '52542', '52356', '43532', '53473', '53640', '54095', '54095', '54095', '54294', '54547', '54095', '54812', '54812', '54812', '55094', '55396', '56120', '56324', '56449', '56494', '56536', '56536', '56568', '56561', '56981', '56981', '57314', '57314', '57314', '57659', '57768', '57768', '57668', '57651', '57659', '57660', '57675', '57685', '57686', '58448', '58649', '58665', '58665', '58665', '59082', '59082', '59082', '59082', '59264', '59474', '62286', '30822', '62744', '62797', '62843', '62843', '62872', '62884', '63738', '63738', '63738', '64237', '6129', '26844', '67602', '67634', '54238', '69405', '70042', '70191', '70582', '70685', '70685', '70572', '71420', '71439', '74586', '74543', '74745', '74772', '74748', '74758', '74763', '74764', '74781', '74747', '74937', '74931', '74993', '74477', '74568', '75037', '75020', '74993', '75075', '74802', '74610', '74700', '75163', '75075', '74802', '74610', '75137', '75152', '75148', '75144', '75142', '75135', '75163', '75171', '75173', '75123', '75171', '75173', '75185', '75202', '75185', '75200', '75183', '75191', '75202', '74721', '74600', '74589', '74597', '74747', '75164', '75208', '75206', '75214', '75164', '75202', '75208', '75221', '75211', '75232', '75235', '75232', '75235', '75244', '75158', '74451', '75249', '75252', '75256', '75255', '75263', '75259', '75263', '75249', '75252', '75256', '75255', '75252', '75259', '75255', '75256', '75249', '75232', '75274', '75277', '75275', '75275', '75286', '75288', '75287', '75285', '75072', '75284', '75277', '75295', '75294', '75294', '75259', '75331', '75331', '75331', '75331', '75337', '75343', '75347', '75343', '75347', '75354', '75357', '75358', '75359', '75360', '75360', '75359', '75358', '75357', '75354', '75359', '75343', '75347', '75354', '75357', '75358', '75360', '75366', '75071', '75371', '75379', '75367', '75384', '75390', '75379', '75384', '75367', '75371', '75342', '75404', '75439', '75429', '74610', '74802', '75075', '75163', '75429', '75439', '75436', '75439', '75456', '75455', '75284')

BACKTRACE
Отправлено спустя 7 минут 31 секунду:
rxu, на основу можно ставить обновленную склейку? или лучше подождать пока еще ктото потестирует?

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

Posts merging by rxu

Сообщение rxu »

Должно нормально работать, изменений в основной функционал практически не вносилось.

Закрыто