← Back to team overview

banking-addons-team team mailing list archive

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