From c87181138ea4a8bef278ff02a69c3e5ef15f254e Mon Sep 17 00:00:00 2001 From: msh-openerp Date: Thu, 19 Apr 2012 11:18:48 +0530 Subject: [PATCH] [FIX]Fixed the issue of IE8 which doesn't syntax of Option, here we are not able to find the childnodes of option. bzr revid: msh@tinyerp.com-20120419054848-uhs9obcbsn8h6ucs --- addons/web/static/src/js/data_export.js | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/addons/web/static/src/js/data_export.js b/addons/web/static/src/js/data_export.js index 3d15a4e3607..ae9e4aeafdc 100644 --- a/addons/web/static/src/js/data_export.js +++ b/addons/web/static/src/js/data_export.js @@ -64,6 +64,10 @@ openerp.web.DataExport = openerp.web.Dialog.extend({ var $fmts = this.$element.find('#export_format'); _(formats).each(function (format) { var opt = new Option(format.label, format.tag); + //To fix IE8 Issue which doesn't support above syntax + if(!opt.childNodes.length){ + $(opt).html(format.label); + } if (format.error) { opt.disabled = true; opt.replaceChild( @@ -348,7 +352,12 @@ openerp.web.DataExport = openerp.web.Dialog.extend({ var field_list = this.$element.find('#fields_list'); if (this.$element.find("#fields_list option[value='" + field_id + "']") && !this.$element.find("#fields_list option[value='" + field_id + "']").length) { - field_list.append(new Option(string, field_id)); + var opt = new Option(string, field_id); + //To fix IE8 Issue which doesn't support above syntax + if(!opt.childNodes.length){ + $(opt).html(string); + } + field_list.append(opt); } }, get_fields: function() {