small bug? (loop) with additional images in 2.0.22c

[slammy]

Hi,

I am currently running jml 2.5.14 and vm sr 2.0.22c at my live-site

I think I´ve found a small bug(?) in 2.0.22c -> concerning additional-Images in components/com_virtuemart/views/productdetails/default_images.php, which is loading at components/com_virtuemart/views/productdetails/tmpl/Default.php with loadTemplate('images');

I have some products with multiple Images:

If I have 5 additional images on a product, all five images will be repeated 5 times.

I found a "for-loop" at components/com_virtuemart/views/productdetails/default_images.php:

Code Select Expand

<div class="additional-images">
                <?php
                for ($i = 0; $i < $count_images; $i++) {
                        $image = $this->product->images[$i];
                        ?>
            <div class="floatleft">
                    <?php
                        echo $image->displayMediaFull('class="product-image" style="cursor: pointer"',false,"");
                    ?>
            </div>
                        <?php
                }
                ?>


Additional Images will be shown only onetime if I set the Loop to 0

Code Select Expand

for ($i = 0; $i < 0; $i++) (alternatively you can comment it out!)
This worked for me.

Seems that the loop handles all additional images in one, and repeats them x-times - equals to the amount of additional images you have.

regards jens 


[attachment cleanup by admin]