No Code Attached Yet bug
avatar ghost
ghost
4 Apr 2019

Reminder for @wilsonge.

avatar franz-wohlkoenig franz-wohlkoenig - open - 4 Apr 2019
avatar joomla-cms-bot joomla-cms-bot - change - 4 Apr 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 4 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Apr 2019
Title
#4
[4.0] Migration path for workflows state permissions
avatar franz-wohlkoenig franz-wohlkoenig - edited - 4 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Apr 2019
Labels Added: J4 Issue
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 4 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Apr 2019
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Apr 2019
Category com_workflow
avatar franz-wohlkoenig franz-wohlkoenig - change - 7 Apr 2019
Labels Added: ?
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 7 Apr 2019
avatar wilsonge
wilsonge - comment - 9 Apr 2019

edit state and edit own now actually somewhat depend on the permissions on the workflow transitions. as a result we need to figure out how to migrate people especially those with custom article level permissions onto workflows

avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Apr 2019
Labels Added: ?
Removed: J4 Issue ?
avatar franz-wohlkoenig franz-wohlkoenig - unlabeled - 20 Apr 2019
avatar wilsonge wilsonge - change - 12 Mar 2020
Labels Added: ?
avatar wilsonge wilsonge - labeled - 12 Mar 2020
avatar brianteeman
brianteeman - comment - 27 May 2020

This is made more complex with v3 as wf can now be disabled or even partially disabled

avatar bembelimen
bembelimen - comment - 28 Nov 2020

Do we need a migration path at all now as the workflow is disabled by default?

avatar brianteeman
brianteeman - comment - 28 Nov 2020

Unless you dont want any existing site to be able to use workflows then we still need something for them. Don't we?

avatar bembelimen
bembelimen - comment - 29 Nov 2020

Depends :)
As @wilsonge said, the "conflict" is, that user with core.edit.state permissions (directly in an article) suddenly can't change the state and featured values anymore, because then a transition is needed.
But my feeling is, that due the fact that an admin has to enable the workflow intentional, probably we have to point out this fact but probably don't have to fix this "gap".

avatar brianteeman
brianteeman - comment - 29 Nov 2020

To me thats a lame solution by avoiding the problem

avatar bembelimen
bembelimen - comment - 5 Dec 2020

I don't disagree with you, it's a bit lame.
But compared with the effort to fix it (one workflow for each article and user group with that permission + categories) I would suggest to fix the "gap" with documentation.

avatar wilsonge
wilsonge - comment - 5 Dec 2020

Is there potential for some sort of cli command which would do a one off migration of edit.state permission to execute transition (for those who can run cli)? On top of docs.

avatar bembelimen
bembelimen - comment - 5 Dec 2020

Ofc there is, but then we will have (I think) in the worst case hundreds (or thousands) of similar workflows just with different execute transition permission, based on article, category and global edit.state settings...

avatar alikon
alikon - comment - 13 Mar 2021

for what i can tell you this should be degraded from Release Blocker to some kind of "nice to have"

avatar alikon
alikon - comment - 13 Mar 2021

after all workflow is disabled by default

avatar brianteeman
brianteeman - comment - 13 Mar 2021

@alikon but what happens when a user upgrades from j3 and then enables the workflows? That is where the problem lies.

avatar alikon
alikon - comment - 13 Mar 2021

well i've to admit that this scenario i've never had explored it yet myself, but i hope that who will do that have taken an informed decision
thats why imho it's docmentation matter and not a RB

avatar rdeutz rdeutz - change - 6 Apr 2021
Labels Removed: ?
avatar rdeutz rdeutz - unlabeled - 6 Apr 2021
avatar rdeutz
rdeutz - comment - 6 Apr 2021

We need to add some documentation and we are done.

avatar brianteeman
brianteeman - comment - 6 Apr 2021

Until we have the documentation its still a release blocker - nothing has changed

avatar rdeutz
rdeutz - comment - 6 Apr 2021

Until we have the documentation its still a release blocker - nothing has changed

No it wasn't a release blocker from the start.

avatar brianteeman
brianteeman - comment - 6 Apr 2021

Well that's one way to resolve blockers

avatar chmst chmst - change - 9 Feb 2022
Labels Added: No Code Attached Yet
Removed: ? ?
avatar chmst chmst - unlabeled - 9 Feb 2022
avatar Hackwar Hackwar - change - 19 Feb 2023
Labels Added: bug
avatar Hackwar Hackwar - labeled - 19 Feb 2023

Add a Comment

Login with GitHub to post a comment