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.
Problem/Motivation
The body field on the base entity table has a serialised array of value & format stored in a string long field. It has been pointed out that a text_long would be the appropriate type for this (credit @Berdir - thank you!). Without this change it can cause issues for those using the rest API for instance.
Proposed resolution
Create new base storage definition and migrate content from the old to the new then discard the old.
Remaining tasks
Test.
User interface changes
Minor label tweaks.
API changes
Anyone using the 'body' value directly modifying the module will need to use 'message' instead. No methods or functions are changing.
Data model changes
New field 'message' to replace 'body.
Comment | File | Size | Author |
---|---|---|---|
#16 | convert_body_to_use-2886020-16.patch | 410.66 KB | scott_euser |
#10 | convert_body_to_use-2886020-10.patch | 442.32 KB | scott_euser |
| |||
#8 | convert_body_to_use-2886020-8.patch | 333.56 KB | scott_euser |
#6 | convert_body_to_use-2886020-6.patch | 0 bytes | scott_euser |
#4 | 2886020-2-4.txt | 0 bytes | scott_euser |
Comments
Comment #2
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedFirst attempt. Needs update tests still, but setting to needs review to trigger tests.
Comment #3
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedComment #4
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedRefreshed previous update test.
Comment #6
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedAnd with binary change in the patch....
Comment #8
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedAttempting again to add binary to patch.
Comment #10
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedComment #12
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedDoh, database dump is drupal core 8.3, will attempt again
Comment #13
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedFailure to update to 8.4 appears to be a result of https://www.drupal.org/node/2874827
Comment #14
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedCan update the update tests to use 8.4x until the following are solved:
https://www.drupal.org/node/2882456
https://www.drupal.org/node/2874827
And the tests run against 8.4x
Comment #15
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedGiven up for now, testing against d8 stable instead.
Comment #16
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedComment #17
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commentedComment #18
scott_euser CreditAttribution: scott_euser as a volunteer and at Fat Beehive commented