Ticket #12799: 12799-2.diff

src/wp-includes/media.php

@@ -1393 +1393 @@
  *     @type string       $ids        A comma-separated list of IDs of attachments to display. Default empty.
  *     @type string       $include    A comma-separated list of IDs of attachments to include. Default empty.
  *     @type string       $exclude    A comma-separated list of IDs of attachments to exclude. Default empty.
+ *     @type string       $maxdisp    Sets a maxiumum list of attachments to display.
  *     @type string       $link       What to link each image to. Default empty (links to the attachment page).
  *                                    Accepts 'file', 'none'.
  * }
@@ -1444 +1445 @@
                 'size'       => 'thumbnail',
                 'include'    => '',
                 'exclude'    => '',
+                'maxdisp'    => 0,
                 'link'       => ''
         ), $attr, 'gallery' );
 
         $id = intval( $atts['id'] );
 
+        $maxdisp = intval( $atts['maxdisp'] );
+        if ( $maxdisp <= 0 )
+                $maxdisp = -1; 
+
         if ( ! empty( $atts['include'] ) ) {
-                $_attachments = get_posts( array( 'include' => $atts['include'], 'post_status' => 'inherit', 'post_type' => 'attachment', 'post_mime_type' => 'image', 'order' => $atts['order'], 'orderby' => $atts['orderby'] ) );
+                $_attachments = get_posts( array( 'include' => $atts['include'], 'numberposts' => $maxdisp, 'post_status' => 'inherit', 'post_type' => 'attachment', 'post_mime_type' => 'image', 'order' => $atts['order'], 'orderby' => $atts['orderby'] ) );
 
                 $attachments = array();
                 foreach ( $_attachments as $key => $val ) {
@@ -1457 +1463 @@
                         $attachments[$val->ID] = $_attachments[$key];
                 }
         } elseif ( ! empty( $atts['exclude'] ) ) {
-                $attachments = get_children( array( 'post_parent' => $id, 'exclude' => $atts['exclude'], 'post_status' => 'inherit', 'post_type' => 'attachment', 'post_mime_type' => 'image', 'order' => $atts['order'], 'orderby' => $atts['orderby'] ) );
+                $attachments = get_children( array( 'numberposts' => $maxdisp, 'post_parent' => $id, 'exclude' => $atts['exclude'], 'post_status' => 'inherit', 'post_type' => 'attachment', 'post_mime_type' => 'image', 'order' => $atts['order'], 'orderby' => $atts['orderby'] ) );
         } else {
-                $attachments = get_children( array( 'post_parent' => $id, 'post_status' => 'inherit', 'post_type' => 'attachment', 'post_mime_type' => 'image', 'order' => $atts['order'], 'orderby' => $atts['orderby'] ) );
+                $attachments = get_children( array( 'numberposts' => $maxdisp, 'post_parent' => $id, 'post_status' => 'inherit', 'post_type' => 'attachment', 'post_mime_type' => 'image', 'order' => $atts['order'], 'orderby' => $atts['orderby'] ) );
         }
 
         if ( empty( $attachments ) ) {
                 return '';
+        
         }
+        
+         // Necessary because in get_posts(), when specifying include="", the numberposts parameter is ignored (it's set to match the number if items in include="")    
+        if ( $maxdisp > 0 )
+        $attachments = array_slice( $attachments, 0, $maxdisp, true );
 
         if ( is_feed() ) {
                 $output = "\n";