no need to continue the loop when we have found the enries and prevent NPE

git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1359838 13f79535-47bb-0310-9956-ffa450edef68

2 files changed, 7 insertions, 1 deletions

diff --git a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/main.js b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/main.js
index 1d4bb9d8e..639edc1e2 100644
--- a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/main.js
+++ b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/main.js
@@ -269,6 +269,7 @@ function() {
                     ko.utils.arrayFirst(self.artifactMenuItems.concat(self.usersMenuItems, self.administrationMenuItems), function(p) {
                         if ( p.href == "#"+self.activeMenuId()) {
                           p.func();
+                          return;
                         }
                     });
                     
diff --git a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/search.js b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/search.js
index bbc4ba4e0..98299440a 100644
--- a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/search.js
+++ b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/search.js
@@ -43,7 +43,9 @@ define("archiva.search",["jquery","i18n","jquery.tmpl","choosen","knockout","kno
 
     breadCrumbEntries=function(){
       // root level ?
-      if (!self.parentBrowseViewModel) return [];
+      if (!self.parentBrowseViewModel){
+        return [];
+      }
       return calculateBreadCrumbEntries(self.groupId);
     }
 
@@ -69,6 +71,9 @@ define("archiva.search",["jquery","i18n","jquery.tmpl","choosen","knockout","kno
   }
 
   calculateBreadCrumbEntries=function(groupId){
+    if (!groupId){
+      return [];
+    }
     var splitted = groupId.split(".");
     var breadCrumbEntries=[];
     var curGroupId="";