Tags

None

Referenced Files

	F15507143: D19583.id46814.diff
	Tue, Apr 15, 4:46 PM

	F15456327: D19583.id46837.diff
	Sun, Mar 30, 8:43 AM

	F15439228: D19583.id.diff
	Wed, Mar 26, 6:25 AM

	F15437532: D19583.diff
	Tue, Mar 25, 8:07 PM

	F15382398: D19583.diff
	Mar 14 2025, 12:09 PM

	Unknown Object (File)
	Feb 26 2025, 4:50 AM

	Unknown Object (File)
	Feb 9 2025, 6:19 AM

	Unknown Object (File)
	Feb 9 2025, 6:19 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T13164: Plans: 2018 Week 31 - 33 Bonus Content

Commits

rP24d444584548: Remove pointless requireCapabilities() method from PhabricatorRepositoryEditor

Summary

Depends on D19582. Ref T13164. It's not possible to reach the editor without passing through a CAN_EDIT check, and it shouldn't be necessarily to manually specify that edits require CAN_EDIT by default.

Test Plan

Grepped for RepositoryEditor, verified that all callsites pass through a CAN_EDIT check.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 14 2018, 8:30 PM

Harbormaster completed remote builds in B20614: Diff 46814.Aug 14 2018, 8:31 PM

epriestley requested review of this revision.Aug 14 2018, 8:31 PM

epriestley added a child revision: D19584: Move the hierarchical edit policy check in Phriction from requireCapabilities() to validateTransactions().Aug 14 2018, 9:37 PM

amckinley accepted this revision.Aug 14 2018, 10:08 PM

This revision is now accepted and ready to land.Aug 14 2018, 10:08 PM

Closed by commit rP24d444584548: Remove pointless requireCapabilities() method from PhabricatorRepositoryEditor (authored by epriestley). · Explain WhyAug 16 2018, 5:53 PM

This revision was automatically updated to reflect the committed changes.

Remove pointless requireCapabilities() method from PhabricatorRepositoryEditor
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 46837

src/applications/repository/editor/PhabricatorRepositoryEditor.php

Remove pointless requireCapabilities() method from PhabricatorRepositoryEditorClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 46837

src/applications/repository/editor/PhabricatorRepositoryEditor.php

Remove pointless requireCapabilities() method from PhabricatorRepositoryEditor
ClosedPublic
Actions

Revision Contents
Changeset List