DGNum/gestioCOF

Fork 2

Clean the `bda.views.catalogue` view #516

Merged

mpepin merged 0 commits from bda_pep8 into master

2017-04-06 21:24:20 +02:00

mpepin commented

2017-04-05 00:51:59 +02:00

(Migrated from git.eleves.ens.fr)

Rewrite the error handling
Add some tests

- Rewrite the error handling - Add some tests

delobell commented

2017-04-05 18:48:40 +02:00

(Migrated from git.eleves.ens.fr)

Tirage.DoesNotExist plutôt

`Tirage.DoesNotExist` plutôt

delobell commented

2017-04-05 18:55:01 +02:00

(Migrated from git.eleves.ens.fr)

Ici aussi. Et d'ailleurs l'exception est levée uniquement si le tirage n'est pas trouvé. Alors que le message d'erreur laisse entendre que d'autres éléments devraient être pris en compte.

mpepin commented

2017-04-05 20:44:21 +02:00

(Migrated from git.eleves.ens.fr)

Indeed

mpepin commented

2017-04-05 20:45:25 +02:00

(Migrated from git.eleves.ens.fr)

Ce n'est pas très précis effectivement.

J'ai été paresseux. Je vais faire une passe de nettoyage un peu plus poussée sur cette vue.

Ce n'est pas très précis effectivement. J'ai été paresseux. Je vais faire une passe de nettoyage un peu plus poussée sur cette vue.

mpepin commented

2017-04-05 20:54:43 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

853a239e - Rewrite bda.views.catalogue

Compare with previous version

added 1 commit * 853a239e - Rewrite bda.views.catalogue [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/212/diffs?diff_id=383&start_sha=a26298317054ec6c33c2d0b470e6be135d1b1d13)

mpepin commented

2017-04-05 20:55:52 +02:00

(Migrated from git.eleves.ens.fr)

marked as a Work In Progress

marked as a **Work In Progress**

mpepin commented

2017-04-05 20:55:52 +02:00

(Migrated from git.eleves.ens.fr)

changed title from {-PEP8 compliance in bda.views-} to {+WIP: Clean the bda.views.catalogue view+}

changed title from **{-PEP8 compliance in bda.views-}** to **{+WIP: Clean the `bda.views.catalogue` view+}**

delobell commented

2017-04-05 21:13:09 +02:00

(Migrated from git.eleves.ens.fr)

tirage.spectacle_set.all() apparaît deux fois.
Tu dois pouvoir économiser encore plus en ne faisant qu'un seul appel à:
tirage.spectacle_set.values_list('id', flat=True) qui évite l'instanciation des objets spectacles (à moins que django ne le fasse directement auquel cas, clap clap clap)

`tirage.spectacle_set.all()` apparaît deux fois. Tu dois pouvoir économiser encore plus en ne faisant qu'un seul appel à: `tirage.spectacle_set.values_list('id', flat=True)` qui évite l'instanciation des objets spectacles (à moins que django ne le fasse directement auquel cas, clap clap clap)

mpepin commented

2017-04-05 21:15:11 +02:00

(Migrated from git.eleves.ens.fr)

Bien vu.

Je doute que Django fasse ce genre d'optimisation… Mais effectivement, si c'est le cas 👏 👏 👏 ^^

Bien vu. Je doute que Django fasse ce genre d'optimisation… Mais effectivement, si c'est le cas :clap: :clap: :clap: \^\^

delobell commented

2017-04-05 21:17:45 +02:00

(Migrated from git.eleves.ens.fr)

Ouais le .all() doit tout foutre en l'air mais sans il pourrait peut-être réussir à find the way...

Ouais le `.all()` doit tout foutre en l'air mais sans il pourrait peut-être réussir à find the way...

delobell commented

2017-04-05 21:17:46 +02:00

(Migrated from git.eleves.ens.fr)

resolved all discussions

mpepin commented

2017-04-05 22:03:26 +02:00

(Migrated from git.eleves.ens.fr)

added 3 commits

ff9cee5f - line too long in bda.models
9ec151e0 - A query optimization in bda.views.catalogue
981759f5 - Adds tests covering the bda-catalogue JSON API

Compare with previous version

added 3 commits * ff9cee5f - line too long in bda.models * 9ec151e0 - A query optimization in bda.views.catalogue * 981759f5 - Adds tests covering the bda-catalogue JSON API [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/212/diffs?diff_id=385&start_sha=853a239e6ec6525cd360b39c49db6c0601dd11bd)

mpepin commented

2017-04-05 22:03:42 +02:00

(Migrated from git.eleves.ens.fr)

unmarked as a Work In Progress

unmarked as a **Work In Progress**

delobell commented

2017-04-06 21:24:21 +02:00

(Migrated from git.eleves.ens.fr)

merged

delobell commented

2017-04-06 21:24:23 +02:00

(Migrated from git.eleves.ens.fr)