- Issue created by @smustgrave
- ๐บ๐ธUnited States phenaproxima Massachusetts
Alright, started an MR. Let's see what breaks.
- Merge request !10275Remove the call to node_add_body_field() from NodeTypeForm โ (Open) created by phenaproxima
- ๐ฌ๐งUnited Kingdom catch
Discussed this briefly with @Gรกbor Hojtsy in slack and he approved this change from a product manager perspective. It changes what happens with the form, but since the field is already deletable it doesn't affect other expectations outside the UI.
- ๐บ๐ธUnited States phenaproxima Massachusetts
OK, so all PHP tests are now passing but Nightwatch tests are failing because they need to manually add the body field after creating a new content type. I have no Nightwatch experience.
- ๐ฌ๐งUnited Kingdom catch
Tried clicking the button in a different way etc. but did not really get anywhere.
- ๐ญ๐บHungary Gรกbor Hojtsy Hungary
I think this makes sense, it fits into the philosophy we are taking Drupal core towards, being more of an enabler that other layers can rely on, not trying to be a complete solution.
- ๐ฌ๐งUnited Kingdom alexpott ๐ช๐บ๐
Can we get a follow-up to fix the nightwatch tests. I will do that but after we land this.
We also need a change record so we can highlight the behaviour change in the 11.1 release. Drupal has added a body field for you for a very long time.
- ๐บ๐ธUnited States phenaproxima Massachusetts
Well that certainly is a workaround. :) I feel okay RTBCing this because it's different than the approach I took. Effectively a different patch.
This is important for the future of Drupal CMS so, knowing that we have follow-up issues in place to address this properly and to deprecate/remove this accursed function, and that if we don't do this now, we won't get another chance until 11.2...
...let it fly.
- ๐ฌ๐งUnited Kingdom catch
Here's the follow-up ๐ Change automatic body field creation to use formatted text field instead of text_with_summary Active .
I can draft the CR now too.
- ๐ฌ๐งUnited Kingdom alexpott ๐ช๐บ๐
Committed and pushed dd2982b5324 to 11.x and c4e3cfcfe05 to 11.1.x. Thanks!
-
alexpott โ
committed c4e3cfcf on 11.1.x
Issue #3488742 by catch, phenaproxima, gรกbor hojtsy: Stop calling...
-
alexpott โ
committed c4e3cfcf on 11.1.x
-
alexpott โ
committed dd2982b5 on 11.x
Issue #3488742 by catch, phenaproxima, gรกbor hojtsy: Stop calling...
-
alexpott โ
committed dd2982b5 on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.