Wrong typhing for EntityStorageInterface::loadMultiple()/resetCache()

Thanks, I'm finding this produces silly errors in my 11.1 builds also.

Added feedback to MR, otherwise this is a nice quick fix.

I think we should go with @param array<mixed>|null $ids as this is consistent with \Drupal\Core\Entity\EntityStorageInterface::load(), \Drupal\Core\Entity\EntityStorageInterface::loadUnchanged - or we should change those here too... Given the type hint on \Drupal\Core\Entity\EntityInterface::id() is @return string|int|null I guess the latter option is best.

There are test failures which have to be unrelated and known recurring failures. I think this is RTBC otherwise.

I did observe that problem as well. The type hints string|int and array<string|int>|null are the correct choice.

catch → closed merge request !11232

Looks right. Committed/pushed to 11.x, thanks!

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