Produits Entr'ouvert » Calebasse

C'est ça.

Serghei Mihai a écrit :

Si je comprend bien, les quotations ANAP sont les déficiences et il faut filtrer d'après ce champs.
Le filtre "sans quotations" existait déjà, j'ai rajouté celui pour les quotations ANAP

Bizarre que l'URL de retour aux dossiers soit le point, cela devrait être '..' si l'URL est bien backslashée comme il faut. En général il faut préférer les {% url "" %} plutôt que des URLs relatives.

Pour le formulaire de recherche et la checkbox ce serait bien d'utiliser une sous-classe de django.forms.Form plutôt que de faire des request.GET.without_quotations == 'on'. Ça simplifierait le code du template et le rendrait plus robuste. De même pour le listing, voir du coté de django-tables2 qui est très pratique, il permet de définir un tableau avec pagination via une classe comme pour un formulaire ou un modèle.

Il me semble que ce code:

853            anap = reduce(lambda f1, f2: f1 or f2, deficiencies)

est équivalent à ça:

anap = any(deficiencies)

Je mettrai la liste des champs de déficience au niveau du modèle calculé une fois pour toute dans une constante:

Model.DEFICIENCY_FIELDS = [field for field in Model._meta.get_all_field_names() if field.startswith('deficiency_')]

Ceci:

qs = qs.filter(mises_1__isnull=True).filter(mises_2__isnull=True).filter(mises_3__isnull=True)

peut s'écrire plus succinctement ainsi:

qs = qs.filter(mises_1__isnull=True, mises_2__isnull=True, mises_3__isnull=True)