Review: Needs Fixing

* There is still a few places where $element is set manually instead of using 
setElement() or replaceElement():
  - addons/web/static/src/js/view_form.js:4218
  - addons/web/static/src/js/view_form.js:4231
  - addons/web_kanban/static/src/js/kanban.js:716 (OldWidget)
  - addons/web_mobile/static/src/js/chrome_mobile.js:52 (OldWidget)

* Some part of the code assume that $element is always set (which is not the 
case anymore):
  - addons/web/static/src/js/chrome.js:978 (code do not execute do_login -> 
this.login not rendered)
  - addons/web/static/src/js/chrome.js:89
  - addons/web_dashboard/static/src/js/dashboard.js:250
  - ...

-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-backbonify-widgets-xmo/+merge/109844
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-backbonify-widgets-xmo.

_______________________________________________
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