Состоялся релиз системы управления контентом MLUG

А я ещё ни разу не встречал идиота и чтобы не с виндой.
Все идиоты поголовно пользуются виндой, т.к. изначально читали правильные книги.

Либо ты до конца выходных напишешь такой камент к любой статье на МЛУГе, либо ты — самое толстое трепло ЛОРа за всю его историю.

Ладно бы ещё это было php, где ничего нет подобного.

4.2

Там есть PSR-1, PSR-2, которых придерживаются большинство новых проектов. Инструменты, позволяющие автоматически отформатировать код так, чтобы он соответствовал этим рекомендациям тоже имеются.

Угу, это баг в рассчётах, буду исправлять.
И про высоту таба тоже надо что-нибудь придумать.

админку, управление пользователями и правами, как создается и редактируется контент

Конечно, можно развернуть у себя клон репозитория, создать суперпользователя и посмотреть.
В будущем возможно сделаю демо админки, но пока она к этому не готова.

Да, я уже заметил, исправлю

Это фича

$ while $(true); do curl -v -e "https://mlug.ru/cms/demo" --cookie "csrftoken=$CSRFTOKEN" -d "csrfmiddlewaretoken=$CSRFTOKEN&uuid=$UUID&back_url=%2Fcms%2Fresults&option=3" https://mlug.ru/polls/; done

Значение $CSRFTOKEN и $UUID берёте из кода странички.

csrfmiddlewaretoken=$CSRFTOKEN

Мне кажется, что CSRFTOKEN может быть любым, не обязательно тем, что на той странице.

Тебе кажется

Мне больше интересно, в чем он результаты опроса хранит, и будет-ли там переполнение, и как оно будет выглядить.

После 2147483647 сбросится в -2147483648 и будет ходить кругами пока не надоест. https://bitbucket.org/mlug/mlug.ru-django/src/e8d1e46f8944c9101df7646c2826c04...

HTML = '<p></p>'
HTML вперемешку с питоном

Aww, ностальгия прям нахлынула! Web 1.0, cgi-скрипты на перле, романтика.

Хранит в IntegerField, вроде бы джанга должна выкинуть исключение при попытке переполнить (?), он исключения не ловит и будет 500 на POST-запрос и счётчик остановится на максимальном значении. Наверно.

Там и помимо переполнения дофига проблем. https://bitbucket.org/mlug/mlug.ru-django/src/e8d1e46f8944c9101df7646c2826c04...

option.votes_counter += 1
option.save()

Привет гонкам.

Что интересного нашел - если в качестве 'option' указать '/0', то оно возвращает 500. ТС, оно у тебя там на нуль делит, что-ли?

Ну, вот, дожили. Даже у php'шников появились стандарты и они их придерживаются. А Питонисты всю жизнь имеют pep8 и не придерживаются его.

Просто вместо того, чтобы запихнуть option (и заодно ID опроса) в URL и валидировать роутером, ТС решил передавать их в POST и валидировать только ID опроса. А 500 из-за того, что питон пытается скастовать в int и не может.

cms/utils.py:get_next_2nd_friday_or_last_saturday Можешь начинать переписывать.

Многие питонисты считают, что 79-символов-на-строку — драконовское ограничение и вписывают строки в 119 символов. Это впритык совместимо с гитхабом, да и в гуглестиле примерно то же.

Это всё потому что у них до сих пор не было инструментов, которые могли бы отформатировать код автоматически. Сейчас вот погуглил, нашёл пару штук, которые появились совсем недавно, да и те inspired by gofmt. Так-то.

Нет, HTML вперемешку с питоном — это вот так: https://bitbucket.org/mlug/mlug.ru-django/src/e8d1e46f8944c9101df7646c2826c04...

Есть же mako, jinja2, chameleon или что там в django сейчас используется. Нет, хотим жрать говно гипертекстовую лапшу.

Лол)

И есть .format или %. Ну и implied line continuation вместо плюсов. Но PEP8 же никто не читал.

но во множестве мест в исходном коде жестко прописан https

Но почему? В django же есть {% url 'view' %} и reverse('view', args=(arg,)) и т.п.

Потому, что Состоялся релиз системы управления контентом MLUG (комментарий) и ТС не в курсе, что можно не указывать протокол в URL вообще.

Алсо

Это демонстрационная страница, просто чтобы было проще посомтреть как оно выглядит и работает.
посомтреть

Привет гонкам.

Обоснуй-ка, мне интересно стало.

Вообще, после этого треда довольно забавно смотрятся твои постоянные попердывания о ценниках в Job.

У ТС, кстати, админка (или что это у него) поехала, поле «логин» криво отображается. https://i.imgur.com/mqzlaiX.png

Из базы берётся значение счётчика для опции, инкрементируется и сохраняется. Когда процессов/потоков больше одного — получаем, например,

1. Процесс1 взял значение счётчика 300
2. Процесс2 взял значение счётчика 300
3. Процесс1 инкрементировал и получил 301
4. Процесс2 инкрементировал и получил 301
5. Процесс2 сохранил 301 в базу
6. Процесс1 сохранил 301 в базу

Потерян 1 голос.

Чтобы такого не было, в джанге есть F-expressions, но это ж нужно осиливать документацию.

В случае нормальной бд (постгрес) и нормального уровня изоляции транзакций будет ошибка сериализации и повтор попытки.

Джанга не умеет раскрывать += в атомарный update (неудивительно: мало ли что ты там делаешь с полем). stackoverflow говорит точно то же самое.

Я просто написал перед комментом про F-expressions. Ну постгрес все равно должен защитить от подобного, другое дело, что по дефолту там кажись не SERIALIZABLE, и повтор нужно руками писать.

Ох лол. Да ещё и форматирование строк не используется.

Алсо почему у тебя при каждом вызове вот этой твоей странички куки каждый раз заново генерируются, хотя они у тебя только через год протухают?

Не-а. python автоматом скастует к long, он не умеет в integer overflow по дефолту.

Вы только посмотрите на этого мамкиного борцуна со стандартами.

Мне вообще его код вот это чем-то напомнил.

он исключения не ловит и будет 500

Вот и посмотрим что будет, 0.0005% до переполнения уже успели пройти
И посмотрим что будет раньше — я протестирую, напишу юниттесты и закрою возможные дырки или какой-то толстый с лора переполнит буфер и завалит меня репортами о пятисотой ошибке.

Оно пытается сделать ему int(), но не получается. Я лоханулся и не обработал этот вариант, должно быть 400, исправлю.

именно так, будем валидировать в следующей версии

Да, надо рефакторить

Потому что.
Для функционирования не критично и даже не важно, в будущем переделаю как надо.

Нельзя не указывать протокол для внешней ссылки.
Внутренние ссылки вообще локальные все.

Это к млугу не относится, там много чего поехало.

А ещё более глупый вопрос ты придумать можешь?

Ну тогда и исправлять нечего

Можно писать <a href=«//example.com»>вот так</a>. ЛОР-парсер так не может, но браузеры поймут и по щелчку будет переход по тому же протоколу, что и на странице, где эта ссылка расположена.

Не пори херни и просто посмотри в каких местах используется хардкод 'https://'