Page MenuHomePhabricator
Differential D13171

Support PhabricatorSpacesInterface in ApplicationSearch UI
ClosedPublic
Actions

Authored by epriestley on Jun 5 2015, 5:10 PM.
Tags
None
Referenced Files
F18844481: D13171.diff
Wed, Oct 29, 5:45 AM
F18820761: D13171.id.diff
Wed, Oct 22, 5:00 PM
F18817086: D13171.diff
Tue, Oct 21, 12:13 PM
F18652550: D13171.id31833.diff
Sep 21 2025, 8:32 AM
F18591086: D13171.id.diff
Sep 12 2025, 8:12 AM
F18583703: D13171.diff
Sep 11 2025, 10:49 AM
F18505798: D13171.id.diff
Sep 5 2025, 1:00 AM
F18498278: D13171.diff
Sep 4 2025, 6:34 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T8441: Integrate Spaces into ApplicationSearch
Commits
Restricted Diffusion Commit
rPb5dfd34e4a89: Support PhabricatorSpacesInterface in ApplicationSearch UI
Summary

Ref T8441. Does what it says, provided other conditions (like using the new SearchField stuff) are fulfilled.

Test Plan

Screen Shot 2015-06-05 at 10.09.29 AM.png (940×1 px, 153 KB)

Screen Shot 2015-06-05 at 10.09.30 AM.png (940×1 px, 158 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 31832.Jun 5 2015, 5:10 PM
epriestley retitled this revision from to Support PhabricatorSpacesInterface in ApplicationSearch UI.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T8441: Integrate Spaces into ApplicationSearch.
Herald added a subscriber: epriestley. · View Herald TranscriptJun 5 2015, 5:10 PM
epriestley updated this revision to Diff 31833.Jun 5 2015, 5:11 PM
  • Fix a warning when no spaces exist yet.
btrahan accepted this revision.Jun 5 2015, 6:19 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 5 2015, 6:19 PM
Closed by commit rPb5dfd34e4a89: Support PhabricatorSpacesInterface in ApplicationSearch UI (authored by epriestley, committed by epriestley). · Explain WhyJun 5 2015, 6:21 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 31844

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/paste/query/PhabricatorPasteSearchEngine.php

Loading...
View Options

src/applications/search/engine/PhabricatorApplicationSearchEngine.php

Loading...
View Options

src/applications/search/field/PhabricatorSearchSpacesField.php

Loading...
View Options

src/applications/spaces/phid/PhabricatorSpacesNamespacePHIDType.php

Loading...
View Options

src/applications/spaces/query/PhabricatorSpacesNamespaceQuery.php

Loading...
View Options

src/applications/spaces/typeahead/PhabricatorSpacesNamespaceDatasource.php

Loading...
Phabricator · Built by Phacility