Page MenuHomePhabricator
Differential D13911

Provide a more modern way to load packages owning a set of files
ClosedPublic
Actions

Authored by epriestley on Aug 15 2015, 7:54 PM.
Tags
None
Referenced Files
F14312979: D13911.diff
Wed, Dec 18, 3:48 AM
Unknown Object (File)
Mon, Dec 16, 12:53 AM
Unknown Object (File)
Fri, Dec 13, 11:13 PM
Unknown Object (File)
Fri, Dec 13, 3:45 PM
Unknown Object (File)
Thu, Dec 5, 10:24 AM
Unknown Object (File)
Wed, Dec 4, 7:46 PM
Unknown Object (File)
Sun, Dec 1, 4:50 AM
Unknown Object (File)
Wed, Nov 27, 1:48 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T8004: Highlight files owned by self/project when viewing revision
T8320: Bring Owners out of the dark ages
Commits
Restricted Diffusion Commit
rP36a50ccbae1e: Provide a more modern way to load packages owning a set of files
Summary

Ref T8320. Ref T8004. This just tries to generally modernize

It also replaces the nonfunctional "Find Owners" link with a new property that just shows owning packages.

Test Plan
  • Created and edited packages.

Screen Shot 2015-08-15 at 12.43.54 PM.png (278×510 px, 22 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 33575.Aug 15 2015, 7:54 PM
epriestley retitled this revision from to Provide a more modern way to load packages owning a set of files.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added tasks: T8320: Bring Owners out of the dark ages, T8004: Highlight files owned by self/project when viewing revision.
chad accepted this revision.Aug 15 2015, 8:01 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 15 2015, 8:01 PM
Closed by commit rP36a50ccbae1e: Provide a more modern way to load packages owning a set of files (authored by epriestley, committed by epriestley). · Explain WhyAug 15 2015, 8:06 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 33577

View Options

src/applications/diffusion/controller/DiffusionBrowseController.php

Loading...
View Options

src/applications/owners/controller/PhabricatorOwnersDetailController.php

Loading...
View Options

src/applications/owners/controller/PhabricatorOwnersPathsController.php

Loading...
View Options

src/applications/owners/editor/PhabricatorOwnersPackageTransactionEditor.php

Loading...
View Options

src/applications/owners/query/PhabricatorOwnersPackageQuery.php

Loading...
View Options

src/applications/owners/storage/PhabricatorOwnersPackage.php

Loading...
View Options

src/applications/owners/storage/PhabricatorOwnersPath.php

Loading...
Phabricator · Built by Phacility