An entity without a label causes an uncaught exception for the navigation:title component

Just cross-referencing things, I did run into the same error, but for me, ✨ SDC ComponentElement: Transform slots TranslatableMarkup values to #markup instead of throwing an exception Active helps because it's a TranslatableMarkup object, not null.

I assume SDC has optional slots, but I think this is a navigation issue. I think an label should either fall back to some fallback string like "- No label -" or possibly skip the whole plugin as it can't really provide meaningful information?

Good find. Tagging as Needs test, it's not obvious how to reproduce it. I was able to reproduce it in a very hacky way by adding the following implementation of label() to \Drupal\node\Entity\Node, just to illustrate the issue:

/**
   * {@inheritdoc}
   */
  public function label() {
    return NULL;
  }

IMO, the plugin should be skipped if there isn't a useful title or badge. We skip the badge, avoiding this issue if there isn't one. We should do the same for the label.

Fork is a little bit behind...

This patch applies to 11.2.0 (tested)

The Commerce Invoice module encounters this bug.

abhijith s → made their first commit to this issue’s fork.

Moving to major since this issue essentially blocks displaying content when en entity without label is used.

Getting the same error when I view the logs using the AI logging sub-module with Drupal CMS.

Drupal: 11.2.2

We are running into this issue when trying to translate content. The updated check for the label didn't solve our issue, however, casting $label to a string got us past our issue. Not sure if this introduces additional translation issues, but so far it has been working.

          '#slots' => [
            'content' => Element::isRenderArray($label) ? $label : (string) $label,
          ],

Tested with 11.2

Added new iteration to latest MR including code from #12

@plopsec, your MR works perfectly and solved the issue for me.