Дорогие друзья, здесь можно посрать в комменты, потому что я переделал детектор дублирования комментариев.
главное — не показывать код Зверьку. ‎· псы в рапиде
^ эти три точки поставлены сознательно! новый код это хэндлит, wow! ‎· псы в рапиде
это чо типа ночной чат что ли ‎· пингвинишко
@ayoshi, в результате я сделал старую добрую SQL-таблицу с уникальным индексом, и храню все к там, к чертям. в Redis нет соответствующих атомарных операций. ‎· псы в рапиде
@kolsanova, да, задача насрать столько же сколько и здесь: https://mokum.ru/squadette/17752 но при этом без (специальных) дублирований. ‎· псы в рапиде
ага, наблюдаю какие-то 404 Not Found, щас буду смотреть в логи. ‎· псы в рапиде
First and the last ‎· wisdom of wombats
Pep8 all the things ‎· wisdom of wombats
так, исправил 404, щас будет деплой и будут временные ошибки. ‎· псы в рапиде
tralala ‎· liquidgold
ну-тко ну-тко! ‎· псы в рапиде
Plenty of 404s (?!) ‎· breedable with other kitties
Пленти с какого момента? ‎· псы в рапиде
Sometimes (as in the previous comment, 404 happens just before the comment is successfully posted) ‎· breedable with other kitties
От 4 и дальше. Там, правда, случился интересный трюк — поскольку форма для комментария чистится не раньше чем запостится последний посланный, ‎· breedable with other kitties
да, предположительно, я это починил! ‎· псы в рапиде
ага, нихуя не починил. понял. ‎· псы в рапиде
То я открыл новую кликом по Comment ‎· breedable with other kitties
Оно тут же предложило мне выкинуть какие-то изменения ‎· breedable with other kitties
Ну и я согласился. На этом получил первый 404 ‎· breedable with other kitties
красота ‎· ☠☠☠☠☠☠
красота ‎· ☠☠☠☠☠☠
блабла ‎· ☠☠☠☠☠☠
и щас оно мне 404 в начале предложило ‎· ☠☠☠☠☠☠
Ну а дальше их было по одному на коммент. Но в общем значительно устойчивей пока — раньше я дубли рисовал на заказ, а теперь могу добиться тормозов и 404, но дублей пока не смог. ‎· breedable with other kitties
Поправка: форма исчезает, но можно не ждать, а открыть новую. Тут-то JS и спрашивает, в своем ли я уме. Если я в мэриэнином, то начинается путаница. А если в своем, то я еще не пробовал. ‎· breedable with other kitties
Вообще конечно надо по Enter сразу ставить disable на кнопку, даром что все хэндлится на ура. ‎· breedable with other kitties
Ну и "Discard changes" пока комментарий в полете — это не ок. Но ‎· breedable with other kitties
^ кажется, я не нажимал там Enter. Ну да и хрен с ним, от попапа досталось. В общем я к тому что это намного, намного лучше чем вчера. А придирки — ну, косметика, полезно пофиксить поскольку напрямую задевают UX, но на полет влияет постольку поскольку. ‎· breedable with other kitties
блаааа ‎· liquidgold
блааааа2 ‎· liquidgold
блааааа3 ‎· liquidgold
Вообще говоря, коллега, JS все так же болен. Я вижу 9 (!!!) одинаковых запросов на один комментарий. ‎· breedable with other kitties
Плюс еще один, первый, с form data _uuid=b664cb46-8d9b-4579-b569-2229e8822c20 _river_uuid="" (пустая строка) comment[text]=8 utf8=✓, который и словил 404 ‎· breedable with other kitties
У 9 (девяти) успешных запросов from data точно такие же. ‎· breedable with other kitties
Ага, то есть все же проблема исходно в том, что у меня что-то не так с колл-бэками, ага. Интересно! ‎· псы в рапиде
И с ноги — пиздык-хуяк ‎· breedable with other kitties
^ вот эта хуета породила 14 запросов, из них два словили 404. ‎· breedable with other kitties
так, кажется я починил еще что-то относящееся к 404 Not Found. Прошу записать в летописи, что я первый раз в жизни возвращаю 204 No Content. ‎· псы в рапиде
Девять? Nein! ‎· псы в рапиде
А у меня всего один! ‎· псы в рапиде
а нихуя, у меня четыре. ВОТ И ВСКРЫЛАСЬ ИСТИНА! ах ты сука! ‎· псы в рапиде
=))) Я тем временем извинюсь за неумение запостить скриншот. Почему-то command-shift-4 у меня перестало сохранять скриншоты, только щелкает прикольно. ‎· breedable with other kitties
та же хуйня возникает при редактировании комментов, блин. --chg ‎· псы в рапиде
Sarcasm? I's? Nooooo!!! Круто что поймалось, please do have fun ‎· breedable with other kitties
это многое объясняет! Если каждый юзер фактически делает четыре запроса, то понятно, почему МОКУМ ТОРМОЗИТ! ‎· псы в рапиде
Ну вот да, какой-то легкий пиздец на клиенте. ‎· breedable with other kitties
Четыре — это для слабаков. Щаз я потрекаю... ‎· breedable with other kitties
@alf, перестань, спасибо! я получил инфу, буду думать. ‎· псы в рапиде
20, двадцать запросов на одну единичку! Ага, отвалил, налью Batemans 2013 Vintage за успех. ‎· breedable with other kitties
я тоже! ‎· псы в рапиде
Слушай, а нафига ты новую сессионную куку на каждый ответ выставляешь? ‎· breedable with other kitties
<бегает глазами из стороны в сторону> ‎· псы в рапиде
какую еще куку? ‎· псы в рапиде
Set-Cookie _freefeed_session=YWtWcWtFVHNpdzFGK1owbzRIaEl1QUkvQ25QZWlWRDVVRW9QZVpvb0lwTytmUHpUeFJUaU0rTTRHdDhvUnY2T3cxWk5VcnQvMUVPb3hrRlJmTk4xU1hYK25DZGhXMGlXSkowMGJBMUdwaGFPOVVvV3V4Wm02UGlHTnd3U0RxZk1YTEgwdEIveWk0ZDFiYUFlelFaalpnbGNZSm9iNlRXSlFpaGRNUFI0eEhMTHBTRVZ0VHF6RUdtMFU5QUV1aFBhTDhidjM5czlHejNTeWZiMmxhYzR3YjRVMHloVjNYV3ZNUERNRFllQnk5cz0tLVdUVklwMlNnZXd3d3JvMEovaW5QOHc9PQ%3D%3D--e17801e1fd471e7e722043d5058d2932879345b2; path=/; HttpOnly ‎· breedable with other kitties
Такая вот фигня на каждый ответ. Каждый раз разная. ‎· breedable with other kitties
К слову, у меня такое ощущение будто количество запросов в точности равно номеру комментария (1-based, counting from last reload). Там нет случаем лишней подписки на событие при клике на Comment? Старые подписчики все отписались? ‎· breedable with other kitties
ничего себе. ‎· псы в рапиде
второй ‎· псы в рапиде
третий? ‎· псы в рапиде
да нет, там какой-то совершенно невменяемый пиздец. по-моему, мне нужна консультация Комбинирена! ‎· псы в рапиде
all I wanna do is get high on the beach. ‎· псы в рапиде
^ ain't we all? ‎· breedable with other kitties
ж ‎· 50% ash
o ‎· 50% ash
got 404 ‎· 50% ash
п ‎· 50% ash
а ‎· 50% ash
Таак. ‎· 50% ash
404, но коммент пришел, проверяю три updates, updates are clean ‎· 50% ash
фак! @alf крутой ‎· псы в рапиде
он круче всех мужчин ‎· псы в рапиде
и я пою — jquery blues ‎· псы в рапиде
в соседнем тредике только что удваивались ‎· earlyadopter
test ‎· 50% ash