Problem/Motivation

Steps to reproduce

1. Create node in en language (revision translation affected for en = 1);
2. Create translation in de language with changing untranslated field (revision translation affected = 1 for both en and de).
3. Create more revisions for en to have 2nd revision more than configure amount of revisions.
4. Run node revision delete queue.

Result:
de translation is lost, because 2nd revision deleted by amount plugin.

CommentFileSizeAuthor
#3 3543480-3.patch2.38 KBkiseleva.t

Issue fork node_revision_delete-3543480

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

kiseleva.t created an issue. See original summary.

kiseleva.t opened merge request !42

kiseleva.t’s picture

kiseleva.t commented
Status: Needs work » Needs review
StatusFileSize
new 3543480-3.patch2.38 KB
er.garg.karan’s picture

er.garg.karan
Primary language English
Location Chandigarh
commented
Version: 2.0.x-dev » 2.0.2

Facing this issue D11.
Please confirm if the patch can be ported into the actual module.

alexpott made their first commit to this issue’s fork.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Priority: Normal » Critical

This is a critical bug - this module must not cause data loss. I've added a test for the situation that fails without the fix on this issue. Once I've got this green I'll merge and make a release.

  • alexpott committed 81c876d5 on 2.0.x 
    fix: #3543480 Revision translation affected for more than one language...

  • alexpott committed 20bd3064 on 2.1.x 
    fix: #3543480 Revision translation affected for more than one language...
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.