Problem/Motivation

We potentially need to use the default revision time instead of the current time in Created but then we had have to load the node. We've already done that so let's pass it along even though we're not using it yet.

Proposed resolution

Pass node along

Remaining tasks

None

User interface changes

None

API changes

Added $node to \Drupal\node_revision_delete\Plugin\NodeRevisionDeleteQueryInterface::getRevisionsToDelete and \Drupal\node_revision_delete\Plugin\NodeRevisionDeleteQueryInterface::getRevisionsToProtect

Data model changes

None

Issue fork node_revision_delete-3586447

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

alexpott created an issue. See original summary.

alexpott opened merge request !74

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs review » Reviewed & tested by the community

As this is just a simple API widening on a new interface in 2.1.x going to merge.

  • alexpott committed 77a8552e on 2.1.x 
    task: #3586447 Expand \Drupal\node_revision_delete\Plugin\...
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » 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.