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
If you have an entity type that implements EntityOwnerInterface
but does not use uid
for the owner field, then content translation will add a content_translation_uid
field instead of using the existing owner field.
Proposed resolution
- Remove the
$this->checkFieldStorageDefinitionTranslatability('uid')
check inContentTranslationHandler::hasAuthor()
- Use
EntityOwnerInterface::setOwner()
inContentTranslationMetadataWrapper::setAuthor()
if there is nocontent_translation_uid
field
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#8 | Content_Translation-2850400-8.patch | 1.6 KB | Mayuresh7 |
Comments
Comment #2
tstoecklerComment #3
tstoecklerComment #4
shashikant_chauhan CreditAttribution: shashikant_chauhan as a volunteer and at Iksula commentedComment #5
Mayuresh7 CreditAttribution: Mayuresh7 commentedi am working on it
Comment #6
Mayuresh7 CreditAttribution: Mayuresh7 commentedComment #7
Mayuresh7 CreditAttribution: Mayuresh7 at Iksula commentedComment #8
Mayuresh7 CreditAttribution: Mayuresh7 at Iksula commentedupdated patch
Comment #20
smustgrave CreditAttribution: smustgrave at Mobomo commentedSo I believe this still applies in D9.5
So question what if we just did?
Would that solve the issue? Believe it generates less test breaks too.
Comment #21
ravi.shankar CreditAttribution: ravi.shankar at OpenSense Labs commentedAdded a patch as per comment #20, let's see which tests break.
Comment #24
SocialNicheGuru CreditAttribution: SocialNicheGuru commented#21 missed a piece of #8.
#8 is the one to use