Parent Issue

#2359437: [meta] Remove entity_load* family of functions usage from the code base

Problem/Motivation

There are cases where entities of the entity_form_display entity type are retrieved using the functional loader, entity_load()

Proposed resolution

Replace calls to entity_load('entity_form_display', ...) with EntityFormDisplay::load*()

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

valthebald created an issue. See original summary.

danielmrichards’s picture

I'll take a look at this as part of the DrupalCon NOLA sprints

danielmrichards’s picture

danielmrichards’s picture

danielmrichards’s picture

Status: Active » Needs review
Mile23’s picture

Status: Needs review » Reviewed & tested by the community

The patch in #4 applies. My IDE says there's no occurrences of entity_load*'entity_form_display' after applying it. Also it looks like it's within scope and therefore all is well.

John Cook’s picture

Issue summary: View changes

Updated summary.

  • catch committed 0d9fa5c on 8.2.x
    Issue #2723571 by danielmrichards: Remove entity_load* usage for...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed 0d9fa5c and pushed to 8.2.x. Thanks!

Status: Fixed » Closed (fixed)

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