BlockLibraryController typehints LazyContextRepository, not the interface

Trivial MR flying in.

Commit url applies cleanly on my site and fixes the issue for me: https://git.drupalcode.org/project/drupal/-/commit/900e76dac8973176991a0...

Trivial MR flying in.

Commit url applies cleanly on my site and fixes the issue for me.

And pipeline is green now.

Wonder if while we are at it we should use constructor promotion so all the params are correct typehinted.

Just to be safe though I wonder if we should add a simple deprecation that triggers if someone passes in the old type. Just to be safe.

I can confirm the mr/patch fixes the issue for me as well. Was getting the exact same error as in the IS.

Will leave it at NW for #5.

@smustgrave
> Just to be safe though I wonder if we should add a simple deprecation that triggers if someone passes in the old type. Just to be safe.

Sounds like sth got confused: Is so "passes in the old type" (LazyContextRepository), it is always an instance of ContextRepositoryInterface (the "new type") and will always be. So nothing to deprecate here.

RTBC as of #6.

Committed to 11.x and backported to 10.4, 10.5 and 11.1

No backport to 11.0 and 10.3 as they're now security only.

larowlan → closed merge request !10191

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