Closed (fixed)
Project:
Workflows Field
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
25 Aug 2017 at 04:17 UTC
Updated:
15 Feb 2020 at 23:53 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
collin.rickford commentedHere is a patch which adds and enforces permissions on transitions, per workflow.
Comment #3
sam152 commentedNice! We'll need some tests for this though.
Comment #4
benjy commentedI've attached a new patch that adds the access control to the widget as well, still need some tests. One of the existing tests did pick up the error below.
This is wrong it should have been $field->value.
Tests still failing because they trigger the new violation. Will pick it up tomorrow.
Comment #5
sam152 commentedComment #7
benjy commentedI pulled the constraint tests into their own class since they now handle users and permissions. I've added a new test for the constraint and additional tests for getSettableOptions. Everything should be green.
Comment #8
sam152 commentedThis looks awesome!
Comment #10
sam152 commentedComment #12
CraigBertrand commentedI don't see any permissions for each state. Do I need to do something special to turn on the permissions?
Comment #13
CraigBertrand commentedNevermind, If anyone else is not seeing the permissions per state, you need to use the dev version of the module.