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)
Sat, May 10, 2:09 PM
Unknown Object (File)
Mon, May 5, 4:24 PM
Unknown Object (File)
Mon, May 5, 8:58 AM
Unknown Object (File)
Wed, Apr 30, 11:10 AM
Unknown Object (File)
Tue, Apr 29, 12:23 AM
Unknown Object (File)
Apr 23 2025, 12:13 PM
Unknown Object (File)
Apr 22 2025, 4:26 PM
Unknown Object (File)
Apr 18 2025, 5:02 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
Branch
toc4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7647
Build 8332: [Placeholder Plan] Wait for 30 Seconds
Build 8331: arc lint + arc unit

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 33575

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