when setting the url, use the plain path

Fixes #152
elastic · Jul 25, 2014 · d10af1e · d10af1e
1 parent 7ff55bf
commit d10af1e
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/kibana/directives/saved_object_finder.js b/src/kibana/directives/saved_object_finder.js
@@ -83,12 +83,14 @@ define(function (require) {
           var url = $scope.makeUrl(hit);
           if (!url || url.charAt(0) !== '#') return;
 
+          // angular wants the '/path', not '#/path'
           var path = url.substr(1);
           if ($route.matches(path)) {
             $event.preventDefault();
+
             // change works with paths, but we are only here because the paths
-            // are the same, so we have to change the whole url
-            $route.changeUrl(url);
+            // are the same, so we have to change the whole url to be the new path
+            $route.changeUrl(path);
           }
         };