Part of meta-issue #2225347: [META] Replace calls to ENTITY_TYPE_load() and ENTITY_TYPE_load_multiple() with static method calls

See the detailed explanations and examples there.

CommentFileSizeAuthor
#1 drupal8-entity_system-menu-link-content-load-2321913-1.patch2.1 KBtemoor

Comments

temoor’s picture

temoor commented
Status: Active » Needs review
StatusFileSize
new drupal8-entity_system-menu-link-content-load-2321913-1.patch2.1 KB

Actually it was menu_link_content entity.

m1r1k’s picture

m1r1k commented
Issue tags: +#ams2014contest
Michael Hodge Jr’s picture

Michael Hodge Jr commented
Status: Needs review » Reviewed & tested by the community

I applied the patch, grepped the code, and can confirm everything looks as it should.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 1: drupal8-entity_system-menu-link-content-load-2321913-1.patch, failed testing.

Status: Needs work » Needs review

Temoor queued 1: drupal8-entity_system-menu-link-content-load-2321913-1.patch for re-testing.

dawehner’s picture

dawehner
Primary language German
commented
Title: Replace all instances of menu_link_load(), menu_link_load_multiple(), entity_load('menu_link') and entity_load_multiple('menu_link') with static method calls » Replace all instances of entity_load('menu_link_content') and entity_load_multiple('menu_link_content') with static method calls
Status: Needs review » Reviewed & tested by the community

The issue title is BS

alexpott’s picture

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

Committed 341e8d6 and pushed to 8.0.x. Thanks!

  • alexpott committed 341e8d6 on 8.0.x 
    Issue #2321913 by Temoor: Replace all instances of entity_load('...

Status: Fixed » Closed (fixed)

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