Problem/Motivation

Spin-off from #3477043: Change automatic body field creation to use formatted text field instead of text_with_summary to see if we can do the absolute minimum to unblock Drupal CMS. That issue will still be relevant.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3488735

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

catch created an issue. See original summary.

catch opened merge request !10273

catch’s picture

catch
he/him
Primary language English
commented
Status: Active » Needs review
catch’s picture

catch
he/him
Primary language English
commented
Title: Make the body field optional and deletable » Make the body field optional

Already deleteable via the API, just not via the UI.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

I don't think that changes much in practice so far, if it doesn't have dependencies that might not be fullfilled, still going to be always installed?

If it's optional, we need to change node_add_body_field() to return early if the field doesn't exist and document that it might return null, which is a bit of an API change?

dmitry.korhov’s picture

dmitry.korhov
Primary language English
Location Warsaw, Poland
commented

If that's for Drupal CMS and not for every installation, why not to use separate `node/config/install` sub-folder that will install the relevant config?

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Closed (duplicate)

The idea was to allow recipes to install node module with a text_long body field, but recipes can already do that since they skip installing module config. So yeah this doesn't do anything, we need #3488742: Stop calling node_add_body_field() from NodeTypeForm instead. Closing as duplicate of the other two isseus.