WordPress Development Stack Exchange is a question and answer site for WordPress developers and administrators. Join them; it only takes a minute:

Sign up
Here's how it works:
  1. Anybody can ask a question
  2. Anybody can answer
  3. The best answers are voted up and rise to the top

I'm working on WordPress gallery. I display images with wp_get_attachment_image_src function:

foreach($slideshow_data['attachment_ids'] as $attachment_id) {
    $img_data = wp_get_attachment_image_src( $attachment_id, 'full' );
    echo "<img src=\"{$img_data[0]}\" width=\"{$img_data[1]}\" height=\"{$img_data[2]}\">";

Which results in over 140 database queries per page for large galleries. Of course this can be cached, but I feel that this is not correct way.

My question is, am I doing something wrong?

Thank you.

share|improve this question
up vote 3 down vote accepted

Usually for gallery post types with lots of attachments, you would do it like this:


Inside a "Wordpress Loop":

$args = array( 'post_type' => 'attachment', 'numberposts' => -1, 'post_status' => null, 'post_parent' => $post->ID ); 
$attachments = get_posts($args);
if ($attachments) {
    foreach ( $attachments as $attachment ) {
        echo apply_filters( 'the_title' , $attachment->post_title );
        the_attachment_link( $attachment->ID , false );
share|improve this answer

Your Answer


By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.