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
In #2949815: Flagging translations as outdated in a pending revision causes multiple translations to be marked as affected we have hidden the "Flag as outdated" functionality for moderated content due to how late this issue was discovered in the 8.5 development cycle, however this feature is not fundamentally incompatible with pending revisions so we should try to bring it back.
Proposed resolution
An attempt was made to save a new revision for every translation, however it was deemed too risky and not tested enough to be committed.
Remaining tasks
- Propose a valid solution
- Write a patch
- Reviews
User interface changes
The "Flag as outdated" checkbox is restored.
API changes
None
Data model changes
None
Comment | File | Size | Author |
---|---|---|---|
#14 | 2950626-14.patch | 9.19 KB | ravi.shankar |
#13 | allow-translations-outdated-flagging-with-content-moderated-2950626-13.patch | 9.19 KB | Kosa Ilma |
#11 | 2950626-11.patch | 9.17 KB | marcoscano |
#2 | ct-outdated_restore-2950626-2.patch | 9.24 KB | plach |
Comments
Comment #2
plachComment #3
timmillwoodI feel this section especially is lacking documentation. Why does a new revision need to be created to mark it as outdated?
Comment #6
crizJust adding that this patch works for us. Having this functionality back would be good to improve the editor experience when working with translations!
Comment #11
marcoscanoRe-roll for 9.2.x, which also includes some refactoring of the test, to stop using deprecated code.
Comment #13
Kosa Ilma CreditAttribution: Kosa Ilma at Brainsum for Tieto commentedI couldn't apply the patch from #11 on Drupal 9.3.3.
Here is the modified patch.
Comment #14
ravi.shankar CreditAttribution: ravi.shankar at OpenSense Labs commentedAdded reroll of patch #11.