Page MenuHomePhabricator

Remove ancient "PhabricatorQuickSearchEngineExtension" compatibility class
ClosedPublic

Authored by epriestley on May 9 2019, 6:14 PM.
Tags
None
Referenced Files
F15529492: D20509.diff
Tue, Apr 22, 8:12 PM
F15502305: D20509.diff
Sun, Apr 13, 11:42 PM
F15438573: D20509.id48902.diff
Wed, Mar 26, 1:39 AM
F15427125: D20509.id48938.diff
Mar 23 2025, 1:02 PM
F15426503: D20509.id.diff
Mar 23 2025, 9:10 AM
F15402273: D20509.diff
Mar 17 2025, 9:42 PM
F15378040: D20509.diff
Mar 13 2025, 12:18 PM
F15282796: D20509.diff
Mar 4 2025, 2:28 AM
Subscribers
None

Details

Summary

Ref T5378. This class was renamed more than a year ago, in D19087. Remove the leftover compatiblity layer.

Test Plan

grep

Diff Detail

Repository
rP Phabricator
Branch
sref1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22775
Build 31237: Run Core Tests
Build 31236: arc lint + arc unit

Event Timeline

I didn't go all the way down the rabbit hole on this, but it's not immediately obvious to me that this is actually relevant to T5378.

This revision is now accepted and ready to land.May 16 2019, 12:57 PM

Oh, the relevance is that PhabricatorDatasourceEngineExtension now handles "doing magic stuff when you type things into the global search bar" (like jumping to a task when you type T123), and the next change was making "type a URI" mean "navigate to that uri" -- which got a mention in T5378, although it isn't the primary focus of that task.