Skip to content

Commit 2f21663

Browse files
committed
fix(unified-search): prevent provider disabling on content filter apply
When date range or person filters were applied, providers that didn't support these filters were automatically disabled in the UI. This made the in-folder filter appear auto-applied and prevented users from searching non-compatible providers. Remove automatic provider disabling logic from updateDateFilter(), applyPersonFilter(), and removeFilter(). Content filters now apply only to compatible providers via existing compatibility checks while keeping all providers available for selection.
1 parent 00e1264 commit 2f21663

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

core/src/components/UnifiedSearch/UnifiedSearchModal.vue

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -567,10 +567,6 @@ export default defineComponent({
567567
this.filters[existingPersonFilter].name = person.displayName
568568
}
569569
570-
this.providers.forEach(async (provider, index) => {
571-
this.providers[index].disabled = !(await this.providerIsCompatibleWithFilters(provider, ['person']))
572-
})
573-
574570
this.debouncedFind(this.searchQuery)
575571
unifiedSearchLogger.debug('Person filter applied', { person })
576572
},
@@ -628,7 +624,6 @@ export default defineComponent({
628624
for (let i = 0; i < this.filters.length; i++) {
629625
if (this.filters[i].id === filter.id) {
630626
this.filters.splice(i, 1)
631-
this.enableAllProviders()
632627
break
633628
}
634629
}
@@ -669,9 +664,6 @@ export default defineComponent({
669664
this.filters.push(this.dateFilter)
670665
}
671666
672-
this.providers.forEach(async (provider, index) => {
673-
this.providers[index].disabled = !(await this.providerIsCompatibleWithFilters(provider, ['since', 'until']))
674-
})
675667
this.debouncedFind(this.searchQuery)
676668
},
677669

0 commit comments

Comments
 (0)