- Issue created by @tobiasb
- Status changed to Postponed: needs info
about 1 year ago 2:30pm 13 December 2023 - π³π±Netherlands Lendude Amsterdam
- Used a clean(ish) Umami install, no contrib installed
- Added a new workflow
- Gave the authenticated user permissions for this workflow
- Added the role based permission for authenticated users to a View
- Deleted the new Workflow
- View was still there
Am I missing steps? Can you reproduce this on a clean Drupal install?
- Assigned to tobiasb
- π©πͺGermany tobiasb Berlin
Yes. I did not check this on a clean drupal just on another drupal site and with a custom role.
- Status changed to Active
about 1 year ago 3:05pm 13 December 2023 - Issue was unassigned.
Are you saying the steps to reproduce require a patch from π WorkflowDeleteForm should extend EntityDeleteForm Needs work ?
- π©πͺGermany tobiasb Berlin
No. I just mention the patch so that nobody wonders why looks differently.
- π³π±Netherlands Lendude Amsterdam
Yup, the new steps show the problem and the patch certainly helps see it! (strangely enough, it only shows the View getting deleted the second time I try to delete the workflow? Are others seeing that too?)
A little digging, \Drupal\views\Entity\View::onDependencyRemoval gets called with the editor role dependency, which is not getting removed but edited. But the View reacts to it like it's getting removed, and within the context of \Drupal\views\Entity\View::onDependencyRemoval it is impossible to determine if that dependency gets edited or removed. Since it's called 'onDependencyRemoval' I would assume it is only invoked when things get removed? But not sure.
- π³π±Netherlands Lendude Amsterdam
This feels like a configuration management issue from the first little dig I did, so moving it there for now. I don't think Workflows is doing anything wrong, seems more like Views is getting the wrong information, or Views is reaction to the information in the wrong way.
- π©πͺGermany hexabinaer Berlin, Germany
I would have expected that an affected view would at most react like it would upon a missing field: "missing or broken role".