Blocks cause WSOD; Cannot generate default URL

The default argument to toUrl is 'canonical', I'm not sure why this makes a difference?

The described fix works for me.

Same thing happened to me after updating core to 10.2.1 (amongst other modules).
This fix seems to help, thanks.

From what I can see Drupal\aggregator\Entity\Item is missing the "links" annotation.
Drupal\aggregator\Entity\Feed has one.

This was caused by this change to EntityBase::toUrl() in Core version 10.2.x: https://git.drupalcode.org/project/drupal/-/commit/9c4bc86ccbeb7c28b9217.... canonical is no longer the default, NULL is. Then if the rel is NULL, it checks to see if the entity type defines either a canonical or edit_form link. If neither of those is defined, then it throws the exception shown here. So basically Core doesn't support not having a links defined in your entity annotation anymore, like aggregator_item.

Entering canonical as the rel argument fixes this problem because it bypasses the new code added in that commit. EntityBase::toUrl() then falls back to the annotated uri_callback, which is what builds the Item URLs.

Although we can insert canonical into the block's build() function, I'm not convinced this is the best fix. Apparently, Core wants to remove the uri_callback annotation because it was a BC hack for D7 compatibility or something. See 📌 [meta] convert ‘uri_callback’ entities param to EntityInterface::uri() method Active for details. Since we have to make a change to fix this problem, maybe we should take action to remove the uri_callback now.

But if anyone needs to fix a broken site right now, then putting canonical into build() will work at the moment.

We should be reporting this on the core issue too, seems like a BC break

We should be reporting this on the core issue too, seems like a BC break

I was thinking the same.

I have done that

Thank you. I misunderstood and was looking for an existing issue about the BC break.

Looks good to me 🙌

There were several test failures. I'll have to finish working on them tomorrow because I don't yet know why they're happening.

Added 📌 Drupal 10.2.3-dev breaks Aggregator tests Active as a parent issue since the broken block now causes Aggregator's tests to fail. The error described here shows up in the failure output.

I reverted the change for the unrelated test failures just to keep this commit clean. I'll redo that change in 📌 Drupal 10.2.3-dev breaks Aggregator tests Active .

Thank you, everyone! This will be released in version 2.2.0 later today. I'm going to see if I can get the other test failures fixed first. But if I can't, then I'll go ahead and release this bug fix despite the failing tests.

Version 2.2.0 has been released.

Many thanks.

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

Wow, sorry for causing trouble due to the core issue! Opened 🐛 [regression] Entity::toUrl() without argument is broken for entity types with a URI callback Needs review now to rectify this, so hopefully at least no-one else will be hit by this.