/taxa filters allows empty array (as no-op)

olzraiti · olzraiti · commit ae9cdea1b4e9 · 2025-06-18T17:35:54.000+03:00
diff --git a/src/taxa/taxa-elastic-query.ts b/src/taxa/taxa-elastic-query.ts
@@ -181,7 +181,7 @@ const applyFiltersToElasticQuery = (filters: TaxaFilters = {}, filtersSchema: JS
 			if (typeof arg === "string") {
 				arg = [arg];
 			}
-			if (Array.isArray(arg) && typeof arg[0] === "string") {
+			if (Array.isArray(arg) && arg.every(a => typeof a === "string")) {
 				const inclusions: string[] = [];
 				const exclusions: string[] = [];
 				arg.forEach(subArg => (subArg[0] === "!" ? exclusions : inclusions).push(subArg.replace(/^!/, "")));

Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,7 @@ const applyFiltersToElasticQuery = (filters: TaxaFilters = {}, filtersSchema: JS`
`181`	`181`	`if (typeof arg === "string") {`
`182`	`182`	`arg = [arg];`
`183`	`183`	`}`
`184`		`- if (Array.isArray(arg) && typeof arg[0] === "string") {`
	`184`	`+ if (Array.isArray(arg) && arg.every(a => typeof a === "string")) {`
`185`	`185`	`const inclusions: string[] = [];`
`186`	`186`	`const exclusions: string[] = [];`
`187`	`187`	`arg.forEach(subArg => (subArg[0] === "!" ? exclusions : inclusions).push(subArg.replace(/^!/, "")));`