Is anyone working on a D6 update for this? I will try...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joemoraca’s picture

FileSize
16.07 KB

Using the "coder" module I have it converted ... but I think I have a problem with the theme / output side. It "seems" to me that the variables are being set and passed to the slideshow.tpl.php but it is not using them correctly. (I just copied the code from the original module) ...

I have attached a zip of where I am now.

joemoraca’s picture

Title: D6 ? » Help upgrading Slideshow to Drupal Version 6
Category: task » support
FileSize
16.12 KB

I have made some small progress ... at least the picture shows up.

Now I think the JavaScript may be part of the problem -- or something is not getting passed into the JS -- but I don't really know yet....

Attached is another zip with my latest work

intu.cz’s picture

Hello, I would like to help, the only thing I can do, though, is testing (little knowledge of PHP, no knowledge of JavaScript).

I looked at the last version (v2) and Firebug shows the following JavaScript error:

$("Only local images are allowed.").src is not a function
preloadImages(2)slideshow.js (line 14)
(no name)()slideshow.js (line 44)
e([div#slideshow-32.slideshow], preloadImages(i), undefined)jquery.js (line 13)
e(preloadImages(i), undefined)jquery.js (line 13)
(no name)()slideshow.js (line 6)
e()jquery.js (line 13)
e()jquery.js (line 13)
e([function(), function()], function(), undefined)jquery.js (line 13)
e()jquery.js (line 13)
[Break on this error] n.image = $('Only local images are allowed.').src(n.src).load(function() { preloadImages(i); ...

twice...

Roman

michael_ah’s picture

are there any news about "slideshow" for drupal 6? is there any possiblilty to accelerate the upgrade? spending any money for the developer of this modul?

i think it is a very nice modul - i would very like to use it in drupal 6

dragonwize’s picture

Title: Help upgrading Slideshow to Drupal Version 6 » D6 Version
Category: support » task
FileSize
14.13 KB

Using joemoraca's slideshow_v2, I've created slideshow_3. The javascript slideshow and the admin settings work now. The non-javascript fallback is not working for me and I haven't tested the revision features that it appears the module has. But it is working well enough for me for now!

joemoraca’s picture

dragonwize,

Thanks for helping out.... I have plenty to learn so I need to look at what you did.

It worked first try for me .... don't really care about non-JS or revisions

awolfey’s picture

It's not working for me.

3 things.

1. The page never finishes loading and the slideshow never starts.
2. If I use the next image link, it cycle through with increasing numbers twice. If I have 3 images, it counts, 1 2 3 4 5 6 where 1 2 3 and 4 5 6 are the same.
3. If I use the previous link I get page not found.

I would be happy to try to work on this. Can anyone point me in the right direction?

Thanks.

awolfey’s picture

Here's what I've found.

1. This happens because the preload function hangs.
2. For a reason I could not figure out, $images or maybe $key seems to be set to include the images or count twice.

Here is the result for three images attached.

<!--//--><![CDATA[//><!--
jQuery.extend(Drupal.settings, { "basePath": "/", "admin_menu": { "margin_top": 1 }, "slideshow": { "slideshow-8": { "images": { "1": { "src": "http://example.com/sites/default/files/slideshow/bagels.jpg", "title": "bagels.jpg" }, "2": { "src": "http://example.com/sites/default/files/slideshow/bagels2.jpg", "title": "bagels2.jpg" }, "3": { "src": "http://example.com/sites/default/files/slideshow/bike.jpg", "title": "bike.jpg" }, "4": { "src": "http://example.com/sites/default/files/slideshow/bagels.jpg", "title": "bagels.jpg" }, "5": { "src": "http://example.com/sites/default/files/slideshow/bagels2.jpg", "title": "bagels2.jpg" }, "6": { "src": "http://example.com/sites/default/files/slideshow/bike.jpg", "title": "bike.jpg" } }, "total": [ 3, 3 ], "current": [ 1, 1 ] } } });
//--><!]]>

kirkcaraway’s picture

Is there any more progress on porting this to D6? Very interested in using it. Thanks for all your work on this.

chasz’s picture

Status: Active » Needs work

subscribing +1

vikramy’s picture

Hi,

I just tested slideshow_v3.tar_.gz in Drupal6.6 and its working..

But I havent tested to full extent but will get back in a while.

Thanks for the great work.

helderluis’s picture

Captions not working in 6.9. Besides that it works fine...
This is a module that should have been bundled in Drupal a long time ago, it's a simple and very effective slideshow. My clients love it.

How can we port this to Drupal 6 the right way?

Todd Nienkerk’s picture

Re: #5 above: The posted 6.x version of the module (slideshow_v3.tar_.gz) includes a possible bug I found in the 5.x version: #152261: DIVs not closed properly?.

Line 173 of the new version of the module includes the possibly superfluous closing div:

    '#suffix' => '</div>',
snorkers’s picture

I installed the module at #5, and commented out the suffix (#13) just to try the module out. Pretty simple and worked OK in D6, only issue was when the module was disabled, the CCK content type Slideshow was still listed, albeit with a few few fields deleted.

Drupalec’s picture

I'm using 'non-official' version of slideshow for drupal v6. in v5 i was able to remove Previous, Number # of #, and Next text/links when there was only one image in slideshow (like this: http://drupal.org/node/147757).

In new version there is seperate html file. how can i accomplish same thing with newer version? any help appreciated.

kkaefer’s picture

There will be an official version of slideshow.module for Drupal 6 within the next two weeks. I promise.

Drupalec’s picture

I think the best way to hide 'header' div (Previous | Image 2 of 91 | Next ) is to use javascript so i put function like:

    $(document).ready(function() {
      if (slideshow.total = 1) {
        $('div.header').hide();
        }
    });

but i'm not that good at jquery and drupal and it doesn't work. any sugestions?
If i put this code to slideshow.js file it hides all 'header' divs for all slideshows:


    $(document).ready(function() {
       $('div.header').hide();
     });

thanks!

kkaefer’s picture

Status: Needs work » Fixed

Updated version is in CVS. Thanks!

Status: Fixed » Closed (fixed)

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