Page MenuHomePhabricator

Don't filter "Closed" results (disabled users, bots, mailing lists) out of global typeahead
AbandonedPublic

Authored by epriestley on Apr 11 2017, 7:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 8:36 AM
Unknown Object (File)
Fri, Apr 5, 8:21 PM
Unknown Object (File)
Mon, Apr 1, 11:15 AM
Unknown Object (File)
Fri, Mar 22, 11:19 PM
Unknown Object (File)
Feb 11 2024, 1:58 PM
Unknown Object (File)
Jan 30 2024, 7:09 AM
Unknown Object (File)
Dec 23 2023, 7:08 PM
Unknown Object (File)
Dec 23 2023, 2:01 PM
Subscribers

Details

Summary

Ref T12538. Currently, we filter disabled users, mailing lists, and bots out of the global search typehead unless only disabled users, mailing lists and bots appear in the results.

This rule interacts badly with other rules when a query matches both projects and bot/mailing list users.

There are probably broader fixes available here but I think this resolves the immediate issue without too much collateral damage.

Test Plan

Screen Shot 2017-04-11 at 12.08.07 PM.png (178×417 px, 16 KB)

Diff Detail

Repository
rP Phabricator
Branch
global1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16396
Build 21811: Run Core Tests
Build 21810: arc lint + arc unit

Event Timeline

do they only show up for exact matches like other typeaheads?

I don't think "exact match" is actually the rule anywhere.

This change makes this typeahead work less like other typeaheads than it did before this change.

The rule for most typeaheads is "only show closed results if the entire result set is closed". This removes that rule, because of the bad interaction it has in T12538.

This rule is probably not the right rule to use anywhere, but it is more obviously wrong here than in other contexts.

Can we still filter disabled items? I'd agree bots and mailing lists should show, but I don't want old projects / etc surfacing... unless I'm missing something?

epriestley added a subscriber: swisspol.

I'll do this correctly when I get a chance, but you have to handle @swisspol.