Bug in the validation constraint "No transition exists to move from [EMPTY] to [STATE]."

Created on 6 May 2021, about 3 years ago

Updated 15 November 2023, 7 months ago

Problem/Motivation

I added the workflow to an existing content type with content created for it. Now it's impossible to save the node because there is no state transition from an empty value to a state.

Steps to reproduce

Create a content type and add some content.
Create a new workflow with some states
Add a new workflow field to the content type
Edit an existing node and try to select a state and save
You will get an error like "No transition exists to move from [EMPTY] to [STATE]."

Proposed resolution

The easiest way would be to update the constraint to skip empty field values.

Remaining tasks

Update the code

User interface changes

N/A

API changes

N/A

Data model changes

N/A

🐛 Bug report

Status

Needs work

Version

2.0

Component

Code

Created by

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Live updates comments and jobs are added and updated live.

Needs tests
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Incomplete comments

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment 9 months ago →
🇨🇳China Lugir
RTBC +1
Status changed to Needs work 7 months ago8:36am 15 November 2023
Comment 7 months ago →
🇦🇺Australia dpi Perth, Australia
Needs reroll.

Needs tests, this should be straightforward.

I'd prefer a NULL check rather than empty(). empty is always a code smell.

Please only use MR's going forward.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
last update 7 months ago
Composer require failure
Comment 7 months ago →
🇧🇪Belgium BramDriesen Belgium 🇧🇪
Fixed your remark and rebased the issue fork. Still needs tests.

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024