This function does not accept the breadcrumb parameter.

/**
 * Return a themed breadcrumb trail.
 *
 * @param $breadcrumb
 *   An array containing the breadcrumb links.
 * @return
 *   A string containing the breadcrumb output.
 */
function corporateclean_breadcrumb($variables){
  $breadcrumb = $variables['breadcrumb'];
  if (!empty($breadcrumb)) {
    $breadcrumb[] = drupal_get_title();
    return '<div class="breadcrumb">' . implode(' <span class="breadcrumb-separator">/</span> ', $breadcrumb) . '</div>';
  }
}
CommentFileSizeAuthor
#2 quickfix_1429176-1.patch635 byteschi
#1 quickfix_1429206-0.patch647 byteschi

Comments

chi’s picture

chi commented
Status: Active » Needs review
StatusFileSize
new quickfix_1429206-0.patch647 bytes
chi’s picture

chi commented
Assigned: gtsopour » chi
Status: Active » Needs review
Issue tags: -corporate, -Corporate Theme, -Corporate Clean Theme
StatusFileSize
new quickfix_1429176-1.patch635 bytes

deleted

gtsopour’s picture

gtsopour commented
Status: Needs review » Active

Hello Chi and thank you for reporting this,

Could you check please the following function inside template.php of Drupal 7 Garland theme? I think that the following function returns a themed breadcrumb trail with the same way as corporateclean_breadcrumb().

/**
 * Return a themed breadcrumb trail.
 *
 * @param $breadcrumb
 *   An array containing the breadcrumb links.
 * @return a string containing the breadcrumb output.
 */
function garland_breadcrumb($variables) {
  $breadcrumb = $variables['breadcrumb'];

  if (!empty($breadcrumb)) {
    // Provide a navigational heading to give context for breadcrumb links to
    // screen-reader users. Make the heading invisible with .element-invisible.
    $output = '<h2 class="element-invisible">' . t('You are here') . '</h2>';

    $output .= '<div class="breadcrumb">' . implode(' › ', $breadcrumb) . '</div>';
    return $output;
  }
}

Thank you for your time.
/George

gtsopour’s picture

gtsopour commented
Assigned: chi » gtsopour
Issue tags: +corporate, +Corporate Theme, +Corporate Clean Theme
chi’s picture

chi commented
Assigned: chi » gtsopour
Status: Needs review » Active
Issue tags: +corporate, +Corporate Theme, +Corporate Clean Theme

Yup, I have already filed bug report about garland_breadcrumb(). It seems like many contributed themes just copied garland implementation of this. I am going to report the issues for others.

gtsopour’s picture

gtsopour commented

Hello Chi and thank you for your answer,

i will examine this issue more carefully.

Thanks again
/George

alexandreracine’s picture

alexandreracine commented

Is this fixed? Thanks.