Changeset 33278 for trunk/src/wp-includes/theme.php

Timestamp:

07/15/2015 04:27:17 PM (9 years ago)

Author:

obenland

Message:

Properly clean up header image data when the attachment is deleted.

Props CreativeInfusion.
Fixes #33005.

File:

• trunk/src/wp-includes/theme.php (modified) (1 diff)

trunk/src/wp-includes/theme.php

@@ -1932 +1932 @@
  *
  * If true it removes the theme modification which would be pointing at the deleted
- * attachment
+ * attachment.
  *
  * @access private
  * @since 3.0.0
- * @param int $id the attachment id
+ * @since 4.3.0 Also removes `header_image_data`.
+ *
+ * @param int $id The attachment id.
  */
 function _delete_attachment_theme_mod( $id ) {
     $attachment_image = wp_get_attachment_url( $id );
-    $header_image = get_header_image();
+    $header_image     = get_header_image();
     $background_image = get_background_image();
 
-    if ( $header_image && $header_image == $attachment_image )
+    if ( $header_image && $header_image == $attachment_image ) {
         remove_theme_mod( 'header_image' );
-
-    if ( $background_image && $background_image == $attachment_image )
+        remove_theme_mod( 'header_image_data' );
+    }
+
+    if ( $background_image && $background_image == $attachment_image ) {
         remove_theme_mod( 'background_image' );
+    }
 }