fix: arrIncludes autoremove filterFn (#5623)

lukebui · web-flow · commit 2efaf575f555 · 2025-02-11T14:56:58.000-06:00
The `filterValue` for this function is not an array. Therefore, we should not check for its length.
diff --git a/packages/table-core/src/filterFns.ts b/packages/table-core/src/filterFns.ts
@@ -50,7 +50,7 @@ const arrIncludes: FilterFn<any> = (
   return row.getValue<unknown[]>(columnId)?.includes(filterValue)
 }
 
-arrIncludes.autoRemove = (val: any) => testFalsey(val) || !val?.length
+arrIncludes.autoRemove = (val: any) => testFalsey(val)
 
 const arrIncludesAll: FilterFn<any> = (
   row,

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ const arrIncludes: FilterFn<any> = (`
`50`	`50`	`return row.getValue<unknown[]>(columnId)?.includes(filterValue)`
`51`	`51`	`}`
`52`	`52`
`53`		`-arrIncludes.autoRemove = (val: any) => testFalsey(val) \|\| !val?.length`
	`53`	`+arrIncludes.autoRemove = (val: any) => testFalsey(val)`
`54`	`54`
`55`	`55`	`const arrIncludesAll: FilterFn<any> = (`
`56`	`56`	`row,`