Merge pull request #51 from AddSearch/sc-8379/uri-encode-hierarchical-facets-and-range-facets

[sc-8379] encode hierarchicalFacets & rangeFacets parameter

Author: haoAddsearch

dist/addsearch-js-client.min.js

@@ -1,6 +1,6 @@
 {
   "name": "addsearch-js-client",
-  "version": "0.8.4",
+  "version": "0.8.5",
   "description": "AddSearch API JavaScript client",
   "main": "index.js",
   "jsdelivr": "./dist/addsearch-js-client.min.js",

package.json

@@ -82,8 +82,7 @@ var executeApiFetch = function(apiHostname, sitekey, type, settings, cb, fuzzyRe
         settingToQueryParam(settings.numFacets, 'numFacets') +
         settingToQueryParam(settings.cacheResponseTime, 'cacheResponseWithTtlSeconds') +
         settingToQueryParam(settings.searchOperator, 'defaultOperator') +
-        settingToQueryParam(settings.analyticsTag, 'analyticsTag') +
-        settingToQueryParam(settings.hierarchicalFacetSetting, 'hierarchicalFacets');
+        settingToQueryParam(settings.analyticsTag, 'analyticsTag');
 
       // Add sortBy and sortOrder
       if (Array.isArray(settings.paging.sortBy)) {
@@ -113,7 +112,12 @@ var executeApiFetch = function(apiHostname, sitekey, type, settings, cb, fuzzyRe
 
       // Range facets
       if (settings.rangeFacets) {
-        qs = qs + '&rangeFacets=' + JSON.stringify(settings.rangeFacets);
+        qs = qs + '&rangeFacets=' + encodeURIComponent(JSON.stringify(settings.rangeFacets));
+      }
+
+      // Hierarchical facets
+      if (settings.hierarchicalFacetSetting) {
+        qs = qs + '&hierarchicalFacets=' + encodeURIComponent(JSON.stringify(settings.hierarchicalFacetSetting));
       }
 
 

src/apifetch.js

@@ -176,7 +176,7 @@ var settings = function() {
   }
 
   this.addHierarchicalFacetSetting = function(setting) {
-    this.settings.hierarchicalFacetSetting = JSON.stringify(setting);
+    this.settings.hierarchicalFacetSetting = setting;
   }
 
   this.addRangeFacet = function(field, ranges) {