Page MenuHomePhabricator
Differential D20509

Remove ancient "PhabricatorQuickSearchEngineExtension" compatibility class
ClosedPublic
Actions

Authored by epriestley on May 9 2019, 6:14 PM.
Tags
None
Referenced Files
F18849797: D20509.id.diff
Thu, Oct 30, 12:19 PM
F18846825: D20509.diff
Wed, Oct 29, 9:18 PM
F18821231: D20509.diff
Wed, Oct 22, 8:44 PM
F18808792: D20509.id48902.diff
Sun, Oct 19, 8:38 AM
F18722911: D20509.id.diff
Sep 29 2025, 11:36 PM
F18716979: D20509.id.diff
Sep 29 2025, 12:59 PM
F18707921: D20509.diff
Sep 28 2025, 8:32 PM
F18679729: D20509.diff
Sep 26 2025, 1:27 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T5378: Phabricator full task URLs for that instance do not provide hover/popup text (like T1234 does)
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.May 9 2019, 6:14 PM
Harbormaster completed remote builds in B22775: Diff 48902.May 9 2019, 6:15 PM
epriestley requested review of this revision.May 9 2019, 6:15 PM
epriestley edited the summary of this revision. (Show Details)May 9 2019, 6:21 PM
epriestley added a task: T5378: Phabricator full task URLs for that instance do not provide hover/popup text (like T1234 does).
epriestley added a child revision: D20510: When a user pastes a Phabricator URI into the search box, redirect to the URI.May 9 2019, 6:23 PM
amckinley accepted this revision.May 16 2019, 12:57 PM

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
epriestley added a comment.May 16 2019, 4:40 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.

Closed by commit rPf2feb0378f54: Remove ancient "PhabricatorQuickSearchEngineExtension" compatibility class (authored by epriestley). · Explain WhyMay 16 2019, 6:14 PM
This revision was automatically updated to reflect the committed changes.
epriestley added an edge: rPf2feb0378f54: Remove ancient "PhabricatorQuickSearchEngineExtension" compatibility class.

Revision Contents
Changeset List

PathSize
src/
applications/
search/
engineextension/

Diff 48938

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/search/engineextension/PhabricatorQuickSearchEngineExtension.php

Loading...
Phabricator · Built by Phacility