I'm trying to unpublish content via Rules during a cron run. When I run cron and the action fires I get messages like: "You do not have permission to moderate Foo".

I'm using the "Set moderation state" action (workbench_moderation_set_state_rules_action()). Because cron runs as user 0, I need to use the "Force transition" checkbox to bypass the node access checks.

In commit 808b1a23afbd47f58aee5d5482b357149acd0531 from issue #1787214: VBO action to change state has a blank checkbox and sets the state back to the original state after it is changed the bypass for $context['force_transition'] was ripped out.

CommentFileSizeAuthor
#12 force_transition-2476489-12.patch728 bytesanrikun
#7 workbench_moderation-reimplement-force_transaction-247648907.patch719 bytesbenstallings

Comments

gabyyy’s picture

gabyyy commented

Same problem here with Rules 2.9 and Workbench moderation 1.4
Looks like the update to 1.4 broke this feature.

joel_osc’s picture

joel_osc commented

Ran into this too, did a patch -R on the patch mentioned above fixed the issue.

gabyyy’s picture

gabyyy commented

joel_osc, which patch are you referring to?

joel_osc’s picture

joel_osc commented

In the issue description: #1787214: VBO action to change state has a blank checkbox and sets the state back to the original state after it is changed - remove this patch and force transition works again.

gabyyy’s picture

gabyyy commented

Any chance of fixing this issue without having to unpatch the 1.4 version?

codevoice’s picture

codevoice commented

+1 same problem here.

benstallings’s picture

benstallings commented
Status: Active » Needs review
StatusFileSize
new workbench_moderation-reimplement-force_transaction-247648907.patch719 bytes

Here is a patch to re-implement force_transition. I'm not very experienced with patches, so I hope it's in the right format!

Status: Needs review » Needs work

The last submitted patch, 7: workbench_moderation-reimplement-force_transaction-247648907.patch, failed testing.

alexverb’s picture

alexverb commented
Status: Needs work » Needs review

The patch looks good to me. I don't understand why it's failing. Let's try one more time.

jlashomb’s picture

jlashomb commented

This patch worked for me.

guillaumeduveau’s picture

guillaumeduveau
Location Toulouse
commented

Also interested

anrikun’s picture

anrikun commented
Version: 7.x-1.4 » 7.x-3.x-dev
StatusFileSize
new force_transition-2476489-12.patch728 bytes

Let's try this one.

Status: Needs review » Needs work

The last submitted patch, 12: force_transition-2476489-12.patch, failed testing.

tessa bakker’s picture

tessa bakker
she/they
Primary language Dutch
commented
Status: Needs work » Postponed
Related issues: +#2832600: CI error on tests, because of old test module

Patch in #2832600: CI error on tests, because of old test module needs to be committed first, until then, every test will fail.

tessa bakker’s picture

tessa bakker
she/they
Primary language Dutch
commented
Status: Postponed » Needs review
sgdev’s picture

sgdev commented

@anrikun, thanks for adding the updated patch. Looks like a simple fix... we'll test and if it works as expected will return and mark as RTBC.

benstallings’s picture

benstallings commented
Status: Needs review » Closed (outdated)

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.