Changeset 38113 for trunk/tests/phpunit/tests/ajax/MediaEdit.php

Timestamp:

07/20/2016 07:34:54 AM (8 years ago)

Author:

mikeschroder

Message:

Media: Clean up prior image edits if IMAGE_EDIT_OVERWRITE is true.

When IMAGE_EDIT_OVERWRITE is set to true, edited image files are
supposed to be deleted when an image is restored to the original.

However, when an image was edited more than once, and then restored,
files created during previous edits were left behind.

Fixes this behavior by updating wp_save_image() to clean up
leftover images after each edit when IMAGE_EDIT_OVERWRITE is true.

Props bradt, chriscct7, joemcgill.
Fixes #32171.

File:

• trunk/tests/phpunit/tests/ajax/MediaEdit.php (modified) (1 diff)

trunk/tests/phpunit/tests/ajax/MediaEdit.php

@@ -52 +52 @@
         $this->assertArrayHasKey('medium', $media_meta['sizes'], 'cropped attachment should have data for medium size');
     }
+
+    /**
+     * @ticket 32171
+     */
+    public function testImageEditOverwriteConstant() {
+        define( 'IMAGE_EDIT_OVERWRITE', true );
+
+        include_once( ABSPATH . 'wp-admin/includes/image-edit.php' );
+
+        $filename = DIR_TESTDATA . '/images/canola.jpg';
+        $contents = file_get_contents( $filename );
+
+        $upload = wp_upload_bits( basename( $filename ), null, $contents );
+        $id = $this->_make_attachment( $upload );
+
+        $_REQUEST['action'] = 'image-editor';
+        $_REQUEST['context'] = 'edit-attachment';
+        $_REQUEST['postid'] = $id;
+        $_REQUEST['target'] = 'all';
+        $_REQUEST['do'] = 'save';
+        $_REQUEST['history'] = '[{"c":{"x":5,"y":8,"w":289,"h":322}}]';
+
+        $ret = wp_save_image( $id );
+
+        $media_meta = wp_get_attachment_metadata( $id );
+        $sizes1 = $media_meta['sizes'];
+
+        $_REQUEST['history'] = '[{"c":{"x":5,"y":8,"w":189,"h":322}}]';
+
+        $ret = wp_save_image( $id );
+
+        $media_meta = wp_get_attachment_metadata( $id );
+        $sizes2 = $media_meta['sizes'];
+
+        $file_path = dirname( get_attached_file( $id ) );
+
+        foreach ( $sizes1 as $key => $size ) {
+            if ( $sizes2[ $key ]['file'] !== $size['file'] ) {
+                $files_that_shouldnt_exist[] = $file_path . '/' . $size['file'];
+            }
+        }
+
+        foreach ( $files_that_shouldnt_exist as $file ) {
+            $this->assertFalse( file_exists( $file ), 'IMAGE_EDIT_OVERWRITE is leaving garbage image files behind.' );
+        }
+    }
 }