Bug #5011
Ajouter la gestion des déficiences sur la page des quotations
100%
Files
Associated revisions
History
Updated by Mikaël Ates over 10 years ago
- Subject changed from Créer une page qui permet de lister les patients qui n'ont pas de déficiences d'indiqués to Ajouter la gestion des déficiences sur la page des quotations
- Target version changed from 1.3 to 1.2 Page des quotations et déficiences
- Renomer le lien en "Quotations et déficiences"
- Ajouter une colonne ANAP qui affiche une coche s'il y a une déficience ANAP de renseignée (au moins une seule qui n'est pas à non).
- Permettre la recherche des patients sans quotation mises, sans quotations ANAP, ou sans les deux.
Updated by Serghei Mihai about 10 years ago
- File display-and-filter-anap-deficiencies.patch display-and-filter-anap-deficiencies.patch added
- % Done changed from 0 to 50
- Patch proposed changed from No to Yes
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
Updated by Benjamin Dauvergne about 10 years ago
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)
Updated by Serghei Mihai about 10 years ago
- Status changed from Nouveau to Résolu (à déployer)
- % Done changed from 50 to 100
Appliqué par commit calebasse|commit:feffc42192411db2c692e799642a2fd6d56b453a.
dossiers: display anap quotations and filter by them
Closes #5011