From 5294b046d9d5efc24486158a2b5d9d4238d8d5c7 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Fri, 25 Mar 2011 14:23:51 +0100 Subject: [PATCH] [FIX] don't call /base/menu/action when the current menu does not have an id bzr revid: xmo@openerp.com-20110325132351-tqzzmgqj9ldwzv54 --- addons/base/static/openerp/js/base_chrome.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/addons/base/static/openerp/js/base_chrome.js b/addons/base/static/openerp/js/base_chrome.js index ddbb3f92b29..3b9c1b0eb00 100644 --- a/addons/base/static/openerp/js/base_chrome.js +++ b/addons/base/static/openerp/js/base_chrome.js @@ -479,7 +479,11 @@ openerp.base.Menu = openerp.base.Controller.extend({ this.$secondary_menu.find('.menu_accordion').hide(); // TODO: ui-accordion : collapse submenus and expand the good one $secondary.show(); - this.rpc('/base/menu/action', {'menu_id': id}, this.on_menu_action_loaded); + + if (id) { + this.rpc('/base/menu/action', {'menu_id': id}, + this.on_menu_action_loaded); + } $('.active', this.$element.add(this.$secondary_menu)).removeClass('active'); $parent.addClass('active');