Страница 5 из 7

Posts merging by rxu

Добавлено: 19 ноя 2014, 21:08
Alecto
собственный тег IMW

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

Posts merging by rxu

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

Posts merging by rxu

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

Posts merging by rxu

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

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

Posts merging by rxu

Добавлено: 19 ноя 2014, 22:40
Алг
Есть, но мне по-другому нужно(вздыхая). Мне нужно создавать форумы( программно) уже с опцией не использовать склейку

Posts merging by rxu

Добавлено: 20 ноя 2014, 19:34
rxu
Добавлена опция отключения склейки на уровне сообщений.

Posts merging by rxu

Добавлено: 20 ноя 2014, 20:23
Alecto
ура!!!!!!!!!!!!!!!!!!!
щас затестируем )))

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

Изображение

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

Posts merging by rxu

Добавлено: 20 ноя 2014, 20:26
rxu
Скорее всего, тут что-то с экстом спасибок. Если синхронизировать счетчики?

Posts merging by rxu

Добавлено: 20 ноя 2014, 21:01
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, на основу можно ставить обновленную склейку? или лучше подождать пока еще ктото потестирует?

Posts merging by rxu

Добавлено: 20 ноя 2014, 21:03
rxu
Должно нормально работать, изменений в основной функционал практически не вносилось.