Posts merging by rxu
Posts merging by rxu
собственный тег IMW
Отправлено спустя 1 минуту 21 секунду:
вот пример темы http://motobratva.com/viewtopic.php?f=7&t=4117
Отправлено спустя 1 минуту 21 секунду:
вот пример темы http://motobratva.com/viewtopic.php?f=7&t=4117
Posts merging by rxu
На 3.0 работало без проверкиrxu писал(а):QR_BBPOST надо провести кучу проверок на то, можно ли это сделать в принципе, для того, чтобы не вводить в заблуждение пользователя
Опция, на самом деле, очень удобная
Posts merging by rxu
А мне нужно, чтобы была такая опция на уровне форума. Тогда на форумы без склейки будет меньшая нагрузкаrxu писал(а):На самом деле, не всё так просто, как кажется.
Чтобы показать опцию "Не склеивать с предыдущим", надо провести кучу проверок на то, можно ли это сделать в принципе, для того, чтобы не вводить в заблуждение пользователя. Это лишний SQL запрос и дополнительная нагрузка на страницу.
Надо подумать, имеет ли смысл городить этот огород ради одной галки на странице.
- rxu
- Сообщения: 1125
- Зарегистрирован: 21 янв 2014, 21:20
- Откуда: Krasnoyarsk
- Github repo: https://github.com/rxu
- Контактная информация:
Posts merging by rxu
Это неправильно. В таком случае галка будет появляться независимо от того, чье последнее сообщение в теме и прошел ли период склейки, например.Татьяна5 писал(а):QR_BBPOST На 3.0 работало без проверки
Отправлено спустя 26 секунд:
Она есть на уровне и тем, и форумов. См. админкуАлг писал(а):QR_BBPOST А мне нужно, чтобы была такая опция на уровне форума.
Posts merging by rxu
Есть, но мне по-другому нужно(вздыхая). Мне нужно создавать форумы( программно) уже с опцией не использовать склейку
- rxu
- Сообщения: 1125
- Зарегистрирован: 21 янв 2014, 21:20
- Откуда: Krasnoyarsk
- Github repo: https://github.com/rxu
- Контактная информация:
Posts merging by rxu
Добавлена опция отключения склейки на уровне сообщений.
Posts merging by rxu
ура!!!!!!!!!!!!!!!!!!!
щас затестируем )))
Отправлено спустя 15 минут 59 секунд:
на первый взгляд вроде работает.
но, есть одно НО... у одного из расклеяных постов появилась благодарность от другого пользователя
сами понимаете, что на тесте ДРУГИХ пользователей НЕТ, а я ее не ставил, да и время какое-то стремное.
проделывал те же или похожие действия несколько раз в других темах - не повторил.
щас затестируем )))
Отправлено спустя 15 минут 59 секунд:
на первый взгляд вроде работает.
но, есть одно НО... у одного из расклеяных постов появилась благодарность от другого пользователя
сами понимаете, что на тесте ДРУГИХ пользователей НЕТ, а я ее не ставил, да и время какое-то стремное.
проделывал те же или похожие действия несколько раз в других темах - не повторил.
- rxu
- Сообщения: 1125
- Зарегистрирован: 21 янв 2014, 21:20
- Откуда: Krasnoyarsk
- Github repo: https://github.com/rxu
- Контактная информация:
Posts merging by rxu
Скорее всего, тут что-то с экстом спасибок. Если синхронизировать счетчики?
Posts merging by rxu
нажал обновление счетчиков - вывалилась ошибка
Отправлено спустя 7 минут 31 секунду:
rxu, на основу можно ставить обновленную склейку? или лучше подождать пока еще ктото потестирует?
Код: Выделить всё
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
rxu, на основу можно ставить обновленную склейку? или лучше подождать пока еще ктото потестирует?
- rxu
- Сообщения: 1125
- Зарегистрирован: 21 янв 2014, 21:20
- Откуда: Krasnoyarsk
- Github repo: https://github.com/rxu
- Контактная информация:
Posts merging by rxu
Должно нормально работать, изменений в основной функционал практически не вносилось.