Review: Needs Fixing

Hello,

12      +        clickable = this.node.attrs.clickable;
13      +        var result = true;
14      +        if(clickable == undefined) result = false;
15      +        if(result == true && clickable.toLowerCase() == 'true' || 
clickable == '1')

correct with
clickable = this.node.attrs.clickable;
if(clickable != undefined && clickable.toLowerCase() in ['true', '1'])

Thanks
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-stages-clickable-apa/+merge/117240
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-stages-clickable-apa.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to