New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ошибка при создание шаблона #2
Comments
Я объясню почему в рантайме. Если посмотреть внимательно настройки, то можно увидеть что эти поля отключаются для редактировании после того как запись создана. Для чего? Если вдруг по своей глупости Вася или Петя, что работают в другом отделе вдруг случайно решат изменить slug и сломают критически важный шаблон для отправки почты. В итоге страдают все. В настройках имеется соответствующая константа. Жду пул реквеста :) Спасибо. PS - Кстати в модели можно валидировать при clean или валидатор к полю добавить. Что бы нельзя было вытворить нечто подобное в консольке питона. |
Зачем это понятно. Это надо решать на уровне отображение формы. Завтра пришлю. А то все ломается. |
Честно говоря пока еще подобных проблем не наблюдалось ни в одном проекте. Sentry на этот счет тоже молчит. Иначе бы починил уже. Каким образом создавался slug со значением None? |
да вот странно как-то я его ввел свой слаг, нажал сохранить и сохранилось с None я еще не разбирал, ясно одно что потом ломается класс админ-модели. |
|
Смог воспроизвести ошибку. Помню что в самом начале использовал get_prepopulated_fields и get_readonly_fields, но отказался, из-за того что в версиях ниже 1.7 эти методы не поддерживаются. Сейчас поправил ошибку. Проверяйте. |
Надо было править начиная со 140 строки, пул реквест отправил. Только травис ругается почему-то |
В любом случае, так как было, подменять нельзя. У кого ниже 1.7 пусть ручками вбивают слаг. |
Надо было ответвляться от development. Лучше сразу покрыть этот момент тестами. Раз уже наткнулись, что бы в дальнейшем не натыкаться на те же грабли. Ошибку воспроизводил таким образом:
|
Так какие дальнейшия действия? Мне перепулить? Или сам поправишь ? |
Давай лучше перепулишь в ветку development. За одно напиuи тестики в файлике test_cases.py. Будем карпеть над той веткой. После вольем. Кстати новые фитчи что ты хочешь, лучше писать так же с development в отдельной ветке. |
5 мин |
У меня есть сомнения что тест кейс покроет этот случай, нудно делать LiveTest чтобы удостовериться что в цикле тестирования используется один и тот же экземпляр админ-модели |
Это можно легко проверить используя старый код и новый. Сразу выявиться) Главное что бы один клиент был. Если не получится, то бог с ним. Главное функционал покрыть. Если писать плагины. |
Тревис все равно ругается, добавил в ридми инфомарцию по дев версии |
Странно что он ругается. Мой билд проходил без проблем. Локально пробовал tox запустить в корне проекта? |
Нет не пробовал но если ты посмотришь лог оно сломалось раньше моего пула |
Зря ты ридми тронул. У нас есть файл для этого CONTRIBUTING.rst. Поправь этот момент. Так же лучше убрать Development installation, так как есть Local demo installation. Так же описанный в документации http://django-db-mailer.readthedocs.org/dev_install.html и вот еще http://django-db-mailer.readthedocs.org/contributing.html. Пробегись по документации |
Ок, но позже - работать надо |
Оки. Не спешим. |
Стоит ветку локально подтянуть development. Было очень много новых коммитов. Если есть возможность, погоняй функционал плагинов, который ты так хотел:) |
Если ранее был создан шаблон с slug=None то класс MailTemplateAdmin ломается до след. перезапуска. Ошибка кроется скорей всего тут, позже покурю и пришлю пул реквест:
Вообще сама идея что-то менять в рантайме у ModelAdmin крайне опасная затея, так как в райнтайме используется всегда один экземпляр данного класса. Класс инстанциируется один раз и больше новых экземпляров не будет.
The text was updated successfully, but these errors were encountered: