_s: Simplifies the attached image template tag by only querying for ids.

This commit is contained in:
obenland 2013-07-02 17:41:51 -04:00
parent d722bf10ef
commit 77b0fa51ae
1 changed files with 16 additions and 13 deletions

View File

@ -119,35 +119,38 @@ function _s_the_attached_image() {
$next_attachment_url = wp_get_attachment_url(); $next_attachment_url = wp_get_attachment_url();
/** /**
* Grab the IDs of all the image attachments in a gallery so we can get the URL * Grab the IDs of all the image attachments in a gallery so we can get the
* of the next adjacent image in a gallery, or the first image (if we're * URL of the next adjacent image in a gallery, or the first image (if
* looking at the last image in a gallery), or, in a gallery of one, just the * we're looking at the last image in a gallery), or, in a gallery of one,
* link to that image file. * just the link to that image file.
*/ */
$attachments = array_values( get_children( array( $attachment_ids = get_posts( array(
'post_parent' => $post->post_parent, 'post_parent' => $post->post_parent,
'fields' => 'ids',
'numberposts' => -1,
'post_status' => 'inherit', 'post_status' => 'inherit',
'post_type' => 'attachment', 'post_type' => 'attachment',
'post_mime_type' => 'image', 'post_mime_type' => 'image',
'order' => 'ASC', 'order' => 'ASC',
'orderby' => 'menu_order ID' 'orderby' => 'menu_order ID'
) ) ); ) );
// If there is more than 1 attachment in a gallery... // If there is more than 1 attachment in a gallery...
if ( count( $attachments ) > 1 ) { if ( count( $attachment_ids ) > 1 ) {
foreach ( $attachments as $k => $attachment ) { foreach ( $attachment_ids as $attachment_id ) {
if ( $attachment->ID == $post->ID ) if ( $attachment_id == $post->ID ) {
$next_id = current( $attachment_ids );
break; break;
} }
$k++; }
// get the URL of the next image attachment... // get the URL of the next image attachment...
if ( isset( $attachments[ $k ] ) ) if ( $next_id )
$next_attachment_url = get_attachment_link( $attachments[ $k ]->ID ); $next_attachment_url = get_attachment_link( $next_id );
// or get the URL of the first image attachment. // or get the URL of the first image attachment.
else else
$next_attachment_url = get_attachment_link( $attachments[0]->ID ); $next_attachment_url = get_attachment_link( array_shift( $attachment_ids ) );
} }
printf( '<a href="%1$s" title="%2$s" rel="attachment">%3$s</a>', printf( '<a href="%1$s" title="%2$s" rel="attachment">%3$s</a>',