Differential D21674

Add an "Authority" control to Packages to support "Watcher" packages
ClosedPublic
Actions

Authored by epriestley on Jun 25 2021, 4:30 PM.

Tags

None

Referenced Files

	F13303229: D21674.diff
	Sat, Jun 8, 6:29 AM

	F13287233: D21674.diff
	Tue, Jun 4, 8:30 AM

	F13274506: D21674.diff
	Fri, May 31, 3:47 AM

	F13270046: D21674.diff
	Wed, May 29, 8:45 AM

	F13260554: D21674.diff
	Mon, May 27, 12:03 AM

	F13243361: D21674.id51581.diff
	Thu, May 23, 3:49 AM

	F13243086: D21674.id51580.diff
	Thu, May 23, 3:19 AM

	F13223055: D21674.diff
	May 19 2024, 4:14 AM

Subscribers

Restricted Owners Package

Details

Reviewers: None
Commits: rPa641ec82a323: Add an "Authority" control to Packages to support "Watcher" packages

Summary

See T13657. An install has "watcher" packages which should not allow owners to "Force Accept" other packages.

Test Plan

Created package A, which I own, on "/", with "Weak" authority.
Created package B, which I do not own, on "/src".
Created a revision which touches "/src" and added package B as a reviewer.
Attempted to accept the revision...
- Before patch: permitted to "Force Accept" for package B.
- After patch: not allowed to "Force Accept" for package B.
Verified that setting package "A" back to "Strong" authority allows a force-accept for package B.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 25 2021, 4:30 PM

Owners added a subscriber: Restricted Owners Package.Jun 25 2021, 4:30 PM

Harbormaster completed remote builds in B25385: Diff 51580.Jun 25 2021, 4:31 PM

epriestley requested review of this revision.Jun 25 2021, 4:31 PM

Allow "owners.search" API to read this setting.

Screen Shot 2021-06-25 at 9.59.08 AM.png (239×500 px, 27 KB)

Harbormaster completed remote builds in B25386: Diff 51581.Jun 25 2021, 5:01 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jun 25 2021, 8:48 PM

Closed by commit rPa641ec82a323: Add an "Authority" control to Packages to support "Watcher" packages (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPa641ec82a323: Add an "Authority" control to Packages to support "Watcher" packages.

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20210625.owners.01.authority.sql

2 lines

20210625.owners.02.authority-default.sql

3 lines

src/

__phutil_library_map__.php

applications/

differential/

storage/

DifferentialRevision.php

8 lines

owners/

controller/

PhabricatorOwnersDetailController.php

6 lines

editor/

PhabricatorOwnersPackageEditEngine.php

13 lines

query/

PhabricatorOwnersPackageQuery.php

13 lines

storage/

PhabricatorOwnersPackage.php

45 lines

xaction/

PhabricatorOwnersPackageAuthorityTransaction.php

56 lines

Diff 51583

resources/sql/autopatches/20210625.owners.01.authority.sql

Loading...

resources/sql/autopatches/20210625.owners.02.authority-default.sql

Loading...

src/__phutil_library_map__.php

Loading...

src/applications/differential/storage/DifferentialRevision.php

Loading...

src/applications/owners/controller/PhabricatorOwnersDetailController.php

Loading...

src/applications/owners/editor/PhabricatorOwnersPackageEditEngine.php

Loading...

src/applications/owners/query/PhabricatorOwnersPackageQuery.php

Loading...

src/applications/owners/storage/PhabricatorOwnersPackage.php

Loading...

src/applications/owners/xaction/PhabricatorOwnersPackageAuthorityTransaction.php

Loading...