Review: Needs Fixing
Il y a encore 2-3 appels de logging bizarres, je sais pas si ça peut être
considéré bloquant:
* _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)
* dans loglevels, les noms de loggers avec des parenthèses dedans ça me semble
un peu un abus du mécanisme, et ça rend la sélection de tous les loggers
deprecated plus difficile. Pourquoi ne pas juste ajouter un
`openerp.deprecated.` juste avant le nom du canal? Comme ça on peut tracer tous
les appels à netsvc via le logger `openerp.deprecated`, simple et efficace.
- même bizarrerie dans orm.py, pourquoi __name__$(schema) plutôt que __name__
+ '.schema'?
* Dans openerp/netsvc.py, un `format` est ajouté, mais le précédent (sur la
ligne d'avant) n'est pas enlevé alors qu'il pourrait difficilement être utilisé
* Même fichier, ~ligne 190, les lignes dans logconfig ne sont pas des entiers
donc `i` c'est franchement pourri (et `prefix` est discutable, c'est un nom de
logger pas un préfixe de quoi que ce soit)
* `netsvc.log` est enlevé et remplacé par dispatch_rpc_log, sauf qu'il est
encore utilisé (genre dans ir_cron~135). Et dans la conversion, on perd le
niveau configurable. Et il faudrait laisser le `logger.isEnabledFor` (avec un
`DEBUG` hardcodé), ça éviterait de faire le même boulot dans l'appelant.
- 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)
* le niveau `logging.DEBUG_SQL` est enlevé, mais encore utilisé
(web_services.py~570)
* sql_db.py~260, il y a des `_logger.log(logging.DEBUG, message)` au lieu de
`_logger.debug(message)`? Pareil lignes ~400, ~470
* Dans amount_to_text_en.py, du logging commenté a été remplacé (il peut
probablement être dégagé complètement)
* print qui traine dans config.py
* le `default=` sur --log-handler est normal? (il y a aussi un `my_default=`)
La conf m'a l'air OK de loin (en lisant juste le code), il serait cool de
pouvoir la faire via fichier aussi, genre pour le saas.
--
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
