Parent Issue

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

Problem/Motivation

There are case where entities of the rdf_mapping entity type are retrieved using the functional loader, entity_load*()

Proposed resolution

Replace calls to entity_load*('rdf_mapping', ...) with RdfMapping::load()

CommentFileSizeAuthor
#2 remove_entity_load-2723613-2.patch567 bytesJohn Cook
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

valthebald created an issue. See original summary.

John Cook’s picture

John Cook CreditAttribution: John Cook commented
Status: Active » Needs review
FileSize
remove_entity_load-2723613-2.patch567 bytes

I've created a patch to remove the usage of entity_load('rdf_mapping', ...) that I can find.

John Cook’s picture

John Cook CreditAttribution: John Cook commented
Issue summary: View changes

Updated summary.

valthebald’s picture

valthebald
Location Sofia
CreditAttribution: valthebald as a volunteer commented
Status: Needs review » Reviewed & tested by the community

As far as I can see, there is only one call to entity_load*('tour'), and the patch from #2 fixes it.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed d429cd2 and pushed to 8.2.x. Thanks!

  • catch committed d429cd2 on 8.2.x 
    Issue #2723613 by John Cook: Remove entity_load* usage for rdf_mapping...

Status: Fixed » Closed (fixed)

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