Hi,

The module (6.x-1.0) seems to generate thumbnails correctly (they appear, properly named, in my gallery/flash/thumbs directory), however, as far as I can tell they are not being used by simpleviewer.

Taking a look at flash_gallery.module, I see that line 226 is:

$op[] = 'thumbPath="'. flash_gallery_image_url() .'/"';

This sets the thumbnail path to match the actual image path, as opposed to using the available thumbnails. I think it should be:

$op[] = 'thumbPath="'. flash_gallery_image_url() .'/flash/thumbs/"';

Is this correct?

Ryan

Comments

rjacobs’s picture

rjacobs commented
Status: Active » Needs review

I changed the status as (based on my testing) the small modification I mentioned above seems to fix things.

Has anyone else noticed this issue with 6.x-1.0?

I think this is just a 1-line code change, but should it be submitted as a patch?

Ryan

danieltome’s picture

danieltome commented

Ok.. I will include the change in the next version.

I also have some updates for the dev version, that will be included shortly.

thanks

rjacobs’s picture

rjacobs commented
Status: Needs review » Reviewed & tested by the community

Cool, thanks, I'll mark it as "reviewed" then.

Best,
Ryan