Closed (fixed)
Project:
Jquery Slideshow
Version:
5.x-1.0
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Nov 2008 at 03:27 UTC
Updated:
12 Dec 2008 at 19:32 UTC
Jump to comment: Most recent file
Comments
Comment #1
pman2009 commentedchanged // to ; in module.info now getting
Parse error: syntax error, unexpected '{', expecting '(' in ../modules/jquery_slideshow/jquery_slideshow.module on line 156
Comment #2
pman2009 commentedI removed the theme function at the end of the .module file.
Now I am getting this error on
modules/jquery_slideshow/jquery_slideshow.module on line 84 when I go to admin/cont mgmt/Jquery Slideshows
I am not sure whats going on. I am just going to wait for your response....
Comment #3
nvoyageur commentedWhat version of Imagecache are you using? Right now it just works with the 1.x release of Imagecache. I've also included the patch to fix the missing () in the unused theme function.
Thanks,
Comment #4
pman2009 commentedThanks for the replies and patches. I am using 2.2 of Imagecache. I'll try with 1.x. I'll let you know.
Comment #5
gavacho commentedI have the same issue. It has failed with imagecache 5.x.1.7 and 2.2
Is there anywhere I can get the updated file so I can just replace it without having to bother with the patch? To me patches are a pain in the @#$ at best, and currently my drupal installation is running locally on windows wamp, and I do not have command line access to any Linux. I would really like to use this module but I need to get it working soon. Thanks
Comment #6
nvoyageur commentedPlease use the .dev version from the releases page. It has the patches applied.
http://drupal.org/node/338700/release
Patches are not hard to do manually, it tells you what lines to subtract and what lines to replace.
I'll create another official release soon.
Comment #7
nvoyageur commentedThis is fixed in version 1.1