AI Translate Module Leaves Fields Empty When No Translation is Found

Patch #3 does not work for me, at least for image fields. Translation leaves image fields empty.

As far as I can tell, this isn't related to this module: it only translates text strings (which would make sense) and does not do anything to the images at all.

The call to $entity->addTranslation($lang) in AiTranslateController returns a translation with the image attached in the English lanmguage. Saving it creates a new translation within that field that does not copy the original image over. We don't have anything in the module that affects the saving opf the entity or reacts to it, so I think it's Core's translation interfaces that are not creating the image. It's not an area I know a lot about though, so possibly someone else can see something I've missed.

with Merge request !395, if no translation found for the entity reference, we just copy the reference value on the translated entity.
This way, image and all other reference from a source language will be ported on the translation.
This feel more natural for editors since the all page looks the same instead of just having only the texts translated.

We have the same issue with the entity references.

Example, if a node has an entity reference field that references a paragraph, and that paragraph references a media, the media will not be copied to the translation.

I Updated my MR ( 395) to integrate the fact that an entity reference can reference an other entity and it has to be translated ortakethe value from the default language as well.

So i created a recursive to go through all the references.
It translate any text that has been found by the extractor and copy any reference from the main language if no translation found.

It seem in certain cases, the media are not copied correctly, still need a bit of work to have a good solution.
Also not a big fan of how the undefinded languages are an exception.
if ($referencedEntity->language()->getId() === LanguageInterface::LANGCODE_NOT_SPECIFIED) {
This is most probably the remaining issue i got.
I'll need a bit more time to work on this.

Hello guys, thanks for your work!

In the extractor class, the old code used to check whether the original translation field is empty, but in the current branch, it checks on the translated entity, which would have that field empty, because its not translated.
Please double check the logic against the attached interdiff which solves the problem for me by checking vs the untranslated entity.

Best, Alexei.

Please ignore the prev. interdiff. Further tests revealed its faulty.

Can this be reapplied to 1.1.x please?

There are a lot of out of scope/not needed changes in MR https://git.drupalcode.org/project/ai/-/merge_requests/395, more seems like rebase issue etc.

MR: https://git.drupalcode.org/project/ai/-/merge_requests/248 has only one file change.

I would suggest to create a fresh new branch and cherry-pick changes from from 395 and 248.

Listing out the commits which needs to be picked:
https://git.drupalcode.org/project/ai/-/merge_requests/248/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...
https://git.drupalcode.org/project/ai/-/merge_requests/395/diffs?commit_...

Above commits could have overlapping changes as well.

prabha1997 → changed the visibility of the branch 3482645-copy-entity-reference to hidden.

prabha1997 → changed the visibility of the branch 3482645-copy-entity-reference to active.