Workbench Moderation ships with self-to-self transitions, like draft-to-draft. Users should be able to create this type of transition in the UI, but right now this generates a form error:

CommentFileSizeAuthor
#6 2651190-create-self-transition.patch3.23 KBCrell
#6 interdiff-2651190.txt2.36 KBCrell
#2 2651190-create-self-transition.patch887 bytesCrell
Screen Shot 2016-01-15 at 4.00.43 PM.png47.8 KBbecw
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

becw created an issue. See original summary.

Crell’s picture

Crell CreditAttribution: Crell at Palantir.net for Acquia commented
Status: Active » Needs review
FileSize
2651190-create-self-transition.patch887 bytes

Easy enough.

Status: Needs review » Needs work

The last submitted patch, 2: 2651190-create-self-transition.patch, failed testing.

The last submitted patch, 2: 2651190-create-self-transition.patch, failed testing.

The last submitted patch, 2: 2651190-create-self-transition.patch, failed testing.

Crell’s picture

Crell CreditAttribution: Crell at Palantir.net for Acquia commented
Status: Needs work » Needs review
FileSize
interdiff-2651190.txt2.36 KB
2651190-create-self-transition.patch3.23 KB

Ha, yeah, there's even a test to ensure the behavior we don't want. :-)

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented

This is an odd idea which I haven't seen in any other solution. There's no transition here, you're staying in the same place.

What's the backstory?

Crell’s picture

Crell CreditAttribution: Crell at Palantir.net for Acquia commented

Without this transition, it is impossible to save an entity in the same state it already is. Making it a separate transition means you can control when someone can edit an entity in the same state or not (eg, can you edit a published node and leave it published? If you configure the permissions to let you.)

That question has come up a couple times now, so we need to better document it. Suggestions welcome on how to do that, although that's separate from this issue which is just fixing an admin bug.

becw’s picture

becw CreditAttribution: becw at Palantir.net for Acquia commented
Status: Needs review » Fixed

Great, thanks Crell!

  • becw committed e3c01c5 on 8.x-1.x 
    Issue #2651190 by Crell: Need to be able to create transitions from a...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.