Bugtracker » Slideshow » slideshow easyshop menu » View bug

Jump to 
Filters 
Search
Application Slideshow Version 2.1
Description Fancy slide show of your latest news items or downloads.
Note: Check if your bug report is not already an existing one to avoid duplicates.
Features, ideas and requests should go in the Slideshow requests section.
Owner nlstart
#101 [Core] slideshow easyshop menu
Posted by bryanr1973 Friday 12 April 2013 - 16:55:39 Last updated by nlstart Saturday 13 April 2013 - 14:56:24
Owner nlstart Priority Critical Found in version 1.1
Status Closed Resolution Implemented Fixed in version 2.1
Description
There is a problem where no images display. This is the link that the plugin creates for the images

It seems to have the image name and folder reversed
Related bugs
None.
Developer comments
No comments.

LAN_5

bryanr1973
12 Apr 2013: 16:58
Reply to this
this error was in version 2 - paid version
bryanr1973
13 Apr 2013: 10:36
Reply to this
I checked all strings in slideshow_easyshop_menu.php
They all display correctly (img_path, etc)

However in slideshow_class.php the variables in function gallery_element are inverted

original code read - <img src="'.$p_img_path.$p_image.'" alt="" />

I reordered it to - <img src="'.$p_image.$p_img_path.'" alt="" />

I am not sure where these are assigned their values, but that seems to be the issue. They were assigned inversely. Now img src returns the correct image. However, the image still isn't displaying properly.
bryanr1973
13 Apr 2013: 10:49
Reply to this
Now that images are loaded, they need to be seen. Temporarily removing slideshow_shop.css line 105 display:none allows the inital image to show up.
Line 1 has an overflow limitation set. Removing this shows that all images are stacked correctly.

The righthand column that includes the small navigation tabs are not displaying yet. (assuming there are any)

Also the rotation doesn't seem to be working. I will work on the rotation issue next.
nlstart
13 Apr 2013: 14:52
Reply to this
You're right; the parameters were not passed properly. Only in the easyshop slideshow... Duh. I have fixed it and will update the download section soon.

You must be logged in to make comments on this site - please log in, or if you are not registered click here to signup