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
Unknown Object (File)
Wed, Feb 12, 6:47 AM
Unknown Object (File)
Wed, Feb 12, 6:46 AM
Unknown Object (File)
Sat, Feb 8, 10:05 PM
Unknown Object (File)
Sat, Feb 8, 10:05 PM
Unknown Object (File)
Sat, Feb 8, 5:33 AM
Unknown Object (File)
Tue, Jan 28, 4:28 AM
Unknown Object (File)
Mon, Jan 27, 1:56 PM
Unknown Object (File)
Sat, Jan 25, 1:32 PM
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