Page MenuHomePhabricator
Differential D19582

Remove pointless requireCapabilities() method from PhabricatorProjectColumnTransactionEditor
ClosedPublic
Actions

Authored by epriestley on Aug 14 2018, 8:22 PM.
Tags
None
Referenced Files
F15579237: D19582.id46836.diff
Tue, May 6, 10:46 PM
F15569508: D19582.diff
Sun, May 4, 1:08 AM
F15550951: D19582.id.diff
Sun, Apr 27, 2:42 PM
F15544601: D19582.diff
Sat, Apr 26, 6:16 AM
F15537121: D19582.id46813.diff
Thu, Apr 24, 12:19 PM
F15505390: D19582.id46813.diff
Tue, Apr 15, 2:52 AM
F15441072: D19582.id.diff
Mar 26 2025, 4:14 PM
F15435243: D19582.diff
Mar 25 2025, 7:29 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13164: Plans: 2018 Week 31 - 33 Bonus Content
Commits
rPa39852ae1b4a: Remove pointless requireCapabilities() method from…
Summary

Depends on D19581. Ref T13164. This method has no effect:

  • You must always have CAN_EDIT to reach an Editor in the first place.
  • Per previous change, I'm going to restructure this so transactions explicitly check CAN_EDIT by default anyway.
Test Plan

Tried to edit and hide a project column as a user without permission, hit global permission checks long before reaching this method.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
src/
applications/
project/
editor/
18 lines

Diff 46836

View Options

src/applications/project/editor/PhabricatorProjectColumnTransactionEditor.php

Loading...
Phabricator · Built by Phacility