Generate+Hide option fails when Preserve Title option selected

Open on Drupal.org →

Created on 28 March 2023, over 1 year ago

Updated 13 August 2024, 4 months ago

Problem/Motivation

If the "Automatically generate the label and hide the label field" option is used and the "Preserve already created node titles" option is also enabled, nodes are created without a title which results in the following error:

SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'title' cannot be null: INSERT INTO "test23187028node_field_data" ("nid", "vid", "type", "langcode", "status", "uid", "title", "created", "changed", "promote", "sticky", "default_langcode", "revision_translation_affected") VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2, :db_insert_placeholder_3, :db_insert_placeholder_4, :db_insert_placeholder_5, :db_insert_placeholder_6, :db_insert_placeholder_7, :db_insert_placeholder_8, :db_insert_placeholder_9, :db_insert_placeholder_10, :db_insert_placeholder_11, :db_insert_placeholder_12); Array

Steps to reproduce

* Create a content type.
* Set the Auto EntitLabel option to "Automatically generate the label and hide the label field" and fill in a pattern.
* Enable the "Preserve already created node titles" option.
* Create a node.

What should happen:
* The node is created with the pattern.

What actually happens:
* The error above is output.

Proposed resolution

Add test coverage for this scenario.
Fix the scenario.

Remaining tasks

Test coverage.
Fix the bug.

User interface changes

n/a

API changes

n/a

Data model changes

n/a

🐛 Bug report

Status

Closed: outdated

Version

3.0

Component

User interface

Created by

🇺🇸United States DamienMcKenna NH, USA

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Comments & Activities

Issue created by @DamienMcKenna
Comment over 1 year ago →
🇺🇸United States DamienMcKenna NH, USA
This is related to 🐛 %AutoEntityLabel% title when "Preserve already created node titles." option is enabled Needs work and may ultimately be the same issue, but I thought starting with test coverage would be a good approach here.
Status changed to Needs review over 1 year ago3:10pm 28 March 2023
Comment over 1 year ago →
🇺🇸United States DamienMcKenna NH, USA
Reworking the test coverage so that it doesn't send a "title" field during initial entity creation; this test issue has hidden the bug in plain sight.

This modifies the tests so that it checks both the UI (node/add/page) and the raw Node::create() API.
Comment over 1 year ago →
🇺🇸United States DamienMcKenna NH, USA
Let's try that again.
Comment over 1 year ago →
🇺🇸United States DamienMcKenna NH, USA
*facepalm*
Comment over 1 year ago →
🇺🇸United States DamienMcKenna NH, USA
Once more.
Comment 7 months ago →
🇩🇪Germany Anybody Porta Westfalica
@DamienMcKenna thank you for your great work here! Just to clear things up and eventually update the issue summary: Did this issue appear with a patch from 🐛 %AutoEntityLabel% title when "Preserve already created node titles." option is enabled Needs work applied? Or without any patches?
(It looks like this issue was caused by a patch from there?)

I think both issues should be solved together to ensure the option works as expected afterwards.
Comment 4 months ago →
🇩🇪Germany Anybody Porta Westfalica
I guess this might be fixed now with 🐛 Set Label runs two times on node creation Needs work
@DamienMcKenna could you check that?
Comment 4 months ago →
🇺🇸United States DamienMcKenna NH, USA
I'm afraid I don't have access to the site where we were running into this problem. Feel free to close this.
Status changed to Closed: outdated 4 months ago8:41am 13 August 2024
Comment 4 months ago →
🇩🇪Germany Anybody Porta Westfalica
Presumably fixed by 🐛 Set Label runs two times on node creation Needs work

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024