Problem/Motivation

Remove the following functions and related deprecation tests:

  • node_type_update_nodes
  • node_revision_load
  • node_revision_delete
  • node_get_recent

It may also make sense to remove the $connection argument from \Drupal\node\Form\NodeRevisionDeleteForm::__construct in this issue so we can remove all of NodeDeprecationTest in one issue.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3432871

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:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mstrelan created an issue. See original summary.

mstrelan’s picture

Title: Remove deprecated functions from user.module » Remove deprecated functions from node.module
Component: user.module » node system
Issue summary: View changes
mstrelan’s picture

Issue summary: View changes

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

thhomas’s picture

Status: Active » Needs review

mstrelan’s picture

Status: Needs review » Reviewed & tested by the community

This looks good to me. We've removed all the deprecated functions in node.module and the deprecated constructor param in NodeRevisionDeleteForm, and the tests are all passing. Arguably this could be combined with #3432877: Remove deprecated actions from node module but not sure it's worth the effort now that it's already split.

  • catch committed 0b21da57 on 11.x
    Issue #3432871 by thhomas, mstrelan: Remove deprecated functions from...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 11.x, thanks!

Status: Fixed » Closed (fixed)

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