Closed (duplicate)
Project:
Custom Breadcrumbs
Version:
6.x-2.0-rc1
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
7 Nov 2011 at 02:09 UTC
Updated:
14 Aug 2012 at 18:03 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
boobaaSee #1327960: Invalid use of token_replace API - causes preg_match_all errors in token module.
Comment #2
fcaceres71 commentedNada I still get the error after using patch #8 and patch #14.
Comment #3
tomrenner commentedSame here. Subscribe.
Comment #4
wlcable commentedI am also getting this error after updating to the most recent version...
Comment #5
ltrainI am also getting the error.
Comment #6
bartezz commentedAlso reporting this error!
Comment #7
Anonymous (not verified) commentedalso getting this error
Comment #8
le dendrite commentedSubscribing.
I'm getting a similar error. After updating Token
user notice: The _custom_links_build_links() function called token replacement with an array rather than a string for $text in.... /sites/default/modules/token/token.module on line 263.
However the custom links on the pages pulling the error are still working...
Drupal 6.22
Token 6.x-1.18
Custom Links 6.x-1.4
Comment #9
venusrising commentedWe too are getting this error. patches not working. DB is filling up with entries
Comment #10
MGN commentedHave you tried using the latest token 6.x-1.x-dev? Reading Dave Reid's reply in #1327960: Invalid use of token_replace API - causes preg_match_all errors in token module, it seems like he restored the original functionality.
I just tried the latest dev and don't see any problems (though I was able to verify the problem in 6.x-1.18). I think the way to "fix" this at this point is to use token 6.x-1.x-dev until 6.x-1.19 is released. So long as token maintains this functionality in the 6.x-1.x branch (as Dave's comment indicates), I don't see a reason to change the 6.x-2.x version of custom breadcrumbs. Note that the D7 version of custom breadcrumbs has already been modified to use a string instead of an array because of the new token_replace in the D7 API. So there shouldn't be a problem in the D7 version either.
Comment #11
MGN commentedI take it back. It does make sense to adjust 6.x-2.x slightly to better agree with 6.x-1.x and the token API (as well as the D7 branch). Note: I committed the fix (more or less) in #5 of #1327960: Invalid use of token_replace API - causes preg_match_all errors in token module by Patrizio to the 6.x-2.x-dev branch. Note the patches in
#8or #14 of that issue wouldn't work correctly for 6.x-2.x (unless you are just using node type tokens - #8 was the basis for the patch used in the 6-x.2-x branch).Comment #12
jasom commented#8 solved my problem, tnx
Comment #13
venusrising commented@MGN We are using 6x2 RC1 on a production site. Is there a patch we can apply? BTW for those of use with production sites Dev Token is not a good answer due to potential stability issues with other modules.
Custom Breadcrumbs 6x2 RC1 has been awesome, works great with the exception of DB error not above. It still functions and we are using it with views etc and it does a great job.
Comment #14
nasi commentedSame issue here. It would be great to ideally have a new point release that includes a fix for this issue, or a patch specific to this bug. I'm not keen on pushing a dev release into production.
It's been a while since rc1 was released, is there an idea of a timeframe for another release (e.g. rc2)?
Comment #15
venusrising commentedThis issue is filling up our logs to the point that valuable data like new users events as well as spammers are not be logged since this is filling up our logs and pushing everything else out.
We are using 6x2 RC1 Please can someone let us know how to fix these massive log dumps.
.
Comment #16
jimkont commentedSame here...
subscribing
Comment #17
paulo_graca commentedI've just created this patch for a workaround.
The problem is with the most recent version of Token in D6. I hope it helps!
Comment #18
venusrising commented@paulo_graca
Thanks before I proceed can you tell us what the patch does as this is a production site but the DB logs are a killer so we are very grateful for your help.
Comment #19
venusrising commentedComment #20
nasi commentedWhich version of custom_breadcrumbs is this patch made against? It doesn't seem to correspond to the code for 2.0-rc1.
Comment #21
mmachina commentedupdate with the DEV version and the error goes away!
Comment #22
finedesign commentedSame problem here with 6.x-2.0-rc1
Comment #23
rich.yumul commentedHere's a patch for 6.x-2.0-rc1 for this issue.
Comment #24
adijeff commentedThanks Rich. #23 worked for me on the January 8 version of 6.x-2.0-rc1
Comment #25
slucas commentedThanks the patch works but not automatically, I need to patch manually.
Thanks
Comment #26
drozasThanks Rich. #25 the patch did not work automatically for me neither, I attach a new one (I think it was because of the way it was generated).
Comment #27
sgabe commentedThe patch in #26 seems to work.
Comment #28
caspercash commented#8 solved my problem too! Thanks!
Comment #29
fenstratThis problem has already been resolved and committed to both 6.x-1.x and 6.x-2.x branch in #1327960-42: Invalid use of token_replace API - causes preg_match_all errors in token module
Comment #30
Anonymous (not verified) commentedThe problem is still in 6.x-2.0-rc1
Comment #31
fenstrat@CompuBase, Please look at #1327960-42: Invalid use of token_replace API - causes preg_match_all errors in token module
Comment #32
Anonymous (not verified) commented@fenstat, sorry assumed the rc had a later date than the dev version. I must have missed the year difference.
Comment #33
fenstrat@CompuBase, no worries, I've done that myself more than once.
Comment #34
dkane commentedI can confirm that #26 also worked for me as well.
Thank you everyone for your hard work on this!
Comment #35
fenstrat@dkane, This problem has already been resolved and committed to both 6.x-1.x and 6.x-2.x branch in #1327960-42: Invalid use of token_replace API - causes preg_match_all errors in token module. This issue is a *duplicate* of that issue.
I believe I'm beginning to sound like a broken record here.
Comment #36
mmachina commentedI'm using 6.x-2.0-rc1 and the patch in comment #26 worked perfecto!!!
Comment #37
pcoughlin commentedPatch in #26 solved the problem. Tested with 6.x-2.0-rc1
Comment #38
rrafferty commentedSorry to be dense, but short of running patch at the command line is there anyway to fix this? I do not have access to command line. My log file, however, is growing by the minute.
Comment #39
rrafferty commentedIf you want to apply this patch manually, here is a tutorial. It was actually pretty easy.
http://drupal.org/node/534548
Comment #40
cknoebel commented#26 works for me, thanks for the patch.
Comment #41
bessone commented#26 works for me too, thanks
Comment #42
arlina commented#26 did the trick. thanks!
Comment #43
fenstratPlease see #35.
Comment #44
dewolfe001 commentedI used the latest dev version of Token and that remedied my problems.
Comment #45
MatteNoob commentedI also updated to the latest dev version of token (token-6.x-1.x-dev.tar) and it resolved the issue for me too.
Comment #45.0
MatteNoob commentedToken module updated