Review: Needs Information
Ça m'a l'air bon, perso. Il y a juste un truc qui m'a fait pauser, c'est
if (f['user_id'] and f['user_id'][0]) == vals.get('user_id', False)
qui est pas ultra clair à la première lecture. Mais en fait je vois pas trop
d'alternative qui soit notablement plus claire, elles ont l'air soit moins
claires soit notablement plus longues.
Ptet mettre un commentaire au dessus de cette ligne genre
# f.user_id is False and vals.user_id is False, or f.user_id.id ==
vals.user_id
if (f['user_id'] and f['user_id'][0]) == vals.get('user_id', False)
accessoirement je note un truc bizarre: ça va considérer que le user_id est
False par défaut, s'il est pas fourni. Me semblerait plus logique d'utiliser
uid non? D'ailleurs ça serait ptet aussi un bon défaut pour user_id, par défaut
les filtres sont créés pour le user courant sauf si on override avec un
``False`` explicite.
Par contre je sais pas si j'aime le changement de comportement au niveau du
résultat: avoir un id si création ou False si maj c'était pratique pour décider
si c'est une création ou une maj. Enfin d'un autre côté j'ai une map (id, name)
donc je peux toujours regarder là dedans pour voir si je dois ajouter une row
ou si j'en ai déjà une.
--
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-global-filters-odo/+merge/105458
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-server/trunk-global-filters-odo.
_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help : https://help.launchpad.net/ListHelp
