Review: Needs Fixing
J'ai fixe quelque points. Il reste:
* _logger.error('Exception in call: ' + tb_s) dans print_instance.py
devrait pouvoir être remplacé par _logger.exception et un meilleur message,
ça éviterait de formatter la traceback à la main. web_services.py fait pareil
(lignes ~520, ~700 et ~750)
* Parlant d'appelant, les loggers choisis pour rpc sont quand même bizarres,
pourquoi pas `openerp.netsvc.rpc.full` et `openerp.netsvc.rpc.short`? Et tester
le niveau sur `openerp.netsvc.rpc`, un truc du style? (je sais pas si ça marche
dans ce sens là, mais bon).
reponse: la critique est aisee :) je ne trouve pas ca genial non plus mais j'ai
rien trouve de mieux. en tout cas il faut un moyen de dire :DEBUG sauf rpc full
(qui est vraiment trop verbeux)
* sql_db.py~260, il y a des `_logger.log(logging.DEBUG, message)` au lieu de
`_logger.debug(message)`? Pareil lignes ~400, ~470
reponse: pas grave c'est equivalent c'est juste une question de style, c'etait
pour garder le patch moins intrusif mais j'ai rien contre qu'on uniformise le
style
* le `default=` sur --log-handler est normal? (il y a aussi un `my_default=`)
reponse: oui c'est pour optparse en fait je sais pas pq on utilise my_default
* Dans amount_to_text_en.py, du logging commenté a été remplacé (il peut
probablement être dégagé complètement)
--
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-logging-vmt/+merge/90785
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-server/trunk-logging-vmt.
_______________________________________________
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
