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
F19509575: D17656.diff
Sat, Jan 10, 10:27 PM
F19507808: D17656.diff
Fri, Jan 9, 11:25 PM
F19152124: D17656.id.diff
Dec 11 2025, 12:48 AM
F19071708: D17656.id.diff
Nov 30 2025, 8:26 PM
F19059623: D17656.diff
Nov 29 2025, 6:18 AM
F18851297: D17656.id.diff
Oct 30 2025, 11:38 PM
F18844551: D17656.diff
Oct 29 2025, 6:24 AM
F18734897: D17656.diff
Sep 30 2025, 11:05 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.