Code review comment for lp:~savoirfairelinux-openerp/banking-addons/loose-coupling

Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

Virgil,

1. This is a convention since v7. Earlier it was active:False
2. Readability + less usage of variables.
3. 'State' field has its own usage in OpenERP. Normally, they are not kept editable by end user.

Thanks.

« Back to merge proposal