From 172f16cf52e13edd2a5f66fe54992e5754590f3e Mon Sep 17 00:00:00 2001
From: Antony Lesuisse <al@openerp.com>
Date: Mon, 30 Apr 2012 01:35:39 +0200
Subject: [PATCH] fix sub menu

bzr revid: al@openerp.com-20120429233539-kcpla847o21uktc6
---
 addons/web/static/src/js/chrome.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/addons/web/static/src/js/chrome.js b/addons/web/static/src/js/chrome.js
index fde173df9ae..5b47434a3ce 100644
--- a/addons/web/static/src/js/chrome.js
+++ b/addons/web/static/src/js/chrome.js
@@ -659,19 +659,21 @@ instance.web.Menu =  instance.web.Widget.extend({
             var action_id = $item.data('action-id');
             // If first level menu doesnt have action trigger first leaf
             if (!action_id) {
-                if(this.$element.has($item)) {
+                if(this.$element.has($item).length) {
                     $sub_menu = this.$secondary_menus.find('.oe_secondary_menu[data-menu-parent=' + id + ']');
                     $items = $sub_menu.find('a[data-action-id]').filter('[data-action-id!=""]');
-                    if($items) {
+                    if($items.length) {
                         action_id = $items.data('action-id');
                         id = $items.data('menu');
                     }
-                }
+                } 
             }
             this.open_menu(id);
             this.current_menu = id;
             this.session.active_id = id;
-            this.trigger('menu_click', action_id, id, $item);
+            if (action_id) {
+                this.trigger('menu_click', action_id, id, $item);
+            }
         }
     },
     /**