Returntype incorrect for \Drupal\Core\Entity\EntityDeleteMultipleAccessCheck::$tempStore

Applied the MR and searched for "#^Variable \\$selection in empty and only 1 instance which was fixed

LGTM

It wasn't clear from the diff alone why this mixup existed, but reading EntityDeleteMultipleAccessCheck::construct() locally makes it clear where it got out of sync.

Adding credit for @smustgrave for the scope check.

Committed to 11.x. I went back and forth on whether to backport this. Technically, this could disrupt contrib that was setting the member to the factory because that's what the typehint told them to do. That's edgecase enough that I think it's okay for a minor, but I decided to err on the side of caution and not backport it to a 10.1.x patch release.

Thanks!

Spokje → closed merge request !4795

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