openerp-community-reviewer team mailing list archive

["Holger Brunn \(Therp\)" <hbrunn@xxxxxxxx>]

Review: Needs Fixing

You should put library files into /static/lib (why? If another module uses the same library, it would be loaded twice otherwise and is likely to clash)

Further, #2492 means we can't have apostrophes or quotes in the input mask. Why don't you simply eval() this attribute? This way, the user can fill in either a simple input mask or an object as applicable?

What would be nice is if you set the standard OpenERP error class (oe_form_invalid) if the user's input doesn't match the mask.
-- 
https://code.launchpad.net/~aristobulo/web-addons/web_fields_masks/+merge/219084
Your team Web-Addons Core Editors is subscribed to branch lp:web-addons.