Ticket #25443: 25443-custom-header.diff

wp-admin/custom-header.php

@@ -633 +633 @@
 </table>
 <?php endif;
 
+/**
+ * Fires at the end of the custom header options form
+ *
+ * @since 3.1.0
+ *
+ */
 do_action( 'custom_header_options' );
 
 wp_nonce_field( 'custom-header-options', '_wpnonce-custom-header-options' ); ?>
@@ -690 +696 @@
 
                         $this->set_header_image( compact( 'url', 'attachment_id', 'width', 'height' ) );
 
+                        /**
+                         * Fires just before step of of custom header is finished
+                         *
+                         * @since 2.1.0
+                         * @param string $file Path to the file
+                         * @param int $attachment_id ID of the attachment
+                         *
+                         */
                         do_action('wp_create_file_in_uploads', $file, $attachment_id); // For replication
+
                         return $this->finished();
                 } elseif ( $width > $max_width ) {
                         $oitar = $width / $max_width;
@@ -698 +713 @@
                         if ( ! $image || is_wp_error( $image ) )
                                 wp_die( __( 'Image could not be processed. Please go back and try again.' ), __( 'Image Processing Error' ) );
 
+                        /**
+                         * Fires before creating a replication of the custom header file
+                         *
+                         * @since 2.1.0
+                         * @param string $image Path to the file
+                         * @param int $attachment_id ID of the attachment
+                         *
+                         */
                         $image = apply_filters('wp_create_file_in_uploads', $image, $attachment_id); // For replication
 
                         $url = str_replace(basename($url), basename($image), $url);
@@ -839 +862 @@
                 if ( ! $cropped || is_wp_error( $cropped ) )
                         wp_die( __( 'Image could not be processed. Please go back and try again.' ), __( 'Image Processing Error' ) );
 
+                // duplicate hook
                 $cropped = apply_filters('wp_create_file_in_uploads', $cropped, $attachment_id); // For replication
 
                 $parent = get_post($attachment_id);
@@ -870 +894 @@
 
                 // cleanup
                 $medium = str_replace( basename( $original ), 'midsize-' . basename( $original ), $original );
-                if ( file_exists( $medium ) )
+                if ( file_exists( $medium ) ) {
+                        /**
+                         * Fires before deleting a file
+                         *
+                         * @since 2.1.0
+                         * @param string $medium Path to the file
+                         *
+                         */
                         @unlink( apply_filters( 'wp_delete_file', $medium ) );
-                if ( empty( $_POST['create-new-attachment'] ) && empty( $_POST['skip-cropping'] ) )
+                }
+
+                if ( empty( $_POST['create-new-attachment'] ) && empty( $_POST['skip-cropping'] ) ) {
+                        // duplicate hook
                         @unlink( apply_filters( 'wp_delete_file', $original ) );
+                }
 
                 return $this->finished();
         }