Changeset 25697

Timestamp:

10/06/2013 12:55:38 PM (11 years ago)

Author:

nacin

Message:

Run _fix_attachment_links() when post_status is future or private in addition to publish.

Fixes regression caused by [20308].

props jond3r.
fixes #25268.

File:

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

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

@@ -749 +749 @@
  * @access private
  *
- * @param unknown_type $post_ID
- * @return unknown
- */
-function _fix_attachment_links( $post_ID ) {
-    $post = get_post( $post_ID, ARRAY_A );
+ * @param int|object $post Post ID or post object.
+ * @return void|int|WP_Error Void if nothing fixed. 0 or WP_Error on update failure. The post ID on update success.
+ */
+function _fix_attachment_links( $post ) {
+    $post = get_post( $post, ARRAY_A );
     $content = $post['post_content'];
 
-    // quick sanity check, don't run if no pretty permalinks or post is not published
-    if ( !get_option('permalink_structure') || $post['post_status'] != 'publish' )
+    // Don't run if no pretty permalinks or post is not published, scheduled, or privately published.
+    if ( ! get_option( 'permalink_structure' ) || ! in_array( $post['post_status'], array( 'publish', 'future', 'private' ) ) )
         return;