WordPress.org

Make WordPress Core

Ticket #16661: 16661.2.patch

File 16661.2.patch, 8.8 KB (added by hakre, 10 years ago)

review of do_action_ref_array() calls

  • wp-admin/includes/deprecated.php

     
    502502                        $this->query_where .= " AND $wpdb->users.ID = $wpdb->usermeta.user_id AND meta_key = '{$level_key}'";
    503503                }
    504504
    505                 do_action_ref_array( 'pre_user_search', array( &$this ) );
     505                do_action( 'pre_user_search', $this );
    506506        }
    507507
    508508        /**
  • wp-admin/includes/user.php

     
    176176        }
    177177
    178178        // Allow plugins to return their own errors.
    179         do_action_ref_array('user_profile_update_errors', array ( &$errors, $update, &$user ) );
     179        do_action('user_profile_update_errors', $errors, $update, $user );
    180180
    181181        if ( $errors->get_error_codes() )
    182182                return $errors;
  • wp-includes/admin-bar.php

     
    5757
    5858        $wp_admin_bar->load_user_locale_translations();
    5959
    60         do_action_ref_array( 'admin_bar_menu', array( &$wp_admin_bar ) );
     60        do_action( 'admin_bar_menu', $wp_admin_bar );
    6161
    6262        do_action( 'wp_before_admin_bar_render' );
    6363
  • wp-includes/class-wp.php

     
    298298
    299299                $this->query_vars = apply_filters('request', $this->query_vars);
    300300
    301                 do_action_ref_array('parse_request', array(&$this));
     301                do_action( 'parse_request', $this );
    302302        }
    303303
    304304        /**
     
    373373                if ( $exit_required )
    374374                        exit();
    375375
    376                 do_action_ref_array('send_headers', array(&$this));
     376                do_action( 'send_headers', $this );
    377377        }
    378378
    379379        /**
     
    501501                $this->query_posts();
    502502                $this->handle_404();
    503503                $this->register_globals();
    504                 do_action_ref_array('wp', array(&$this));
     504                do_action( 'wp', $this );
    505505        }
    506506
    507507        /**
  • wp-includes/class.wp-scripts.php

     
    3131        var $default_dirs;
    3232
    3333        function __construct() {
    34                 do_action_ref_array( 'wp_default_scripts', array(&$this) );
     34                do_action( 'wp_default_scripts', $this );
    3535        }
    3636
    3737        /**
  • wp-includes/class.wp-styles.php

     
    2828        var $default_dirs;
    2929
    3030        function __construct() {
    31                 do_action_ref_array( 'wp_default_styles', array(&$this) );
     31                do_action( 'wp_default_styles', $this );
    3232        }
    3333
    3434        function do_item( $handle ) {
  • wp-includes/comment.php

     
    221221                );
    222222
    223223                $this->query_vars = wp_parse_args( $query_vars, $defaults );
    224                 do_action_ref_array( 'pre_get_comments', array( &$this ) );
     224                do_action( 'pre_get_comments', $this );
    225225                extract( $this->query_vars, EXTR_SKIP );
    226226
    227227                // $args can be whatever, only use the args defined in defaults to compute the key
  • wp-includes/feed.php

     
    535535        $feed->set_cache_class('WP_Feed_Cache');
    536536        $feed->set_file_class('WP_SimplePie_File');
    537537        $feed->set_cache_duration(apply_filters('wp_feed_cache_transient_lifetime', 43200, $url));
    538         do_action_ref_array( 'wp_feed_options', array( &$feed, $url ) );
     538        do_action( 'wp_feed_options', $feed, $url );
    539539        $feed->init();
    540540        $feed->handle_content_type();
    541541
  • wp-includes/pluggable.php

     
    256256 * @uses apply_filters() Calls 'wp_mail_from_name' hook to get the from address name.
    257257 * @uses apply_filters() Calls 'wp_mail_content_type' hook to get the email content type.
    258258 * @uses apply_filters() Calls 'wp_mail_charset' hook to get the email charset
    259  * @uses do_action_ref_array() Calls 'phpmailer_init' hook on the reference to
     259 * @uses do_action() Calls 'phpmailer_init' hook on the reference to
    260260 *              phpmailer object.
    261261 * @uses PHPMailer
    262262 * @
     
    459459                }
    460460        }
    461461
    462         do_action_ref_array( 'phpmailer_init', array( &$phpmailer ) );
     462        do_action( 'phpmailer_init', $phpmailer );
    463463
    464464        // Send!
    465465        $result = @$phpmailer->Send();
  • wp-includes/query.php

     
    16271627                if ( '404' == $qv['error'] )
    16281628                        $this->set_404();
    16291629
    1630                 do_action_ref_array('parse_query', array(&$this));
     1630                do_action( 'parse_query', $this );
    16311631        }
    16321632
    16331633        /*
     
    18681868         *
    18691869         * @since 1.5.0
    18701870         * @access public
    1871          * @uses do_action_ref_array() Calls 'pre_get_posts' hook before retrieving posts.
     1871         * @uses do_action() Calls 'pre_get_posts' hook before retrieving posts.
    18721872         *
    18731873         * @return array List of posts.
    18741874         */
     
    18771877
    18781878                $this->parse_query();
    18791879
    1880                 do_action_ref_array('pre_get_posts', array(&$this));
     1880                do_action( 'pre_get_posts', $this );
    18811881
    18821882                // Shorthand.
    18831883                $q = &$this->query_vars;
     
    27322732         * @since 1.5.0
    27332733         * @access public
    27342734         * @uses $post
    2735          * @uses do_action_ref_array() Calls 'loop_start' if loop has just started
     2735         * @uses do_action() Calls 'loop_start' if loop has just started
    27362736         */
    27372737        function the_post() {
    27382738                global $post;
    27392739                $this->in_the_loop = true;
    27402740
    27412741                if ( $this->current_post == -1 ) // loop has just started
    2742                         do_action_ref_array('loop_start', array(&$this));
     2742                        do_action( 'loop_start', $this );
    27432743
    27442744                $post = $this->next_post();
    27452745                setup_postdata($post);
     
    27522752         *
    27532753         * @since 1.5.0
    27542754         * @access public
    2755          * @uses do_action_ref_array() Calls 'loop_end' if loop is ended
     2755         * @uses do_action() Calls 'loop_end' if loop is ended
    27562756         *
    27572757         * @return bool True if posts are available, false if end of loop.
    27582758         */
     
    27602760                if ( $this->current_post + 1 < $this->post_count ) {
    27612761                        return true;
    27622762                } elseif ( $this->current_post + 1 == $this->post_count && $this->post_count > 0 ) {
    2763                         do_action_ref_array('loop_end', array(&$this));
     2763                        do_action( 'loop_end', $this );
    27642764                        // Do some cleaning up after the loop
    27652765                        $this->rewind_posts();
    27662766                }
     
    34853485 * @since 1.5.0
    34863486 *
    34873487 * @param object $post Post data.
    3488  * @uses do_action_ref_array() Calls 'the_post'
     3488 * @uses do_action() Calls 'the_post'
    34893489 * @return bool True when finished.
    34903490 */
    34913491function setup_postdata($post) {
     
    35183518                $multipage = 0;
    35193519        }
    35203520
    3521         do_action_ref_array('the_post', array(&$post));
     3521        do_action( 'the_post', $post );
    35223522
    35233523        return true;
    35243524}
  • wp-includes/rewrite.php

     
    15621562                else
    15631563                        $this->rules = array_merge($this->extra_rules_top, $robots_rewrite, $default_feeds, $registration_pages, $root_rewrite, $comments_rewrite, $search_rewrite,  $author_rewrite, $date_rewrite, $post_rewrite, $page_rewrite, $this->extra_rules);
    15641564
    1565                 do_action_ref_array('generate_rewrite_rules', array(&$this));
     1565                do_action( 'generate_rewrite_rules', $this );
    15661566                $this->rules = apply_filters('rewrite_rules_array', $this->rules);
    15671567
    15681568                return $this->rules;
  • wp-includes/user.php

     
    531531                        $this->query_where .= " AND $wpdb->users.ID NOT IN ($ids)";
    532532                }
    533533
    534                 do_action_ref_array( 'pre_user_query', array( &$this ) );
     534                do_action( 'pre_user_query', $this );
    535535        }
    536536
    537537        /**
  • wp-includes/widgets.php

     
    270270                        $return = $this->form($instance);
    271271                        // add extra fields in the widget form - be sure to set $return to null if you add any
    272272                        // if the widget has no form the text echoed from the default form method can be hidden using css
    273                         do_action_ref_array( 'in_widget_form', array(&$this, &$return, $instance) );
     273                        do_action_ref_array( 'in_widget_form', array($this, &$return, $instance) );
    274274                }
    275275                return $return;
    276276        }