Corrige la liste des impayés dans bda #344

Merged

mpepin merged 0 commits from Kerl/fix_43_unpaid into master

2016-07-01 02:04:09 +02:00

mpepin commented

2016-06-17 02:07:24 +02:00

(Migrated from git.eleves.ens.fr)

La liste des gens n'ayant pas payé leurs places pour un tirage ne
contient plus les participants n'ayant pas eu de place.

Ajoute un compte des impayés dans le template.

Fixes #43

La liste des gens n'ayant pas payé leurs places pour un tirage ne contient plus les participants n'ayant pas eu de place. Ajoute un compte des impayés dans le template. Fixes #43

bclement commented

2016-06-21 22:45:04 +02:00

(Migrated from git.eleves.ens.fr)

Une raison d'utiliser des nnbsp plutôt que des nbsp ?

bclement commented

2016-06-21 22:47:42 +02:00

(Migrated from git.eleves.ens.fr)

Pourquoi pas unpaid = tirage.participant_set.annotate(nb_attributions=Count('attribution')).filter(paid=False, nb_attributions__gte=0) ?
Par ailleurs, je me demande à la lecture de ce code pourquoi le flag paid est sur Participant et pas sur Attribution, ce qui semblerait plus logique.

Pourquoi pas `unpaid = tirage.participant_set.annotate(nb_attributions=Count('attribution')).filter(paid=False, nb_attributions__gte=0)` ? Par ailleurs, je me demande à la lecture de ce code pourquoi le flag `paid` est sur `Participant` et pas sur `Attribution`, ce qui semblerait plus logique.

mpepin commented

2016-06-22 02:07:23 +02:00

(Migrated from git.eleves.ens.fr)

À la première question : pourquoi pas, c'est plus performant je suppose si tu me proposes ça.

Pour l'attribut paid : parce que les gens viennent payer toutes leurs places d'un coup. Et sauf
cas très exceptionnel, ils les obtiennent toutes en même temps, pas besoin de mettre un paid
par attribution.

À la première question : pourquoi pas, c'est plus performant je suppose si tu me proposes ça. Pour l'attribut `paid` : parce que les gens viennent payer toutes leurs places d'un coup. Et sauf cas très exceptionnel, ils les obtiennent toutes en même temps, pas besoin de mettre un `paid` par attribution.

mpepin commented

2016-06-22 02:15:08 +02:00

(Migrated from git.eleves.ens.fr)

Il me semblait que c'était la règle mais manifestement pas pour :

Il me semblait que c'était la règle mais manifestement pas pour `:`

mpepin commented

2016-06-22 02:16:44 +02:00

(Migrated from git.eleves.ens.fr)

Added 1 commit: