Ref T5655. Fixes T6849. This is another take on D11131, which was missing the DB migration and was reverted in rP7c4de0f6be77ddaea593e1f41ae27211ec179a55.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Maniphest Tasks
- T6849: With policies like "'Users A, B, C' and 'Administrators'", the 'Administrators' part is ignored
T5655: Standardize naming conventions - Commits
- Restricted Diffusion Commit
rPb3e196b69400: Rename `PhabricatorPolicyRule` subclasses for consistency
Ran ./bin/storage upgrade and verified that the classes were renamed in the phabricator_policy.policy table.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
This applied cleanly locally and looks good to me.
| resources/sql/autopatches/20150102.policyname.php | ||
|---|---|---|
| 20 | (This comment is copy/paste from elsewhere maybe?) | |