#2529560: Expand support for link objects landed, now let's use it for \Drupal\Core\Breadcrumb\Breadcrumb, so that we have >1 example of RenderableInterface being used.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wim Leers created an issue. See original summary.

Wim Leers’s picture

Status: Active » Needs review
FileSize
1.09 KB
nlisgo’s picture

Would it be appropriate to use the @inheritdoc here?

Wim Leers’s picture

FileSize
1.19 KB
613 bytes

Absolutely, great point!

Status: Needs review » Needs work

The last submitted patch, 4: 2565925-4.patch, failed testing.

Wim Leers queued 4: 2565925-4.patch for re-testing.

Wim Leers’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 4: 2565925-4.patch, failed testing.

Crell’s picture

Not sure what the fails are, but +5 to using RenderableInterface here.

nlisgo queued 4: 2565925-4.patch for re-testing.

nlisgo’s picture

Status: Needs work » Needs review
dawehner’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Novice

Nice!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committing this under the followup proviso in the beta evaluation policy. Committed e03e4e8 and pushed to 8.0.x. Thanks!

  • alexpott committed dc29644 on 8.0.x
    Issue #2565925 by Wim Leers: Let Breadcrumb implement...

Status: Fixed » Closed (fixed)

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