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
F15467746: D13911.id33575.diff
Thu, Apr 3, 5:39 PM
F15467573: D13911.id33575.diff
Thu, Apr 3, 4:23 PM
F15466445: D13911.id.diff
Thu, Apr 3, 5:35 AM
F15465205: D13911.diff
Wed, Apr 2, 4:04 PM
F15456898: D13911.diff
Sun, Mar 30, 12:56 PM
F15392634: D13911.diff
Sat, Mar 15, 4:31 PM
Unknown Object (File)
Feb 12 2025, 6:47 AM
Unknown Object (File)
Feb 12 2025, 6:46 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