Ticket #21309: 21309.9.diff

File 21309.9.diff, 14.6 KB (added by ryan, 9 months ago)

Make sure get_default_post_to_edit() always returns WP_Post. Remove ref returns.

  • wp-includes/post-template.php

     
    101101 * @return string 
    102102 */ 
    103103function get_the_title( $id = 0 ) { 
    104         $post = &get_post($id); 
     104        $post = get_post($id); 
    105105 
    106106        $title = isset($post->post_title) ? $post->post_title : ''; 
    107107        $id = isset($post->ID) ? $post->ID : (int) $id; 
    … …  
    148148 * @return string 
    149149 */ 
    150150function get_the_guid( $id = 0 ) { 
    151         $post = &get_post($id); 
     151        $post = get_post($id); 
    152152 
    153153        return apply_filters('get_the_guid', $post->guid); 
    154154} 
    … …  
    276276 * @return bool 
    277277 */ 
    278278function has_excerpt( $id = 0 ) { 
    279         $post = &get_post( $id ); 
     279        $post = get_post( $id ); 
    280280        return ( !empty( $post->post_excerpt ) ); 
    281281} 
    282282 
    … …  
    11381138 */ 
    11391139function wp_get_attachment_link( $id = 0, $size = 'thumbnail', $permalink = false, $icon = false, $text = false ) { 
    11401140        $id = intval( $id ); 
    1141         $_post = & get_post( $id ); 
     1141        $_post = get_post( $id ); 
    11421142 
    11431143        if ( empty( $_post ) || ( 'attachment' != $_post->post_type ) || ! $url = wp_get_attachment_url( $_post->ID ) ) 
    11441144                return __( 'Missing Attachment' ); 

  • wp-includes/taxonomy.php

     
    30733073 * @return array 
    30743074 */ 
    30753075function get_the_taxonomies($post = 0, $args = array() ) { 
    3076         if ( is_int($post) ) 
    3077                 $post =& get_post($post); 
    3078         elseif ( !is_object($post) ) 
    3079                 $post =& $GLOBALS['post']; 
     3076        $post = get_post( $post ); 
    30803077 
    30813078        $args = wp_parse_args( $args, array( 
    30823079                'template' => '%s: %l.', 
    … …  
    31223119 * @return array 
    31233120 */ 
    31243121function get_post_taxonomies($post = 0) { 
    3125         $post =& get_post($post); 
     3122        $post = get_post( $post ); 
    31263123 
    31273124        return get_object_taxonomies($post); 
    31283125} 

  • wp-includes/post.php

     
    586586 * @return bool|string False on failure or returns the mime type 
    587587 */ 
    588588function get_post_mime_type($ID = '') { 
    589         $post = & get_post($ID); 
     589        $post = get_post($ID); 
    590590 
    591591        if ( is_object($post) ) 
    592592                return $post->post_mime_type; 
    … …  
    37163716        if (strpos($url, home_url('/?attachment_id=')) !== false) 
    37173717                return true; 
    37183718        if ( $id = url_to_postid($url) ) { 
    3719                 $post = & get_post($id); 
     3719                $post = get_post($id); 
    37203720                if ( 'attachment' == $post->post_type ) 
    37213721                        return true; 
    37223722        } 
    … …  
    40124012 */ 
    40134013function wp_get_attachment_metadata( $post_id = 0, $unfiltered = false ) { 
    40144014        $post_id = (int) $post_id; 
    4015         if ( !$post =& get_post( $post_id ) ) 
     4015        if ( !$post = get_post( $post_id ) ) 
    40164016                return false; 
    40174017 
    40184018        $data = get_post_meta( $post->ID, '_wp_attachment_metadata', true ); 
    … …  
    40344034 */ 
    40354035function wp_update_attachment_metadata( $post_id, $data ) { 
    40364036        $post_id = (int) $post_id; 
    4037         if ( !$post =& get_post( $post_id ) ) 
     4037        if ( !$post = get_post( $post_id ) ) 
    40384038                return false; 
    40394039 
    40404040        $data = apply_filters( 'wp_update_attachment_metadata', $data, $post->ID ); 
    … …  
    40524052 */ 
    40534053function wp_get_attachment_url( $post_id = 0 ) { 
    40544054        $post_id = (int) $post_id; 
    4055         if ( !$post =& get_post( $post_id ) ) 
     4055        if ( !$post = get_post( $post_id ) ) 
    40564056                return false; 
    40574057 
    40584058        if ( 'attachment' != $post->post_type ) 
    … …  
    40914091 */ 
    40924092function wp_get_attachment_thumb_file( $post_id = 0 ) { 
    40934093        $post_id = (int) $post_id; 
    4094         if ( !$post =& get_post( $post_id ) ) 
     4094        if ( !$post = get_post( $post_id ) ) 
    40954095                return false; 
    40964096        if ( !is_array( $imagedata = wp_get_attachment_metadata( $post->ID ) ) ) 
    40974097                return false; 
    … …  
    41134113 */ 
    41144114function wp_get_attachment_thumb_url( $post_id = 0 ) { 
    41154115        $post_id = (int) $post_id; 
    4116         if ( !$post =& get_post( $post_id ) ) 
     4116        if ( !$post = get_post( $post_id ) ) 
    41174117                return false; 
    41184118        if ( !$url = wp_get_attachment_url( $post->ID ) ) 
    41194119                return false; 
    … …  
    41404140 */ 
    41414141function wp_attachment_is_image( $post_id = 0 ) { 
    41424142        $post_id = (int) $post_id; 
    4143         if ( !$post =& get_post( $post_id ) ) 
     4143        if ( !$post = get_post( $post_id ) ) 
    41444144                return false; 
    41454145 
    41464146        if ( !$file = get_attached_file( $post->ID ) ) 
    … …  
    41714171                $post_mimes = array(); 
    41724172                if ( is_numeric($mime) ) { 
    41734173                        $mime = (int) $mime; 
    4174                         if ( $post =& get_post( $mime ) ) { 
     4174                        if ( $post = get_post( $mime ) ) { 
    41754175                                $post_id = (int) $post->ID; 
    41764176                                $ext = preg_replace('/^.+?\.([^.]+)$/', '$1', $post->guid); 
    41774177                                if ( !empty($ext) ) { 

  • wp-includes/comment.php

     
    13711371                if ( '0' == $commentdata['comment_approved'] ) 
    13721372                        wp_notify_moderator($comment_ID); 
    13731373 
    1374                 $post = &get_post($commentdata['comment_post_ID']); // Don't notify if it's your own comment 
     1374                $post = get_post($commentdata['comment_post_ID']); // Don't notify if it's your own comment 
    13751375 
    13761376                if ( get_option('comments_notify') && $commentdata['comment_approved'] && ( ! isset( $commentdata['user_id'] ) || $post->post_author != $commentdata['user_id'] ) ) 
    13771377                        wp_notify_postauthor($comment_ID, isset( $commentdata['comment_type'] ) ? $commentdata['comment_type'] : '' ); 

  • wp-includes/media.php

     
    655655                $hwstring = image_hwstring($width, $height); 
    656656                if ( is_array($size) ) 
    657657                        $size = join('x', $size); 
    658                 $attachment =& get_post($attachment_id); 
     658                $attachment = get_post($attachment_id); 
    659659                $default_attr = array( 
    660660                        'src'   => $src, 
    661661                        'class' => "attachment-$size", 

  • wp-includes/link-template.php

     
    9696                $post = $id; 
    9797                $sample = true; 
    9898        } else { 
    99                 $post = &get_post($id); 
     99                $post = get_post($id); 
    100100                $sample = false; 
    101101        } 
    102102 
    … …  
    178178function get_post_permalink( $id = 0, $leavename = false, $sample = false ) { 
    179179        global $wp_rewrite; 
    180180 
    181         $post = &get_post($id); 
     181        $post = get_post($id); 
    182182 
    183183        if ( is_wp_error( $post ) ) 
    184184                return $post; 
    … …  
    271271        if ( !$id ) 
    272272                $id = (int) $post->ID; 
    273273        else 
    274                 $post = &get_post($id); 
     274                $post = get_post($id); 
    275275 
    276276        $draft_or_pending = in_array( $post->post_status, array( 'draft', 'pending', 'auto-draft' ) ); 
    277277 
    … …  
    898898 * @return string 
    899899 */ 
    900900function get_edit_post_link( $id = 0, $context = 'display' ) { 
    901         if ( !$post = &get_post( $id ) ) 
     901        if ( !$post = get_post( $id ) ) 
    902902                return; 
    903903 
    904904        if ( 'display' == $context ) 
    … …  
    927927 * @param int $id Optional. Post ID. 
    928928 */ 
    929929function edit_post_link( $link = null, $before = '', $after = '', $id = 0 ) { 
    930         if ( !$post = &get_post( $id ) ) 
     930        if ( !$post = get_post( $id ) ) 
    931931                return; 
    932932 
    933933        if ( !$url = get_edit_post_link( $post->ID ) ) 
    … …  
    957957        if ( ! empty( $deprecated ) ) 
    958958                _deprecated_argument( __FUNCTION__, '3.0' ); 
    959959 
    960         if ( !$post = &get_post( $id ) ) 
     960        if ( !$post = get_post( $id ) ) 
    961961                return; 
    962962 
    963963        $post_type_object = get_post_type_object( $post->post_type ); 
    … …  
    12041204 */ 
    12051205function get_adjacent_post_rel_link($title = '%title', $in_same_cat = false, $excluded_categories = '', $previous = true) { 
    12061206        if ( $previous && is_attachment() && is_object( $GLOBALS['post'] ) ) 
    1207                 $post = & get_post($GLOBALS['post']->post_parent); 
     1207                $post = get_post($GLOBALS['post']->post_parent); 
    12081208        else 
    12091209                $post = get_adjacent_post($in_same_cat,$excluded_categories,$previous); 
    12101210 
    … …  
    13691369 */ 
    13701370function adjacent_post_link($format, $link, $in_same_cat = false, $excluded_categories = '', $previous = true) { 
    13711371        if ( $previous && is_attachment() ) 
    1372                 $post = & get_post($GLOBALS['post']->post_parent); 
     1372                $post = get_post($GLOBALS['post']->post_parent); 
    13731373        else 
    13741374                $post = get_adjacent_post($in_same_cat, $excluded_categories, $previous); 
    13751375 

  • wp-includes/general-template.php

     
    16271627 
    16281628        if ( is_single() || is_page() ) { 
    16291629                $id = 0; 
    1630                 $post = &get_post( $id ); 
     1630                $post = get_post( $id ); 
    16311631 
    16321632                if ( comments_open() || pings_open() || $post->comment_count > 0 ) { 
    16331633                        $title = sprintf( $args['singletitle'], get_bloginfo('name'), $args['separator'], esc_html( get_the_title() ) ); 

  • wp-includes/deprecated.php

     
    2626function get_postdata($postid) { 
    2727        _deprecated_function( __FUNCTION__, '1.5.1', 'get_post()' ); 
    2828 
    29         $post = &get_post($postid); 
     29        $post = get_post($postid); 
    3030 
    3131        $postdata = array ( 
    3232                'ID' => $post->ID, 
    … …  
    18921892function get_the_attachment_link($id = 0, $fullsize = false, $max_dims = false, $permalink = false) { 
    18931893        _deprecated_function( __FUNCTION__, '2.5', 'wp_get_attachment_link()' ); 
    18941894        $id = (int) $id; 
    1895         $_post = & get_post($id); 
     1895        $_post = get_post($id); 
    18961896 
    18971897        if ( ('attachment' != $_post->post_type) || !$url = wp_get_attachment_url($_post->ID) ) 
    18981898                return __('Missing Attachment'); 
    … …  
    19211921function get_attachment_icon_src( $id = 0, $fullsize = false ) { 
    19221922        _deprecated_function( __FUNCTION__, '2.5', 'wp_get_attachment_image_src()' ); 
    19231923        $id = (int) $id; 
    1924         if ( !$post = & get_post($id) ) 
     1924        if ( !$post = get_post($id) ) 
    19251925                return false; 
    19261926 
    19271927        $file = get_attached_file( $post->ID ); 
    … …  
    19661966function get_attachment_icon( $id = 0, $fullsize = false, $max_dims = false ) { 
    19671967        _deprecated_function( __FUNCTION__, '2.5', 'wp_get_attachment_image()' ); 
    19681968        $id = (int) $id; 
    1969         if ( !$post = & get_post($id) ) 
     1969        if ( !$post = get_post($id) ) 
    19701970                return false; 
    19711971 
    19721972        if ( !$src = get_attachment_icon_src( $post->ID, $fullsize ) ) 
    … …  
    20232023function get_attachment_innerHTML($id = 0, $fullsize = false, $max_dims = false) { 
    20242024        _deprecated_function( __FUNCTION__, '2.5', 'wp_get_attachment_image()' ); 
    20252025        $id = (int) $id; 
    2026         if ( !$post = & get_post($id) ) 
     2026        if ( !$post = get_post($id) ) 
    20272027                return false; 
    20282028 
    20292029        if ( $innerHTML = get_attachment_icon($post->ID, $fullsize, $max_dims)) 
    … …  
    28012801        _deprecated_function( __FUNCTION__, '3.3' ); 
    28022802 
    28032803        if ( ! empty( $GLOBALS['post'] ) && ! empty( $GLOBALS['post']->post_parent ) ) 
    2804                 $post = & get_post($GLOBALS['post']->post_parent); 
     2804                $post = get_post($GLOBALS['post']->post_parent); 
    28052805 
    28062806        if ( empty($post) ) 
    28072807                return; 

  • wp-admin/includes/post.php

     
    160160 
    161161        // Autosave shouldn't save too soon after a real save 
    162162        if ( 'autosave' == $post_data['action'] ) { 
    163                 $post =& get_post( $post_ID ); 
     163                $post = get_post( $post_ID ); 
    164164                $now = time(); 
    165165                $then = strtotime($post->post_date_gmt . ' +0000'); 
    166166                $delta = AUTOSAVE_INTERVAL / 2; 
    … …  
    439439                $post->page_template = 'default'; 
    440440                $post->post_parent = 0; 
    441441                $post->menu_order = 0; 
     442                $post = new WP_Post( $post ); 
    442443        } 
    443444 
    444445        $post->post_content = apply_filters( 'default_content', $post_content, $post ); 
    … …  
    745746 * @return unknown 
    746747 */ 
    747748function _fix_attachment_links( $post_ID ) { 
    748         $post = & get_post( $post_ID, ARRAY_A ); 
     749        $post = get_post( $post_ID, ARRAY_A ); 
    749750        $content = $post['post_content']; 
    750751 
    751752        // quick sanity check, don't run if no pretty permalinks or post is not published 
    … …  
    10121013 * @return array With two entries of type string 
    10131014 */ 
    10141015function get_sample_permalink($id, $title = null, $name = null) { 
    1015         $post = &get_post($id); 
     1016        $post = get_post($id); 
    10161017        if ( !$post->ID ) 
    10171018                return array('', ''); 
    10181019 
    … …  
    10781079 */ 
    10791080function get_sample_permalink_html( $id, $new_title = null, $new_slug = null ) { 
    10801081        global $wpdb; 
    1081         $post = &get_post($id); 
     1082        $post = get_post($id); 
    10821083 
    10831084        list($permalink, $post_name) = get_sample_permalink($post->ID, $new_title, $new_slug); 
    10841085 

  • wp-admin/includes/template.php

     
    705705 */ 
    706706function the_attachment_links( $id = false ) { 
    707707        $id = (int) $id; 
    708         $post = & get_post( $id ); 
     708        $post = get_post( $id ); 
    709709 
    710710        if ( $post->post_type != 'attachment' ) 
    711711                return false; 

  • wp-admin/includes/media.php

     
    893893 * @return unknown 
    894894 */ 
    895895function image_media_send_to_editor($html, $attachment_id, $attachment) { 
    896         $post =& get_post($attachment_id); 
     896        $post = get_post($attachment_id); 
    897897        if ( substr($post->post_mime_type, 0, 5) == 'image' ) { 
    898898                $url = $attachment['url']; 
    899899                $align = !empty($attachment['align']) ? $attachment['align'] : 'none'; 
    … …  
    920920 */ 
    921921function get_attachment_fields_to_edit($post, $errors = null) { 
    922922        if ( is_int($post) ) 
    923                 $post =& get_post($post); 
     923                $post = get_post($post); 
    924924        if ( is_array($post) ) 
    925                 $post = (object) $post; 
     925                $post = new WP_Post( (object) $post ); 
    926926 
    927927        $image_url = wp_get_attachment_url($post->ID); 
    928928 

  • wp-admin/post.php

     
    202202case 'trash': 
    203203        check_admin_referer('trash-post_' . $post_id); 
    204204 
    205         $post = & get_post($post_id); 
     205        $post = get_post($post_id); 
    206206 
    207207        if ( !current_user_can($post_type_object->cap->delete_post, $post_id) ) 
    208208                wp_die( __('You are not allowed to move this item to the Trash.') ); 

  • wp-admin/upload.php

     
    5757                        if ( !$parent_id ) 
    5858                                return; 
    5959 
    60                         $parent = &get_post( $parent_id ); 
     60                        $parent = get_post( $parent_id ); 
    6161                        if ( !current_user_can( 'edit_post', $parent_id ) ) 
    6262                                wp_die( __( 'You are not allowed to edit this post.' ) ); 
    6363 

  • wp-admin/edit.php

     
    103103                case 'delete': 
    104104                        $deleted = 0; 
    105105                        foreach( (array) $post_ids as $post_id ) { 
    106                                 $post_del = & get_post($post_id); 
     106                                $post_del = get_post($post_id); 
    107107 
    108108                                if ( !current_user_can($post_type_object->cap->delete_post, $post_id) ) 
    109109                                        wp_die( __('You are not allowed to delete this item.') );