From e87f417f779d3d60e8e4966736dab5a7554d8fee Mon Sep 17 00:00:00 2001 From: Vincent Cua Date: Tue, 14 Jan 2020 12:37:39 -0800 Subject: [PATCH 1/2] Changes how refinement parameters are set to support newer versions of OCAPI --- src/api/ProductSearchApi.js | 30 +++++++++++++++++++++++++----- 1 file changed, 25 insertions(+), 5 deletions(-) diff --git a/src/api/ProductSearchApi.js b/src/api/ProductSearchApi.js index 906606e..a2f646c 100644 --- a/src/api/ProductSearchApi.js +++ b/src/api/ProductSearchApi.js @@ -67,7 +67,6 @@ export default class ProductSearchApi { const pathParams = {} const queryParams = { q: opts.q, - refine: this.apiClient.buildCollectionParam(opts.refine, 'csv'), sort: opts.sort, start: opts.start, count: opts.count, @@ -75,6 +74,11 @@ export default class ProductSearchApi { currency: opts.currency, locale: opts.locale } + if (Array.isArray(opts.refine)) { + opts.refine.forEach((value, index) => { + queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) + }) + } const headerParams = {} const formParams = {} @@ -155,12 +159,16 @@ export default class ProductSearchApi { const pathParams = {} const queryParams = { q: opts.q, - refine: this.apiClient.buildCollectionParam(opts.refine, 'csv'), sort: opts.sort, start: opts.start, count: opts.count, locale: opts.locale } + if (Array.isArray(opts.refine)) { + opts.refine.forEach((value, index) => { + queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) + }) + } const headerParams = {} const formParams = {} @@ -236,12 +244,16 @@ export default class ProductSearchApi { const pathParams = {} const queryParams = { q: opts.q, - refine: this.apiClient.buildCollectionParam(opts.refine, 'csv'), sort: opts.sort, start: opts.start, count: opts.count, locale: opts.locale } + if (Array.isArray(opts.refine)) { + opts.refine.forEach((value, index) => { + queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) + }) + } const headerParams = {} const formParams = {} @@ -318,13 +330,17 @@ export default class ProductSearchApi { const pathParams = {} const queryParams = { q: opts.q, - refine: this.apiClient.buildCollectionParam(opts.refine, 'csv'), sort: opts.sort, start: opts.start, count: opts.count, currency: opts.currency, locale: opts.locale } + if (Array.isArray(opts.refine)) { + opts.refine.forEach((value, index) => { + queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) + }) + } const headerParams = {} const formParams = {} @@ -399,12 +415,16 @@ export default class ProductSearchApi { const pathParams = {} const queryParams = { q: opts.q, - refine: this.apiClient.buildCollectionParam(opts.refine, 'csv'), sort: opts.sort, start: opts.start, count: opts.count, locale: opts.locale } + if (Array.isArray(opts.refine)) { + opts.refine.forEach((value, index) => { + queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) + }) + } const headerParams = {} const formParams = {} From 77d32bc0d2ad3c8ec7739b23d2d3fcc6e8112a38 Mon Sep 17 00:00:00 2001 From: Vincent Cua Date: Tue, 14 Jan 2020 12:57:28 -0800 Subject: [PATCH 2/2] Fix some lint errors --- src/api/ProductSearchApi.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/api/ProductSearchApi.js b/src/api/ProductSearchApi.js index a2f646c..ba7c0e5 100644 --- a/src/api/ProductSearchApi.js +++ b/src/api/ProductSearchApi.js @@ -76,7 +76,7 @@ export default class ProductSearchApi { } if (Array.isArray(opts.refine)) { opts.refine.forEach((value, index) => { - queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) + queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) }) } const headerParams = {} @@ -166,7 +166,7 @@ export default class ProductSearchApi { } if (Array.isArray(opts.refine)) { opts.refine.forEach((value, index) => { - queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) + queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) }) } const headerParams = {} @@ -251,7 +251,7 @@ export default class ProductSearchApi { } if (Array.isArray(opts.refine)) { opts.refine.forEach((value, index) => { - queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) + queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) }) } const headerParams = {} @@ -338,7 +338,7 @@ export default class ProductSearchApi { } if (Array.isArray(opts.refine)) { opts.refine.forEach((value, index) => { - queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) + queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) }) } const headerParams = {} @@ -422,7 +422,7 @@ export default class ProductSearchApi { } if (Array.isArray(opts.refine)) { opts.refine.forEach((value, index) => { - queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) + queryParams[`refine_${index + 1}`] = this.apiClient.paramToString(value) }) } const headerParams = {}