The Needs Review Queue Bot β tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide β to find step-by-step guides for working with issues.
- Status changed to Postponed: needs info
10 days ago 4:51am 15 August 2025 - π¦πΊAustralia acbramley
Tested this on HEAD by adding the following to
Node::baseFieldDefinitions
and runningdrush si standard -y
, then checking thenode_field_data
table:$fields['foo__bar'] = BaseFieldDefinition::create('boolean') ->setLabel(t('Foo')) ->setRequired(TRUE) ->setDisplayConfigurable('form', FALSE) ->setDisplayConfigurable('view', FALSE);
The column existed as expected. The code mentioned in #13 no longer exists, instead it uses the DefaultTableMapping class to get the column names directly.
Can this issue be closed? Or is there another way to reproduce this?