Review: Needs Fixing What's the point of `!this.dataset.ids.length ? true : false` instead of just `!this.dataset.ids.length`, which already returns `true` or `false`?
Or even better, using underscore and writing `_(this.dataset.ids).isEmpty()`? -- https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-573171-cpa/+merge/99714 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openerp-web/6.1-opw-573171-cpa. _______________________________________________ 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
