Ticket #13429: 13429-6.patch

wp-admin/includes/post.php

@@ -749 +749 @@
  * @return unknown
  */
 function _fix_attachment_links( $post_ID ) {
-        global $_fix_attachment_link_id;
-
         $post = & get_post( $post_ID, 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' )
+                return;
 
-        $search = "#<a[^>]+rel=('|\")[^'\"]*attachment[^>]*>#ie";
-
-        // See if we have any rel="attachment" links
-        if ( 0 == preg_match_all( $search, $post['post_content'], $anchor_matches, PREG_PATTERN_ORDER ) )
+        // Short if there aren't any links or no '?attachment_id=' strings (strpos cannot be zero)
+        if ( !strpos($content, '?attachment_id=') || !preg_match_all( '/<a ([^>]+)>[\s\S]+?<\/a>/', $content, $link_matches ) )
                 return;
 
-        $i = 0;
-        $search = "#[\s]+rel=(\"|')(.*?)wp-att-(\d+)\\1#i";
-        foreach ( $anchor_matches[0] as $anchor ) {
-                if ( 0 == preg_match( $search, $anchor, $id_matches ) )
-                        continue;
+        $site_url = get_bloginfo('url');
+        $site_url = substr( $site_url, (int) strpos($site_url, '://') ); // remove the http(s)
+        $replace = '';
 
-                $id = (int) $id_matches[3];
+        foreach ( $link_matches[1] as $key => $value ) {
+                if ( !strpos($value, '?attachment_id=') || !strpos($value, 'wp-att-')
+                        || !preg_match( '/href=(["\'])[^"\']*\?attachment_id=(\d+)[^"\']*\\1/', $value, $url_match )
+                        || !preg_match( '/rel=["\'][^"\']*wp-att-(\d+)/', $value, $rel_match ) )
+                                continue;
 
-                // While we have the attachment ID, let's adopt any orphans.
-                $attachment = & get_post( $id, ARRAY_A );
-                if ( ! empty( $attachment) && ! is_object( get_post( $attachment['post_parent'] ) ) ) {
-                        $attachment['post_parent'] = $post_ID;
-                        // Escape data pulled from DB.
-                        $attachment = add_magic_quotes( $attachment );
-                        wp_update_post( $attachment );
-                }
+                $quote = $url_match[1]; // the quote (single or double)
+                $url_id = (int) $url_match[2];
+                $rel_id = (int) $rel_match[1];
 
-                $post_search[$i] = $anchor;
-                 $_fix_attachment_link_id = $id;
-                $post_replace[$i] = preg_replace_callback( "#href=(\"|')[^'\"]*\\1#", '_fix_attachment_links_replace_cb', $anchor );
-                ++$i;
-        }
+                if ( !$url_id || !$rel_id || $url_id != $rel_id || strpos($url_match[0], $site_url) === false )
+                        continue;
 
-        $post['post_content'] = str_replace( $post_search, $post_replace, $post['post_content'] );
+                $link = $link_matches[0][$key];
+                $replace = str_replace( $url_match[0], 'href=' . $quote . get_attachment_link( $url_id ) . $quote, $link );
 
-        // Escape data pulled from DB.
-        $post = add_magic_quotes( $post);
+                $content = str_replace( $link, $replace, $content );
+        }
 
-        return wp_update_post( $post);
-}
+        if ( $replace ) {
+                $post['post_content'] = $content;
+                // Escape data pulled from DB.
+                $post = add_magic_quotes($post);
 
-function _fix_attachment_links_replace_cb($match) {
-        global $_fix_attachment_link_id;
-        return stripslashes( 'href='.$match[1] ).get_attachment_link( $_fix_attachment_link_id ).stripslashes( $match[1] );
+                return wp_update_post($post);
+        }
 }
 
 /**