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

CommentFileSizeAuthor
#3 2723521.patch3.97 KBvalthebald
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

valthebald created an issue. See original summary.

valthebald’s picture

valthebald
Location Sofia
CreditAttribution: valthebald as a volunteer commented
Title: Remove entity_load* usage from action module » Remove entity_load* usage for action entity type
valthebald’s picture

valthebald
Location Sofia
CreditAttribution: valthebald as a volunteer commented
Status: Active » Needs review
FileSize
2723521.patch3.97 KB

Let's see how this goes

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Still totally not a fan of EntityType::load() but that's not deprecated and entity_load() is. :-)

The patch does what it says on the tin: IDE couldn't find any occurrence of entity_load*'action'

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed f0e5004 and pushed to 8.2.x. Thanks!

Checked all instances of entity_load.*action are removed from core.

  • alexpott committed f0e5004 on 8.2.x 
    Issue #2723521 by valthebald: Remove entity_load* usage for action...

Status: Fixed » Closed (fixed)

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