Review: Needs Fixing code review Hi Joao,
In the on_change methods (l.124,135,144), I propose to add the "context=None" argument and propagate it to the calls (l.138,147). Thus, if someone want to use the context in a module extending yours, he will be able just be inheriting the views. Apart that, LGTM -- https://code.launchpad.net/~savoirfairelinux-openerp/openerp-product-attributes/product_dependencies/+merge/185082 Your team Savoir-faire Linux' OpenERP is subscribed to branch lp:~savoirfairelinux-openerp/openerp-product-attributes/product_dependencies. -- Mailing list: https://launchpad.net/~savoirfairelinux-openerp Post to : [email protected] Unsubscribe : https://launchpad.net/~savoirfairelinux-openerp More help : https://help.launchpad.net/ListHelp
