Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
breadcrumbs.vars.php is missing code from the D7 branch. This patch re-adds it.
Comment | File | Size | Author |
---|---|---|---|
#9 | re_add_bootstrap-2408041-8.patch | 1.97 KB | neardark |
#1 | re_add_bootstrap-2408041-1.patch | 1.1 KB | neardark |
#3 | re_add_bootstrap-2408041-3.patch | 1.36 KB | neardark |
#7 | re_add_bootstrap-2408041-7.patch | 1.64 KB | neardark |
Comments
Comment #1
neardark CreditAttribution: neardark commentedComment #2
neardark CreditAttribution: neardark commentedComment #3
neardark CreditAttribution: neardark commentedActually, looks like menu_get_item has been removed. Here's the reworked ticket. Can you review when you have time -- sorry about missing this.
Comment #4
neardark CreditAttribution: neardark commentedComment #5
markhalliwellAh, yeah, this looks more D8-ish :)
If possible, we still need this class, did the breadcrumbs array item structure change as well?
This isn't needed,
$breadcrumb
is referenced from above in the code.Comment #6
neardark CreditAttribution: neardark commentedComment #7
neardark CreditAttribution: neardark commentedAhhh, thanks. Added back in active class.
Comment #9
neardark CreditAttribution: neardark commentedBah.
Comment #10
markhalliwellCode looks good to me. I'll mark as RTBC if you think it's good to go too.
Comment #12
neardark CreditAttribution: neardark commentedCommitted. BTW, I'll try and get more organized with dealing with DO's issue queue as well as keeping the two branches as close to parity as possible. J