openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #03834
Re: [Merge] lp:~camptocamp/purchase-wkfl/7.0-fix-button-visibility into lp:purchase-wkfl
Review: Needs Fixing code review
As for Pedro's first review, the last '|' looks out of place.
If we call the conditions A, B, C, D, the expression is now
| A | B | C | D
Where I think it should be
| A | B | C D
because implicitly it is grouped as follows:
| A ( | B ( | C D ) )
thanks!
--
https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-fix-button-visibility/+merge/205304
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.
References