Изменённые данные в форме

Когда-то я придумал для своего багтракинга фичу, которая мне казалась уникальной для багтракингов. Фича связана с тем, как обрабатываются изменения, которые человек сохраняет в тикет, в то время, как тикет уже успел поменяться кем-то другим за время с момента открытия того в окне браузера.

И вот на днях, в процессе переписывания TaCo на Джанго (да, я таки занялся) я обнаружил в джанговских формах фичу, про которую как-то нигде "официально" не рассказывали.

Задачка

Пусть у нас есть Маша и Саша, которые работают в багтракинге над одним проектом. Маша открывает баг про падение программы, и, словив озарение, начинает его тут же фиксить, потому что работы там на 10 минут. Через часок она понимает, что починить все так просто не получилось и решает отложить работу. Перед тем, как пойти налить кофейку, она переключается обратно в окно браузера с багом и переводить его в статус "In progress".

Беда в том, что в это же время Саша, который взял на себя обязанности пастуха багтракинга, тоже нашел этот баг и, справедливо подумав, что падение программы — проблема серьезная, адекватно поменял поле severity на "Major".

Так вот, если сабмит формы тикета обрабатывать стандартным образом, то Машин перевод тикета в "In Progress" помимо этого вернет ему severity в "Normal", потому что именно в таком виде она открыла эту форму час назад.

Решение

TaCo решал эту проблему. Вместо того, чтобы сохранять в тикет данные из формы целиком, он определял, какие именно поля в этой форме пользователь поменял, и применял только эти изменения. То есть, в нашем случае от Маши пришло бы только указание "поменять статус", и severity остался бы нетронутым.

Реализовано это самым очевидным способом: в форме вместе с видимыми юзеру полями рисуются их скрытые дубликаты с именами типа "OldSeverity", "OldOwner" и т.д. При сабмите из формы вынимаются поля и их старые значения, по которым выясняется, что поменялось.

Уголок буквоеда. Да, это все равно не устраняет race condition целиком. Но это и не цель. Мне нужно решить задачу правильного поведения в большинстве практических случаев. В остальных случаях люди просто вручную обновят тикет еще раз как надо.

Django

Я был очень рад и немало удивлен, когда увидел, что в Джанге такой функционал просто встроен в стандартные формы. Оказывается, совсем не идиотский подход я тогда "изобрел", а другие так тоже делают.

Включается это для отдельных полей указанием атрибута show_hidden_initial:

class TaskForm(forms.Form):
    severity = forms.ChoiceField(..., show_hidden_initial=True)
    owner = forms.ChoiceField(..., show_hidden_initial=True)
    # ...

После валидации у формы будет доступен form.changed_data — список с названиями измененных полей, который уже можно использовать по своему усмотрению. Например:

if form.is_valid():
    for fieldname in form.changed_data:
        setattr(obj, fieldname, form.cleaned_data[fieldname])
    obj.save()

Кажется, это хорошая иллюстрация полезности фреймворков. Такая фича — нудная мелочь, которую никто не будет реализовывать прямо сразу, а скорее всего оставит "на потом". А во фреймворке она просто есть, потому что кому-то в мире она уже понадобилась и он ее оформил патчем.

P.S. Я заодно подпилил TagsField, чтобы он поддерживал этот параметр. Правда не опубликовал еще.