banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #00562
Re: [Merge] lp:~therp-nl/banking-addons/ba70-fix_payment_detection into lp:banking-addons/banking-addons-70
Review: Needs Fixing
I think
try:
from openerp.addons import $module_name
except:
...
is a cleaner (and more pythonic) approach to detect the availability of a module
And are lines 18f an artifact of your prerequisite-frenzy or do they have anything to do with the issue at hand?
--
https://code.launchpad.net/~therp-nl/banking-addons/ba70-fix_payment_detection/+merge/162358
Your team Banking Addons Team is subscribed to branch lp:banking-addons/banking-addons-70.
Follow ups
References