Aureplop/1 11/fix querysets #487

Merged

delobell merged 0 commits from aureplop/1_11/fix_querysets into supportBDS

2017-03-19 16:16:06 +01:00

delobell commented

2017-02-25 01:47:26 +01:00

(Migrated from git.eleves.ens.fr)

Fix vue kpsul_checkout_data.

Fix FieldError: Non-relational field given in select_related donné par la vue history_json.
Voir select_related() prohibits non-relational fields for nested relations.

!182 n'est plus nécessaire.

Fix vue ``kpsul_checkout_data``. Fix ``FieldError: Non-relational field given in select_related`` donné par la vue ``history_json``. Voir [select_related() prohibits non-relational fields for nested relations](https://docs.djangoproject.com/en/1.10/releases/1.10/#select-related-prohibits-non-relational-fields-for-nested-relations). !182 n'est plus nécessaire.

mpepin commented

2017-03-18 22:10:51 +01:00

(Migrated from git.eleves.ens.fr)

Good for me, juste deux remarques :

Tant que tu es dans history_json, pourquoi ne pas PEP8-iser cette vue ?
Si tu as la foi, il serait intéressant de commencer à écrire des test unitaires. Cette MR pourrait être l'occasion d'écrire un test pour cette vue. Juste histoire de tester qu'elle ne provoque pas d'erreur et que le résultat attendu n'est pas absurde.

Good for me, juste deux remarques : 1. Tant que tu es dans `history_json`, pourquoi ne pas PEP8-iser cette vue ? 2. Si tu as la foi, il serait intéressant de commencer à écrire des test unitaires. Cette MR pourrait être l'occasion d'écrire un test pour cette vue. Juste histoire de tester qu'elle ne provoque pas d'erreur et que le résultat attendu n'est pas absurde.

lstephan commented

2017-03-19 00:10:47 +01:00

(Migrated from git.eleves.ens.fr)

Sur le PEP8, history_json était PEP8-isée dans !132, donc (vu que j'ai merge) elle l'est dans Aufinal/refactor_history. PEP8-iser à deux endroits sert surtout à créer des conflits dans les merges (c'était la principale cause de conflits entre master et k-fet).

Sur le PEP8, `history_json` était PEP8-isée dans !132, donc (vu que j'ai merge) elle l'est dans `Aufinal/refactor_history`. PEP8-iser à deux endroits sert surtout à créer des conflits dans les merges (c'était la principale cause de conflits entre `master` et `k-fet`).

mpepin commented

2017-03-19 16:13:18 +01:00

(Migrated from git.eleves.ens.fr)

Ok pour PEP8, good point. J'attends juste de savoir si @delobell veut écrire un test unitaire pour cette vue et après je merge.

mpepin commented

2017-03-19 16:16:06 +01:00

(Migrated from git.eleves.ens.fr)

merged

mpepin commented

2017-03-19 16:16:06 +01:00

(Migrated from git.eleves.ens.fr)

mentioned in commit 33dedc7474