Changeset 38232 for trunk/src/wp-includes/ms-blogs.php

Timestamp:

08/09/2016 06:11:12 PM (8 years ago)

Author:

jeremyfelt

Message:

Multisite: Remove unnecessary reference parameters.

There is no advantage to passing parameters by reference to get_site(), get_network(), and update_site_cache().

Props flixos90, ocean90 for review.
See #37615.

File:

• trunk/src/wp-includes/ms-blogs.php (modified) (3 diffs)

trunk/src/wp-includes/ms-blogs.php

@@ -482 +482 @@
  * @return WP_Site|null The site object or null if not found.
  */
-function get_site( &$site = null ) {
+function get_site( $site = null ) {
     if ( empty( $site ) ) {
         $site = get_current_blog_id();
@@ -538 +538 @@
  * @since 4.6.0
  *
- * @param array $sites Array of site objects, passed by reference.
- */
-function update_site_cache( &$sites ) {
+ * @param array $sites Array of site objects.
+ */
+function update_site_cache( $sites ) {
     if ( ! $sites ) {
         return;
@@ -1086 +1086 @@
  * @global WP_Network $current_site
  *
- * @param WP_Network|int|null $network Network to retrieve, passed by reference.
+ * @param WP_Network|int|null $network Optional. Network to retrieve. Default is the current network.
  * @return WP_Network|null The network object or null if not found.
  */
-function get_network( &$network = null ) {
+function get_network( $network = null ) {
     global $current_site;
     if ( empty( $network ) && isset( $current_site ) ) {