WordPress.org

Make WordPress Core


Ignore:
Timestamp:
03/28/2017 02:31:13 AM (3 years ago)
Author:
jeremyfelt
Message:

Multisite: Allow falsy properties to be cached in site-details.

In previous iterations of WP_Site, there was concern that not all properties would be available when storing a site's details in cache. When introduced in [37918], an ms_loaded check was added to address this concern. Any properties that are still false after ms_loaded really are false and can be cached as such.

Props flixos90.
Fixes #40247.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/class-wp-site.php

    r38936 r40344  
    334334            restore_current_blog();
    335335
    336             $cache_details = true;
    337             foreach ( array( 'blogname', 'siteurl', 'post_count', 'home' ) as $field ) {
    338                 if ( false === $details->$field ) {
    339                     $cache_details = false;
    340                     break;
    341                 }
    342             }
    343 
    344             if ( $cache_details ) {
    345                 wp_cache_set( $this->blog_id, $details, 'site-details' );
    346             }
     336            wp_cache_set( $this->blog_id, $details, 'site-details' );
    347337        }
    348338
Note: See TracChangeset for help on using the changeset viewer.