Problem/Motivation

Method getPreviousRevisions() is returning results of old revisions AND current revisions.
This should not include the current revisions.

Proposed resolution

Update filter to exclude current revision.

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

None.

CommentFileSizeAuthor
#2 node_revision_delete-exclude_current_revision-2-8.x.patch677 bytesrobert ngo

Comments

Robert Ngo created an issue. See original summary.

robert ngo’s picture

robert ngo
Location Montreal ⚜️
commented
StatusFileSize
new node_revision_delete-exclude_current_revision-2-8.x.patch677 bytes

Update filter from: 

if ($currently_deleted_revision_id - $vid >= 0) { ... }

to 

if ($currently_deleted_revision_id - $vid > 0) { ... }
robert ngo’s picture

robert ngo
Location Montreal ⚜️
commented
Status: Active » Needs review
adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Status: Needs review » Fixed

  • adriancid committed aac7ed1 on 8.x-1.x authored by Robert Ngo 
    Issue #3074570 by Robert Ngo, adriancid: getPreviousRevisions is...

  • adriancid committed 4392b23 on 7.x-3.x 
    Issue #3074570 by Robert Ngo, adriancid:...

Status: Fixed » Closed (fixed)

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