Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.In #2374125: Create a persistent block_content body field storage we stopped adding body fields by default but I forgot to add the configuration to the standard profile. This issue will add the default configuraiton.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | block-body-test-2381909.w-tests.patch | 3.25 KB | larowlan |
| #5 | interdiff.txt | 737 bytes | larowlan |
| #1 | 2381909.1.patch | 2.53 KB | alexpott |
Comments
Comment #1
alexpottI thought about adding a test to the StandardTest but this feels overkill - we have the configuration - the field will exist - there is plenty of configuration in the standard profile that is not "tested".
Comment #2
larowlanLove that this is all yaml
Comment #3
Dries CreditAttribution: Dries commentedShould we have tests for this (against the Standard profile)?
Comment #4
larowlanadding a test
Comment #5
larowlanComment #6
larowlanmeh, merge lee merge
Comment #7
larowlanComment #8
alexpottWell fine - but I did argue in #1 that tests weren't necessary - or actually I'd rather add a generic test to StandardTest to ensure that all configuration in the standard profile is created as expected. but lets move on here.
Comment #9
alexpottComment #10
Dries CreditAttribution: Dries commentedCommitted the patch in #6! Thanks you. Let's open a follow-up issue for the generic test(s).