Merge pull request #957 from cloudinary/fix/fragments-handling

Fix closing the filestream and deleting the fragment

Author: pereirinha

php/class-utils.php

@@ -670,7 +670,10 @@ public static function download_fragment( $url, $size = 1048576 ) {
 			if ( empty( $index ) ) {
 				add_action( 'shutdown', array( __CLASS__, 'purge_fragments' ) );
 			}
-			self::$file_fragments[ $index ] = $temp_file;
+			self::$file_fragments[ $index ] = array(
+				'pointer' => $pointer,
+				'file'    => $temp_file,
+			);
 			// Get the metadata of the stream.
 			$data = stream_get_meta_data( $pointer );
 			// Stream the content to the temp file.
@@ -710,8 +713,8 @@ public static function purge_fragments() {
 	 */
 	public static function purge_fragment( $index ) {
 		if ( isset( self::$file_fragments[ $index ] ) ) {
-			fclose( self::$file_fragments[ $index ] ); // phpcs:ignore WordPress.WP.AlternativeFunctions.file_system_read_fclose
-			unset( self::$file_fragments[ $index ] );
+			fclose( self::$file_fragments[ $index ]['pointer'] ); // phpcs:ignore WordPress.WP.AlternativeFunctions.file_system_read_fclose
+			unlink( self::$file_fragments[ $index ]['file'] );
 		}
 	}