Make WordPress Core

Ticket #32848: 32848.2.diff

File 32848.2.diff, 5.1 KB (added by MikeNGarrett, 9 years ago)

Patch 2 - replace all isset in option.php

  • src/wp-includes/option.php

     
    5454        if ( ! defined( 'WP_INSTALLING' ) ) {
    5555                // prevent non-existent options from triggering multiple queries
    5656                $notoptions = wp_cache_get( 'notoptions', 'options' );
    57                 if ( isset( $notoptions[ $option ] ) ) {
     57                if ( array_key_exists( $option, $notoptions ) ) {
    5858                        /**
    5959                         * Filter the default value for an option.
    6060                         *
     
    7070
    7171                $alloptions = wp_load_alloptions();
    7272
    73                 if ( isset( $alloptions[$option] ) ) {
     73                if ( array_key_exists( $option, $alloptions ) ) {
    7474                        $value = $alloptions[$option];
    7575                } else {
    7676                        $value = wp_cache_get( $option, 'options' );
     
    319319                return false;
    320320
    321321        $notoptions = wp_cache_get( 'notoptions', 'options' );
    322         if ( is_array( $notoptions ) && isset( $notoptions[$option] ) ) {
     322        if ( is_array( $notoptions ) && array_key_exists( $option, $notoptions ) ) {
    323323                unset( $notoptions[$option] );
    324324                wp_cache_set( 'notoptions', $notoptions, 'options' );
    325325        }
    326326
    327327        if ( ! defined( 'WP_INSTALLING' ) ) {
    328328                $alloptions = wp_load_alloptions();
    329                 if ( isset( $alloptions[$option] ) ) {
     329                if ( array_key_exists( $option, $alloptions ) ) {
    330330                        $alloptions[ $option ] = $serialized_value;
    331331                        wp_cache_set( 'alloptions', $alloptions, 'options' );
    332332                } else {
     
    401401
    402402        // Make sure the option doesn't already exist. We can check the 'notoptions' cache before we ask for a db query
    403403        $notoptions = wp_cache_get( 'notoptions', 'options' );
    404         if ( !is_array( $notoptions ) || !isset( $notoptions[$option] ) )
     404        if ( !is_array( $notoptions ) || !array_key_exists( $option, $notoptions ) )
    405405                /** This filter is documented in wp-includes/option.php */
    406406                if ( apply_filters( 'default_option_' . $option, false ) !== get_option( $option ) )
    407407                        return false;
     
    435435
    436436        // This option exists now
    437437        $notoptions = wp_cache_get( 'notoptions', 'options' ); // yes, again... we need it to be fresh
    438         if ( is_array( $notoptions ) && isset( $notoptions[$option] ) ) {
     438        if ( is_array( $notoptions ) && array_key_exists( $option, $notoptions ) ) {
    439439                unset( $notoptions[$option] );
    440440                wp_cache_set( 'notoptions', $notoptions, 'options' );
    441441        }
     
    502502        if ( ! defined( 'WP_INSTALLING' ) ) {
    503503                if ( 'yes' == $row->autoload ) {
    504504                        $alloptions = wp_load_alloptions();
    505                         if ( is_array( $alloptions ) && isset( $alloptions[$option] ) ) {
     505                        if ( is_array( $alloptions ) && array_key_exists( $option, $alloptions ) ) {
    506506                                unset( $alloptions[$option] );
    507507                                wp_cache_set( 'alloptions', $alloptions, 'options' );
    508508                        }
     
    620620                if ( ! defined( 'WP_INSTALLING' ) ) {
    621621                        // If option is not in alloptions, it is not autoloaded and thus has a timeout
    622622                        $alloptions = wp_load_alloptions();
    623                         if ( !isset( $alloptions[$transient_option] ) ) {
     623                        if ( !array_key_exists( $transient_option, $alloptions ) ) {
    624624                                $transient_timeout = '_transient_timeout_' . $transient;
    625625                                if ( get_option( $transient_timeout ) < time() ) {
    626626                                        delete_option( $transient_option  );
     
    800800function get_user_setting( $name, $default = false ) {
    801801        $all_user_settings = get_all_user_settings();
    802802
    803         return isset( $all_user_settings[$name] ) ? $all_user_settings[$name] : $default;
     803        return array_key_exists( $name, $all_user_settings ) ) ? $all_user_settings[$name] : $default;
    804804}
    805805
    806806/**
     
    847847        $deleted = false;
    848848
    849849        foreach ( $names as $name ) {
    850                 if ( isset( $all_user_settings[$name] ) ) {
     850                if ( array_key_exists( $name, $all_user_settings ) ) {
    851851                        unset( $all_user_settings[$name] );
    852852                        $deleted = true;
    853853                }
     
    993993        $notoptions_key = "{$wpdb->siteid}:notoptions";
    994994        $notoptions = wp_cache_get( $notoptions_key, 'site-options' );
    995995
    996         if ( isset( $notoptions[$option] ) ) {
     996        if ( array_key_exists( $option, $notoptions ) ) {
    997997
    998998                /**
    999999                 * Filter a specific default site option.
     
    10931093
    10941094                // Make sure the option doesn't already exist. We can check the 'notoptions' cache before we ask for a db query
    10951095                $notoptions = wp_cache_get( $notoptions_key, 'site-options' );
    1096                 if ( ! is_array( $notoptions ) || ! isset( $notoptions[$option] ) )
     1096                if ( ! is_array( $notoptions ) || ! array_key_exists( $option, $notoptions ) )
    10971097                        if ( false !== get_site_option( $option ) )
    10981098                                return false;
    10991099
     
    11091109
    11101110                // This option exists now
    11111111                $notoptions = wp_cache_get( $notoptions_key, 'site-options' ); // yes, again... we need it to be fresh
    1112                 if ( is_array( $notoptions ) && isset( $notoptions[$option] ) ) {
     1112                if ( is_array( $notoptions ) && array_key_exists( $option, $notoptions ) ) {
    11131113                        unset( $notoptions[$option] );
    11141114                        wp_cache_set( $notoptions_key, $notoptions, 'site-options' );
    11151115                }
     
    12521252
    12531253        $notoptions_key = "{$wpdb->siteid}:notoptions";
    12541254        $notoptions = wp_cache_get( $notoptions_key, 'site-options' );
    1255         if ( is_array( $notoptions ) && isset( $notoptions[$option] ) ) {
     1255        if ( is_array( $notoptions ) && array_key_exists( $option, $notoptions ) ) {
    12561256                unset( $notoptions[$option] );
    12571257                wp_cache_set( $notoptions_key, $notoptions, 'site-options' );
    12581258        }