[OS-1729] Checklist > extract the personal data checklist status from the admission to the candidate #3054
Conversation
jcougnaud
force-pushed
the
feature/OS-1729
branch
from
4129fb2 to
f30d8d3
Compare
jcougnaud
force-pushed
the
feature/OS-1693
branch
from
73ef30c to
cc420fd
Compare
jcougnaud
force-pushed
the
feature/OS-1693
branch
from
cc420fd to
2ea68ac
Compare
jcougnaud
force-pushed
the
feature/OS-1729
branch
5 times, most recently
from
b461fc1 to
8aef431
Compare
templatetags/admission.py
Outdated
|
|
||
|
|
||
| @register.filter | ||
| def get_attr(obj, attribute): |
There was a problem hiding this comment.
je pense que tu peux mettre ceci dans base.templatetags.getattr
There was a problem hiding this comment.
En fait, je ne savais pas que ce filtre existait déjà mais je pense que je peux le reprendre directement ici, cela évitera de créer un nouveau filtre assez proche.
|
|
||
|
|
||
| @register.filter | ||
| def get_item(dictionary, value): |
There was a problem hiding this comment.
et ceci dans base.templatetags.dictionnar
There was a problem hiding this comment.
Ce serait une bonne idée effectivement mais l'inconvénient est qu'on a déjà un filtre appelé de la même manière dans base.templatetags.dictionnary qui ne fait pas la même chose (celui dans admission renvoie la clé par défaut, celui dans base retourne None). Que préfères-tu qu'on fasse ici ?
jcougnaud
force-pushed
the
feature/OS-1729
branch
from
8aef431 to
a0b80eb
Compare
…comment from the admission to the candidate
jcougnaud
force-pushed
the
feature/OS-1729
branch
from
a0b80eb to
70126a3
Compare
|
|
||
|
|
||
| @register.filter | ||
| def get_item(dictionary, value): |
There was a problem hiding this comment.
@register.filter
def get_item(dictionary, key, value=None):
return dictionary.get(key, value)Si on fait ca dans base; cela serait pas ok?
et toi tu devrais l'appeler ainsi get_item key key ?
There was a problem hiding this comment.
Je ne suis pas sûr qu'on puisse passer 3 arguments à un filtre. Ou alors il faudrait utiliser un simple_tag (comme admission.templatetags.admission.get_item_or_default) ?
There was a problem hiding this comment.
Fais au plus simple :) on peux laisser en 'létat du coup
jcougnaud
force-pushed
the
feature/OS-1693
branch
from
2ea68ac to
fe20555
Compare
No description provided.