Review: Disapprove
I could repeat my first comment almost as is...

Just below your modification, there is a call to file_open. That function has 
some logic to deal with the addons paths. There shouldn't be any reason to add 
more weird logic outside of it.
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-781967-ysa/+merge/62079
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/trunk-bug-781967-ysa.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-web
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-web
More help   : https://help.launchpad.net/ListHelp

Reply via email to