From 87cf418d788c05adeaca0a31083f81bb51e52b98 Mon Sep 17 00:00:00 2001 From: Fabien Meghazi Date: Thu, 12 Jul 2012 14:12:46 +0200 Subject: [PATCH] [FIX] Fixed loop bug bzr revid: fme@openerp.com-20120712121246-m16bo4d0cw7o7oz4 --- addons/web/static/src/js/views.js | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/addons/web/static/src/js/views.js b/addons/web/static/src/js/views.js index 1197ca2b4c8..842536f5d67 100644 --- a/addons/web/static/src/js/views.js +++ b/addons/web/static/src/js/views.js @@ -71,17 +71,15 @@ instance.web.ActionManager = instance.web.Widget.extend({ }, on_breadcrumb_clicked: function(ev) { var $e = $(ev.target); + var id = $e.data('id'); var item; - for (var i = 0; i < this.breadcrumbs.length; i += 1) { + for (var i = this.breadcrumbs.length - 1; i >= 0; i--) { var it = this.breadcrumbs[i]; - if (it.id == $e.data('id')) { + if (it.id == id) { item = it; - continue; - } - if (item) { - // Remove items after selected item - this.remove_breadcrumb(i); + break; } + this.remove_breadcrumb(i); } var index = $e.parent().find('.oe_breadcrumb_item[data-id=' + $e.data('id') + ']').index($e); item.show(index, $e);