Closed (fixed)
Project:
Cooliris (PicLens formerly)
Version:
5.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
10 Jun 2008 at 09:06 UTC
Updated:
12 Jun 2008 at 17:16 UTC
Comments
Comment #1
z.stolar commented(fixed a typo in the title and...)
BTW, firebug reports no javascript errors.
Comment #2
swentel commentedThat's weird, do you have an online example or are you testing on a local machine ?
Comment #3
z.stolar commentedI do: http://www.common-paradise.net/illustration/gallery
Tested with FF on XP, with and without the extension, as well as with FF on Linux.
Comment #4
swentel commentedok thanks for the link. The rss feeds seem to be empty (ie http://www.common-paradise.net/illustration/en/piclens/piclens_views/gal...) - are their any watchdog errors after you surf directly to this url ?
Comment #5
z.stolar commentedOK, I've done some more debugging.
Not yet done, but here are my first results:
In piclens_views module, in function piclens_views_mediarss_items, $args is used twice: Once as a parameter to the functions, which includes the name of the view, and then as the array of arguments passed on to views, in views_build_view.
To fix this, the first line should be:
Now the problem is that the arguments are not passed to the feed. If I add them manually, I get the RSS feed: http://www.common-paradise.net/illustration/en/piclens/piclens_views/gal...
("Growing+children" is the argument)
Comment #6
z.stolar commentedGot it!
I leave you to beautify the code, but I found the second breaking point:
In piclens_views module, in function piclens_views_post_view, the arguments passed to the view, are not passed to the feed_url.
Try this instead:
And now it works :)
Comment #7
swentel commentedWow great debugging! I'll test tonight with my setup at home and commit as soon as I can, I'll let you now when it's committed!
Comment #8
druvision commentedA similar issue: Piclens script not automatically included for image galleries based on the image module
Comment #9
swentel commentedTested and commited to drupal 5 branch. Thanks for the report and fix!