Code review comment for lp://staging/~acespritech/openerp-web/6.1_bug_1006832_IntellecTSeedTechnologies

Revision history for this message
Xavier (Open ERP) (xmo-deactivatedaccount) wrote :

This code is not correct either (inArray returns −1 if it does not find, and the index of the match in all other cases, so this will work *only* if there is only one db and it matches otherwise it will never display error, test should be `!== −1`).

Rejecting this, because we'll be fixing original proposal instead.

review: Disapprove

« Back to merge proposal