The Needs Review Queue Bot β tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide β to find step-by-step guides for working with issues.
- Status changed to Needs review
11 days ago 11:27pm 27 August 2024 - π³π±Netherlands seanB Netherlands
I just ran into the same issue working with translations and content moderation. Specifically when:
- The user adds a translation (from EN to NL for example).
- The user publishes both EN and NL.
- The user then removes the NL translation.
- More revisions are made for EN (these revisions no longer have a copy of the NL translation since it has been removed).
- The the user visits the translation overview, somehow it seems the NL translation still exists, so the user can't create a new translation, and is also not able to delete the NL translation.
When I tried the latest patch I still found two issues:
- The translation overview uses the latest translation affected revision. Which made it seem like the NL translation could not be removed (since the NL langcode had an older translation affected revision). We should probably also check here if the latest translation affected revision is newer than the default.
- The entity repository
EntityRepository::getActive()
also loads the latest translation affected revision. This prevented creating a new translation from the default revision.
An updated patch is attached.
- π³π±Netherlands seanB Netherlands
If the default revision still contains the translation, we can still allow older revisions to be loaded.
- Status changed to Needs work
4 days ago 12:51am 4 September 2024