Project

General

Profile

Bug #5011

Ajouter la gestion des déficiences sur la page des quotations

Added by Mikaël Ates almost 10 years ago. Updated over 9 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Start date:
20 June 2014
Due date:
% Done:

100%

Estimated time:
Patch proposed:
Yes
Planning:

Files

Associated revisions

Revision feffc421 (diff)
Added by Serghei Mihai almost 10 years ago

dossiers: display anap quotations and filter by them

Closes #5011

History

#1

Updated by Mikaël Ates almost 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.
#2

Updated by Mikaël Ates almost 10 years ago

  • Assignee set to Serghei Mihai
  • Patch proposed set to No
#3

Updated by Serghei Mihai almost 10 years ago

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

#4

Updated by Mikaël Ates almost 10 years ago

C'est ça.

#5

Updated by Benjamin Dauvergne almost 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)
#6

Updated by Serghei Mihai almost 10 years ago

  • Status changed from Nouveau to Résolu (à déployer)
  • % Done changed from 50 to 100

Appliqué par commit calebasse|commit:feffc42192411db2c692e799642a2fd6d56b453a.

#7

Updated by Mikaël Ates over 9 years ago

  • Status changed from Résolu (à déployer) to Fermé

Ok

Also available in: Atom PDF