Issue 502: Angularjs $location.search is not safe to use when passing…

… '#'.

This addresses the problem by replacing the `#` (U+0023) character with `%23` for relevant situations (properties in the search).

This needs to be tested for regressions, specifically checking for data in facets and filters that have the `#` (U+0023) character.

src/main/webapp/app/model/discoveryContextModel.js

@@ -64,20 +64,21 @@ sage.model("DiscoveryContext", function ($q, $location, $routeParams, Field, Man
 
     discoveryContext.before(function () {
       var filters = [];
+      var pattern = /#/;
 
       angular.forEach($routeParams, function(value, key) {
         if (key.match(/^f\./i)) {
           var filter = {
-            key: key.replace(/^f\./, ""),
+            key: key.replace(/^f\./, "").replace(pattern, "%23"),
             value: value
           };
           filters.push(filter);
         }
       });
 
       discoveryContext.search = new Search({
-        field: angular.isDefined($routeParams.field) ? $routeParams.field : "",
-        value: angular.isDefined($routeParams.value) ? $routeParams.value : "",
+        field: angular.isDefined($routeParams.field) ? $routeParams.field.replace(pattern, "%23") : "",
+        value: angular.isDefined($routeParams.value) ? $routeParams.value.replace(pattern, "%23") : "",
         label: "",
         filters: filters,
         start: 0,