andrzej-jankowski/ralph — pull request #1 by vi4m (2012-11-26 00:17:42+00:00)
Title: Assets

Trochę się nudziłem dziś, więc nieco poprawek ode mnie.

Główna uwaga - proponuję zamiast dziedziczenia formularzy zrobić ich kompozycję. Szczególnie, że formularze django perfekcyjnie działają - jest do tego prefix dla Form() zapobiegający kolizjom nazw zmiennych. Ponadto zysk jest z dziedziczenia po ModelForm - jak widać DRY jest większe.

Formularze dodawania trzeba tylko przerobić na obsługę 2 formularzy(asset + device, asset + part), powinno to bardzo uprościć kod w views.py, teraz widoczna jest tylko 1 część.