The patch in #916388-161: Convert menu links into entities added a lot of different kind of entity_load_multiple_by_properties() uses.

In general they should be better converted to use entity_query, as entity_load_multiple_by_properties() will maybe disappear one day.

Comments

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok commented
Issue summary: View changes

Updated issue summary.

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
CreditAttribution: wizonesolutions as a volunteer commented
Issue summary: View changes
Parent issue: » #1887058: Remove usages of entity_load_multiple_by_properties() and EntityStorageController::loadByProperties()
dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Issue tags: +Novice

That can be a novice task, I guess ...

felribeiro’s picture

felribeiro CreditAttribution: felribeiro at CI&T commented

There is no more
$menu_links = entity_load_multiple_by_properties('menu_link'... in the code.

Some of them were changed to
$menu_link_manager = \Drupal::service('plugin.manager.menu.link');
$menu_links = $menu_link_manager->loadLinksByRoute('menu_test.menu_name_test');

Others were removed with the entire function.

lokapujya’s picture

lokapujya
Primary language English
Location Boston
CreditAttribution: lokapujya at Babson College commented
Status: Active » Closed (duplicate)

https://www.drupal.org/node/1887058