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
DER 8.x-2.x has a different column for JOINs than the column used for storage.
Proposed resolution
Factor out the relationship entity table join into a method so DER can override it in #2808337: Make use of proposed entity table relationship refactor
Remaining tasks
Commit. There's no need for a test, the existing tests suffice as there's no change whatsoever.
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#3 | 2808333_3.patch | 2.27 KB | chx |
Comments
Comment #2
chx CreditAttribution: chx commentedComment #3
chx CreditAttribution: chx commentedComment #4
chx CreditAttribution: chx commentedComment #5
jibranThe patch just moves two line of code to a function which should be fine imo. I talked about this with @berdir and he is also fine by it.
Comment #6
jibranComment #8
jibranRetesting it.
Comment #9
alexpottSeems a sane refactoring to help DER.