From a081cdf79bb1ea53997a3c8a36deaaa857e9e160 Mon Sep 17 00:00:00 2001 From: "Yogesh (OpenERP)" Date: Wed, 16 Nov 2011 14:53:08 +0530 Subject: [PATCH] [IMP] vieweditor :- improve code in fetch tagname on edit node. bzr revid: ysa@tinyerp.com-20111116092308-9iv5coeqbu1cgdsb --- addons/web/static/src/js/view_editor.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/addons/web/static/src/js/view_editor.js b/addons/web/static/src/js/view_editor.js index fbdaa6ed7de..9bbfbdf8f9a 100644 --- a/addons/web/static/src/js/view_editor.js +++ b/addons/web/static/src/js/view_editor.js @@ -458,12 +458,12 @@ openerp.web.ViewEditor = openerp.web.Widget.extend({ case "side-remove": break; case "side-edit": - var tr = $(this).closest("tr[id^='viewedit-']").find('a').text(); - var tag = _.detect(_.keys(_PROPERTIES),function(res){ - return _.includes(tr, res); - }); - var properties = _PROPERTIES[tag]; - self.on_edit_node(properties, clicked_tr_id, one_object, view_id, view_xml_id, clicked_tr_level); + var xml_arch = QWeb.load_xml($(this).closest("tr[id^='viewedit-']").find('a').text()); + var tag_name = xml_arch.childNodes? xml_arch.childNodes[0].tagName: false; + if (tag_name) { + var properties = _PROPERTIES[tag_name]; + self.on_edit_node(properties, clicked_tr_id, one_object, view_id, view_xml_id, clicked_tr_level); + } break; case "side-up": while (1) {