I'm using this code

    $node->content['slider_piclens_start'] = array('#value' => mediarssapi_piclens_lite_link('', FALSE));

So that each of my piclens enabled nodes can fire off its slide show. This works for a single node page view but not a view or /node page where more then one node is displayed with the link. This appears to be that these links are missing the id of the feed to start.

CommentFileSizeAuthor
#2 piclens_individual_feed.patch1.72 KBswentel

Comments

swentel’s picture

swentel commented

The easiest solution will be to introduce a third argument in that function where you can pass along the the url of the feed. Your url should look like 'piclens/yourmodule/yourargs' then. I'll get this fixed in the weekend normally.

swentel’s picture

swentel commented
Status: Active » Needs review
StatusFileSize
new piclens_individual_feed.patch1.72 KB

Hi, can you test the attached (untested) patch ?
You need to add a third argument to the function as mentioned in #1

dldege’s picture

dldege commented

That worked - nice!

Output is now

<a href="javascript:PicLensLite.start({feedUrl:'http://localhost/sliderdev/piclens/slider_piclens/579'});">Start Slideshow <img width="16" height="12" border="0" align="absmiddle" title="PicLens" alt="PicLens" src="/sliderdev/sites/all/modules/drupal-contrib/piclens/PicLensButton.png"/></a>
swentel’s picture

swentel commented
Status: Needs review » Fixed

great,
commited to D5 and D6 branch. Thanks for the report and the test!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.