Erledigt* Keine Datenbanken, pyc, oder __pycache__ Ordner einchecken
Wird gemacht* Tests schreiben
EbensoPEP-008 anschauen
Gut. Was haeltst du denn generell von meinen todo_* views? Ich habe das Gefuehl, das sei nicht elegant geloest, und koennte vielleicht mit einem Generic Class Based View gemacht werden. Bloss habe ich da nicht so richtig raus, wie ich die anwende.Sachen wie https://github.com/orschiro/todoli/blob ... ews.py#L32 lieber über einen Update query erledigen oder .save(update_fields=['status'])
Tendentiell würde sich ein DetailView (https://docs.djangoproject.com/en/dev/r ... detailview) anbieten, bei dem du POST überschreibst, übrigens liefert int(request.POST['todo_pk']) einen ServerError sobald todo_pk kein Integer ist. Ich würde auch undo/done zu einem View zusammenfassen ala:orschiro hat geschrieben: Gut. Was haeltst du denn generell von meinen todo_* views? Ich habe das Gefuehl, das sei nicht elegant geloest, und koennte vielleicht mit einem Generic Class Based View gemacht werden. Bloss habe ich da nicht so richtig raus, wie ich die anwende.
Code: Alles auswählen
def todo_change(request, action, slug):
action = True if action=='done' else False
...
Ups, als ich das schrieb machte es noch mehr SinnBlackJack hat geschrieben:Das ``if``/``else`` ist unnötig, mit ``action = action == 'done'`` hätte man das selbe Ergebnis. Wobei hier unschön ist, dass der Name an derart unterschiedliche Typen gebunden wird.
