← Back to team overview

openerp-community-reviewer team mailing list archive

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