I just noticed that the exception thrown in WorkflowTypeBase has a strange quote added to its end:

throw new \InvalidArgumentException("The state '$state_id' does not exist in workflow.'");

Seems like an artifact from changing from single quotes to double quotes.

Patch coming up

CommentFileSizeAuthor
#3 2959124.patch3.68 KBborisson_
#3 interdiff-2959124.txt3.27 KBborisson_
#2 2959124.patch699 byteseiriksm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eiriksm created an issue. See original summary.

eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: eiriksm at Ny Media AS, Violinist commented
Status: Active » Needs review
FileSize
2959124.patch699 bytes
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer commented
FileSize
interdiff-2959124.txt3.27 KB
2959124.patch3.68 KB

Yep, that single quote doesn't belong there. However, that same single quote is also in other exceptions thrown from methods in that file, I searched for '" in that file, and fixed all of them at once.

Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Status: Needs review » Reviewed & tested by the community

Hah, good catch.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Reviewed & tested by the community » Fixed

Good find, committed. Thanks!

  • Gábor Hojtsy committed 5a13665 on 8.6.x 
    Issue #2959124 by borisson_, eiriksm: Unexpected quote in workflow...

Status: Fixed » Closed (fixed)

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