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
When we update paragraphs with old data we get a fatal error that indicates that there are references to paragraphs (or other content) that was deleted.
Proposed resolution
Fix it.
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#4 | fatal_error_when_call-2706883-4.patch | 1.45 KB | johnchque |
#4 | interdiff-2706883-2-4.txt | 779 bytes | johnchque |
#4 | fatal_error_when_call-2706883-4-test-only.patch | 779 bytes | johnchque |
#2 | fatal_error_when_call-2706883-2.patch | 706 bytes | johnchque |
|
Comments
Comment #2
johnchqueThis should fix it the ERR part.
Comment #3
BerdirTest coverage for this should be pretty easy.
In an existing test with an entity that references another, delete the referenced entity and then resave the parent, that should be enough to trigger this.
Comment #4
johnchqueYes, the problem is triggered with steps on comment #3. Added tests.
Comment #6
BerdirComment in the test is no very clear, but Miro can improve that on commit I think if he has an idea for improving it?
Comment #7
chr.fritschUsing devel_generate for creating nodes with paragraphs is still borken.
Got this message:
Comment #8
BerdirThat's not the fault of this issue, if anything, we just change the error because it fails differently. ERR needs specific integration for devel_generate.
Comment #9
killua99 CreditAttribution: killua99 commentedCould we add test coverage for PostgreSQL?
I open the bug about update paragraphs 8003 and we use PostgreSQL
Comment #10
BerdirI added a test with PostgreSQL for the patch. This has nothing to do with the database. If it fails then HEAD very likely already fails too.
@Miro: Can you enable on commit testing for PHP7 + mysql/postgresql/sqlite on https://www.drupal.org/node/2411903/qa? just php7 for those should be enough and it's faster.
Comment #12
miro_dietikerThx, fixed.
Comment #13
miro_dietikerAdd missing reference.