a breadcrumb is set on front page if defaut front page settings is set different to "node"

CommentFileSizeAuthor
#1 breadcrumb-on-frontpage.patch720 bytesrstamm

Comments

rstamm’s picture

rstamm commented
StatusFileSize
new breadcrumb-on-frontpage.patch720 bytes

This patch works with system urls but also with aliases.

rstamm’s picture

rstamm commented
Status: Active » Needs review
steph’s picture

steph commented
Status: Needs review » Reviewed & tested by the community

I think this

if (drupal_get_normal_path(variable_get('site_frontpage', 'node')) == $_GET['q'])

could be encapsulated somewhere else, since a similar behaviour seems to be reproduced in some places in the code.

Anyway, the last patch seems to work and code seems fine.

steph’s picture

steph commented

I also reviewed the similar patch here : http://drupal.org/node/58065

I'll let the expert decide which patch is the best :-)

rstamm’s picture

rstamm commented
Status: Reviewed & tested by the community » Fixed

fixed by this http://drupal.org/node/59222

rstamm’s picture

rstamm commented
Status: Fixed » Closed (fixed)