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
F18652550: D13171.id31833.diff
Sun, Sep 21, 8:32 AM
F18591086: D13171.id.diff
Fri, Sep 12, 8:12 AM
F18583703: D13171.diff
Thu, Sep 11, 10:49 AM
F18505798: D13171.id.diff
Sep 5 2025, 1:00 AM
F18498278: D13171.diff
Sep 4 2025, 6:34 PM
F18081606: D13171.id31844.diff
Aug 5 2025, 1:04 AM
F18041412: D13171.id31832.diff
Aug 3 2025, 4:32 AM
F17899096: D13171.id31844.diff
Jul 29 2025, 8:31 AM
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