Changeset 34685 for trunk/src/wp-includes/post-functions.php

Timestamp:

09/29/2015 04:57:02 AM (11 years ago)

Author:

pento

Message:

Rewrite: Redirect attachment URLs when their slug changes.

Using the same logic that we use to redirect posts when their slug changes, we can provide the same functionality for attachments. Attachment pages are posts, too.

Props swissspdy.

Fixes #34043.

File:

• trunk/src/wp-includes/post-functions.php (modified) (3 diffs)

trunk/src/wp-includes/post-functions.php

@@ -1331 +1331 @@
  *              and `use_featured_image` labels.
  * @since 4.4.0 Added the `insert_into_item` and `uploaded_to_this_item` labels.
- * 
+ *
  * @access private
  *
@@ -3281 +3281 @@
                          */
                         do_action( 'edit_attachment', $post_ID );
+                        $post_after = get_post( $post_ID );
+
+                        /**
+                         * Fires once an existing attachment has been updated.
+                         *
+                         * @since 4.4.0
+                         *
+                         * @param int     $post_ID      Post ID.
+                         * @param WP_Post $post_after   Post object following the update.
+                         * @param WP_Post $post_before  Post object before the update.
+                         */
+                        do_action( 'attachment_updated', $post_ID, $post_after, $post_before );
                 } else {
 
@@ -5134 +5146 @@
  */
 function wp_check_for_changed_slugs( $post_id, $post, $post_before ) {
-        // Don't bother if it hasnt changed.
-        if ( $post->post_name == $post_before->post_name )
+        // Don't bother if it hasn't changed.
+        if ( $post->post_name == $post_before->post_name ) {
                 return;
+        }
 
         // We're only concerned with published, non-hierarchical objects.
-        if ( $post->post_status != 'publish' || is_post_type_hierarchical( $post->post_type ) )
+        if ( ! ( 'publish' === $post->post_status || ( 'attachment' === get_post_type( $post ) && 'inherit' === $post->post_status ) ) || is_post_type_hierarchical( $post->post_type ) ) {
                 return;
-
-        $old_slugs = (array) get_post_meta($post_id, '_wp_old_slug');
+        }
+
+        $old_slugs = (array) get_post_meta( $post_id, '_wp_old_slug' );
 
         // If we haven't added this old slug before, add it now.
-        if ( !empty( $post_before->post_name ) && !in_array($post_before->post_name, $old_slugs) )
-                add_post_meta($post_id, '_wp_old_slug', $post_before->post_name);
+        if ( ! empty( $post_before->post_name ) && ! in_array( $post_before->post_name, $old_slugs ) ) {
+                add_post_meta( $post_id, '_wp_old_slug', $post_before->post_name );
+        }
 
         // If the new slug was used previously, delete it from the list.
-        if ( in_array($post->post_name, $old_slugs) )
-                delete_post_meta($post_id, '_wp_old_slug', $post->post_name);
+        if ( in_array( $post->post_name, $old_slugs ) ) {
+                delete_post_meta( $post_id, '_wp_old_slug', $post->post_name );
+        }
 }