Follow-up to #2322639: Replace all instances of node_type_load(), node_type_get_types(), entity_load('node_type') and entity_load_multiple('node_type') with static method calls
Meta issue: #2205673: [META] Remove all @deprecated functions marked "remove before 8.0"

Problem/Motivation

These functions are deprecated:

  • node_type_get_types()
  • node_type_load()
  • node_load()
  • node_load_multiple()

Proposed resolution

Remove them.

Remaining tasks

N/A

User interface changes

N/A

API changes

node_type_get_types() node_type_load() node_load() node_load_multiple() are finally gone.

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Task because removing deprecated functions.
Issue priority Normal, No other usage of these functions.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mac_Weber’s picture

Component: file system » node system
Status: Postponed » Needs review
FileSize
3.64 KB
cilefen’s picture

Status: Needs review » Needs work

Those functions are"remove before 9.0", not "remove before 8.0". Are you sure that is the correct meta issue?

Mac_Weber’s picture

@cilefen I see the code comment list them as to be removed before 9.0, but the meta issue linked list them as to be removed before 8.0 and their uses are already removed. To be honest I'm not sure if they should be removed before 8.0 or 9.0

cilefen’s picture

I am a little confused by the wording, too.

The policy: https://www.drupal.org/node/1354#deprecated
Some background: https://www.drupal.org/node/1876842

cilefen’s picture

Also consider the beta policy. If you do a careful beta evaluation, which this issue needs, you may determine the answer.

Once that is settled, this would need a draft change record or a change record update (the update could be to the change record that deprecated them).

chananapeeyush’s picture

Issue summary: View changes

Adding beta evaluation.Will draft the CR
Edit : This issue removes four functions so ideally this needs two CRs/Single one/ Needs updation in Old CRs ?

  1. node_load(), node_load_multiple()
  2. , node_type_get_types() and node_type_load()
JeroenT’s picture

Version: 8.0.x-dev » 9.x-dev
Status: Needs work » Postponed
Related issues: +#2474151: Mark procedural wrappers in entity.inc as deprecated

Those functions are deprecated for version 9.0 (https://www.drupal.org/node/2321969#comment-9867723).

catch’s picture

Version: 9.x-dev » 9.0.x-dev

The 9.0.x branch will open for development soon, and the placeholder 9.x branch should no longer be used. Only issues that require a new major version should be filed against 9.0.x (for example, removing deprecated code or updating dependency major versions). New developments and disruptive changes that are allowed in a minor version should be filed against 8.9.x, and significant new features will be moved to 9.1.x at committer discretion. For more information see the Allowed changes during the Drupal 8 and 9 release cycles and the Drupal 9.0.0 release plan.