Sorting an array in foreach

Question

I am trying to sort an array inside foreach. Following is my code:

<?php $_images = Mage::getModel('catalog/product')->load($_product->getId())->getMediaGalleryImages(); ?>
<?php $i = 0; foreach ($_images as $_image){ $i++; ?>
<img src="<?php echo $this->helper('catalog/image')->init($_product, 'thumbnail', $_image->getFile())->resize(300,300); ?>" style="width:50px;height:50px;">
<?php } ?>

I need the images to be printed out in the correct order. It should be ordered by the value of:

$_image[position_default]

I have tried using Ksort:

<?php $_images = Mage::getModel('catalog/product')->load($_product->getId())->getMediaGalleryImages(); ?>
<?php $i = 0; foreach ($_images as $_image){ $i++; ?>
<img src="<?php echo $this->helper('catalog/image')->init($_product, 'thumbnail', $_image->getFile())->resize(300,300); ?>" style="width:50px;height:50px;">
<?php ksort($_image['position']);  ?>
<?php } ?>

But it does not sort them at all. Maybe i don't use it correctly? Any suggestions? :)

Some output of $_image:

I can't tell if you want to sort by the $_image['position'] key or the $_images[key] key? — h2ooooooo
– h2ooooooo, Commented Nov 21, 2013 at 15:05
$_image is going to be a single image. Sorting a single-element array is pointless. You need to ksort $_images, and since you're sorting a specific key, probably need to use uksort. — Marc B
– Marc B, Commented Nov 21, 2013 at 15:11

rolfv1 · Accepted Answer · 2013-11-21 18:37:44Z

2

You will need to use a custom sorting function, because you will want to compare with the value of one of the "properties" of each image. (see http://www.php.net/manual/en/function.usort.php) So before the loop, put:

function cmp($a, $b)
{
    return strcasecmp($a['_data']['position_default'], $b['_data']['position_default']);
}

usort($_images, "cmp");

Edit: I changed from uksort to usort, made a mistake there.

Edit2: Given the output for $_images that you added, the position-default property is one level deeper, so I added the ['_data'].

edited Nov 21, 2013 at 18:37

answered Nov 21, 2013 at 15:15

rolfv1

5711 gold badge3 silver badges14 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

russellc · Accepted Answer · 2013-11-21 15:05:24Z

1

At first glance, it looks like the ksort should happen before the foreach loop.

<?php ksort($_images);  ?>
<?php $i = 0; foreach ($_images as $_image){ $i++; ?>
<img src="<?php echo $this->helper('catalog/image')->init($_product, 'thumbnail', $_image->getFile())->resize(300,300); ?>" style="width:50px;height:50px;">
<?php } ?>

Give that a try. Let me know how it goes!

answered Nov 21, 2013 at 15:05

russellc

4943 silver badges7 bronze badges

1 Comment

russellc Over a year ago

Do you think you could provide a set of the data (at least the structure) returned by the Mage::getModel() call?

Collectives™ on Stack Overflow

Sorting an array in foreach

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related