Closed (fixed)
Project:
Frequently Asked Questions
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Jan 2011 at 04:51 UTC
Updated:
10 Sep 2011 at 04:51 UTC
Jump to comment: Most recent file
Comments
Comment #1
bensnyder commentedcommented out
Comment #2
ChrisFlink commentedThe Drupal 6 function token_get_list() doesn't exist in the Drupal 7 version.
Below is what the Drupal 6 version looks like, I haven't found the Drupal 7 variant yet...
Comment #3
dave reidYou could always look to the Pathauto version for Drupal 7 to see what you're supposed to do...
Comment #4
ChrisFlink commentedIf I understand it correctly (and please check because I am not familiar with the faq, token or pathauto module) the code should be changed in three ways:
The patch attached contains these three changes, I am unsure about the last one, especially about what the default value should be,
I hope this helps in fixing this issue. Thanks Dave for giving me some new insights!
Comment #5
d.olaresko commentedtested it. Works fine for me. thx
Comment #6
p.brouwers commentedReviewed the patch and confirmed that it works perfectly now.
Comment #7
bluesomewhere commentedpatch worked for me, much thanks.
Comment #8
sarjeet.singh commentedhttp://drupal.org/node/1040468#comment-4018596
patch worked....thanks you.
Comment #9
claudiu.cristeaWorks. Why not commit?
Comment #10
TimelessDomain commentedpatch worked for me - please commit
Comment #11
jief commentedRerolling patch from MadChris to be compatible with Drush Make
Comment #12
damienmckenna+1 for the patch in #11.
Comment #13
jeffwidman commentedsubscribe
Comment #14
bramface commented+2 for the patch in 11 after upgrading to dev versions of pathauto and token in order for pathauto patterns to display.
Comment #15
mstrelan commented+3 for the patch in 11
Comment #16
mxtI've got the same issue: can this patch be commited?
Thank you.
Comment #17
pbuyle commentedIn #1052470: FAQ 7.x uses Pathauto 6.x API, I made a patch that fix this issue and two others undefined function calls related to Pathauto.
Comment #18
SilviaT commentedWaiting for this patch to be committed too.
Comment #19
thijsvdanker commentedsubscribe and +1
Comment #20
teamA commented+1 for commit - solved the issue in #1052470: FAQ 7.x uses Pathauto 6.x API as well.
Comment #21
recidive commented+1 tested an fixed the problem.
Bumping to critical as it makes pathauto module unusable.
Comment #22
mikefyfer commentedpatch in comment 11 worked for me too.
Comment #23
LagPRO-dupe commentedPatch #4 worked.
Comment #24
rmiddle commented+1
Comment #25
dadderley commentedThe patch in 11 fixed it for me.
Comment #26
tinarey commented+1 for #11
Comment #27
stella commentedCommitted, thanks!
Comment #28
rmiddle commentedConfirming the latest dev version fixed this issue for me :).
Thanks
Robert