- 🇮🇳India sanket1007
Couldn't apply the patch #14 in latest Drupal version 10.3.1
- 🇷🇴Romania Kosa Ilma
I can confirm @sanket1007
I have tried on Drupal core 10.3.1 and it is working without the addition of the patch. - First commit to issue fork.
- 🇺🇸United States charginghawk
Unable to test 10.3 right now, but it seems like the outdated option should still be unavailable since the logic hiding it is still there:
https://git.drupalcode.org/project/drupal/-/blob/11.0.4/core/modules/con...
In any case that code should be removed. Also, we should have steps to reproduce.
Additionally, as a commenter noted, "Why does a new revision need to be created to mark it as outdated?" It doesn't. The condition at the end of markRevisionTranslationsOutdated() should use setSyncing(TRUE) to mark the latest translation affected revision as outdated:
$revision_id = $storage->getLatestTranslationAffectedRevisionId($entity_id, $langcode); if (!$revision_id) { continue; } /** @var \Drupal\Core\Entity\ContentEntityInterface $revision */ $revision = $storage->loadRevision($revision_id); $revision_translation = $revision->getTranslation($langcode); $metadata = $this->manager->getTranslationMetadata($revision_translation); if (!$metadata->isOutdated()) { $metadata->setOutdated(TRUE); $revision_translation->setSyncing(TRUE); $storage->save($revision_translation); }
- Status changed to Needs review
7 months ago 8:40pm 3 December 2024 - 🇺🇸United States charginghawk
Rerolling #14 for 10.3.x. Also adding the "setSyncing(TRUE)" described above to setOutdated on the current revision instead of creating a new one.
- 🇺🇸United States smustgrave
Thanks for continuing. Patch should be in an MR though vs patch.
- Status changed to Needs work
about 1 month ago 9:41pm 27 May 2025 - 🇺🇸United States banoodle San Francisco, CA
Pach #14 doesn't apply on 11.1.5.
I re-rolled the patch locally so it applies, but then I get timeout errors when I save nodes with "translations outdated" enabled.
I would submit a MR, but for some reason I don't have sufficient privileges (I'm working to resolve that now).
- 🇺🇸United States banoodle San Francisco, CA
Here is a re-rolled patch that works on D11.1.5.
I realize you prefer an MR. I will try to make one tomorrow once I sort out my permissions/account issues.
- 🇺🇸United States banoodle San Francisco, CA
Update: even though patch 25 works fine in my local, when I deploy it to Pantheon, I still get "The application did not respond in time." when trying to save a node with the "outdated" option enabled.