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
F19030925: D13911.id33577.diff
Mon, Nov 24, 10:38 PM
F18965425: D13911.diff
Fri, Nov 14, 4:53 PM
F18941748: D13911.diff
Tue, Nov 11, 11:04 AM
F18878891: D13911.id.diff
Thu, Nov 6, 5:57 PM
F18862156: D13911.id33575.diff
Sun, Nov 2, 6:15 PM
F18855039: D13911.id.diff
Nov 1 2025, 2:24 AM
F18826254: D13911.id33577.diff
Oct 24 2025, 3:12 AM
F18823447: D13911.diff
Oct 23 2025, 11:59 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