Entity load multiple needed?

[laryn]

I was looking through a codebase for something else and happened to see this line:

foreach (entity_load('file', FALSE, array('uri' => $uri)) as $file) {

It looks like that is an outdated use of entity_load that should be changed to entity_load_multiple.

[herbdool]

Your PR works for me @laryn

[argiepiano]

This change is fine for clarity, but entity_load() in Backdrop actually invokes entity_load_multiple() if it finds that the second argument is an array or FALSE or NULL... so in the end the correct function is used either way.

[laryn]

Yes, I think I went on a little spree one day after greping my codebase and @argiepiano corrected me on another thread too. Oops!

[argiepiano]

To be honest I prefer entity_load_multiple()!