Additional configuration to use tokens? [#794648]

Using custom breadcrumbs 6.x-2.0-beta2 and Token 6.x-1.12 I have a custom breadcrumb for path set up to display the year of the node: [yyyy].
However, when I upgraded to beta3, instead of displaying the year (e.g.: 2010) it displayed: [yyyy]
When I looked at the edit page for the custom breadcrumb I could see all the tokens listed - including [yyyy]

I've reverted to beta2 and the year is now displayed.

I had a look at the configuration page, but couldn't see anything obviously connected with tokens. Is there some additional configuration required?
I have Custom Breadcrumbs for Paths, Views and Taxonomy installed.

Comment	File	Size	Author
#4	794648_cb_paths_preprocess_fix.diff	4.47 KB	MGN
#4

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Honza Pobořil CreditAttribution: Honza Pobořil commented 11 May 2010 at 09:07

Same problem, first installation.

To path field i wrote this:

noticeboard
guestbook
<pathauto>|[menupath]

Is it correct?

Comment #2

MGN CreditAttribution: MGN commented 11 May 2010 at 13:59

Status:

Active

» Postponed (maintainer needs more info)

@Bobik, is [menupath] showing up in the breadcrumb without being properly replaced? Is this for a node-type custom breadcrumb?

Thanks for these reports, I'll be looking into this with a fresh install on a new test site within the week.
So far I have yet to be able to reproduce this, but my test sites are getting a little out of date....

Comment #3

MGN CreditAttribution: MGN commented 13 May 2010 at 04:40

Category:	support	» bug
Status:	Postponed (maintainer needs more info)	» Active

@antcw Thanks for the report. I've found the problem and am working on a solution....

Comment #4

MGN CreditAttribution: MGN commented 14 May 2010 at 03:34

Status:

Active

» Needs review

File	Size
794648_cb_paths_preprocess_fix.diff	4.47 KB

It seems that the problem was introduced by move away from hook_init to hook_preprocess (in issue #627792: Custom Breadcrumb option causes Access Denied in OG Forums). Node type tokens aren't being replaced because custom_breadcrumbs_paths_preprocess is calling _custom_breadcrumbs_paths_set_breadcrumb() after custom_breadcrumbs_paths_nodeapi, wiping out the correct custom breadcrumbs paths breadcrumb.

To prevent this from happening, this patch gives custom_breadcrumbs_paths_nodeapi a higher priority than custom_breadcrumbs_paths_preprocess. Can you test the patch and verify that it fixes the problem?

Comment #5

antcw CreditAttribution: antcw commented 18 May 2010 at 20:16

I finally managed to get around to testing this and I can confirm that it fixes the problem I was seeing.
Thanks very much! - and thank you for a very useful module.