Skip to content

Commit 3609381

Browse files
author
Derek Legenzoff
authored
Merge pull request #35 from jeff-ywang/jeff-ywang-patch-1
Update RemoveFilter function within facets.js
2 parents da72250 + db81961 commit 3609381

File tree

1 file changed

+9
-13
lines changed
  • 02 - Web UI Template/CognitiveSearch.UI/wwwroot/js

1 file changed

+9
-13
lines changed

02 - Web UI Template/CognitiveSearch.UI/wwwroot/js/facets.js

+9-13
Original file line numberDiff line numberDiff line change
@@ -39,18 +39,14 @@ function UpdateFilterReset() {
3939
}
4040

4141
function RemoveFilter(facet, value) {
42-
// Remove a facet
43-
var result = selectedFacets.filter(function (f) { return f.key === facet; })[0];
44-
45-
if (result) { // if that facet exists
46-
var idx = selectedFacets.indexOf(result);
47-
48-
if (result.value.length <= 1) {
49-
selectedFacets.pop(result);
50-
}
51-
else {
52-
result.value.pop(value);
53-
}
42+
var result = selectedFacets.filter(function (f) { return f.key === facet; });
43+
if (result.length > 0) {
44+
//Remove the facet value
45+
result[0].value = result[0].value.filter(v => v !== value);
46+
47+
//Remove facet if there is no value
48+
if (result[0].value.length === 0)
49+
selectedFacets = selectedFacets.filter(v => v.key !== facet);
5450
}
5551

5652
Search();
@@ -144,4 +140,4 @@ function ChooseFacet(facet, value, position) {
144140
}
145141
currentPage = 1;
146142
Search();
147-
}
143+
}

0 commit comments

Comments
 (0)