Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
07/23/2024 01:13:42 AM (2 years ago)
Author:
espellcaste
Message:

WPCS: Part XII: miscellaneous fixes for some of the files of the core component.

Follow-up to [13904]

See #9174 and #9173

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-core/bp-core-wpabstraction.php

    r13395 r13968  
    1010 *
    1111 * @package BuddyPress
    12  * @subpackage WPAbstraction
     12 * @subpackage Core
    1313 * @since 1.2.0
    1414 */
    … …  
    2424 * @global string $wp_version WP Version number.
    2525 *
    26  * @return double
     26 * @return float
    2727 */
    2828function bp_get_major_wp_version() {
    … …  
    3535 * Only add MS-specific abstraction functions if WordPress is not in multisite mode.
    3636 */
    37 if ( !is_multisite() ) {
     37if ( ! is_multisite() ) {
    3838        global $wpdb;
    3939
    … …  
    4141        $wpdb->blogid      = BP_ROOT_BLOG;
    4242
    43         if ( !function_exists( 'get_blog_option' ) ) {
     43        if ( ! function_exists( 'get_blog_option' ) ) {
    4444
    4545                /**
    … …  
    5252                 * @param int    $blog_id     Blog ID to fetch for. Not used.
    5353                 * @param string $option_name Option name to fetch.
    54                  * @param bool   $default     Whether or not default.
     54                 * @param bool   $default     Optional. Default value to return if the option does not exist.
    5555                 * @return mixed
    5656                 */
    … …  
    7272                 * @param string $option_name  Option name to add.
    7373                 * @param mixed  $option_value Option value to add.
    74                  * @return mixed
     74                 * @return bool
    7575                 */
    7676                function add_blog_option( $blog_id, $option_name, $option_value ) {
    … …  
    7979        }
    8080
    81         if ( !function_exists( 'update_blog_option' ) ) {
     81        if ( ! function_exists( 'update_blog_option' ) ) {
    8282
    8383                /**
    … …  
    8888                 * @see update_blog_option()
    8989                 *
    90                  * @param int    $blog_id     Blog ID to update for. Not used.
    91                  * @param string $option_name Option name to update.
    92                  * @param mixed  $value      Option value to update.
    93                  * @return mixed
    94                  */
    95                 function update_blog_option( $blog_id, $option_name, $value ) {
    96                         return update_option( $option_name, $value );
    97                 }
    98         }
    99 
    100         if ( !function_exists( 'delete_blog_option' ) ) {
     90                 * @param int    $blog_id      Blog ID to update for. Not used.
     91                 * @param string $option_name  Option name to update.
     92                 * @param mixed  $option_value Option value to update.
     93                 * @return bool
     94                 */
     95                function update_blog_option( $blog_id, $option_name, $option_value ) {
     96                        return update_option( $option_name, $option_value );
     97                }
     98        }
     99
     100        if ( ! function_exists( 'delete_blog_option' ) ) {
    101101
    102102                /**
    … …  
    109109                 * @param int    $blog_id     Blog ID to delete for. Not used.
    110110                 * @param string $option_name Option name to delete.
    111                  * @return mixed
     111                 * @return bool
    112112                 */
    113113                function delete_blog_option( $blog_id, $option_name ) {
    … …  
    116116        }
    117117
    118         if ( !function_exists( 'switch_to_blog' ) ) {
     118        if ( ! function_exists( 'switch_to_blog' ) ) {
    119119
    120120                /**
    … …  
    126126                 *
    127127                 * @param mixed $new_blog   New blog to switch to. Not used.
    128                  * @param null  $deprecated Whether or not deprecated. Not used.
     128                 * @param null  $deprecated Not used.
    129129                 * @return int
    130130                 */
    … …  
    134134        }
    135135
    136         if ( !function_exists( 'restore_current_blog' ) ) {
     136        if ( ! function_exists( 'restore_current_blog' ) ) {
    137137
    138138                /**
    … …  
    150150        }
    151151
    152         if ( !function_exists( 'get_blogs_of_user' ) ) {
    153 
    154                 /**
    155                  * Retrive blogs associated with user.
     152        if ( ! function_exists( 'get_blogs_of_user' ) ) {
     153
     154                /**
     155                 * Retrieve blogs associated with user.
    156156                 *
    157157                 * @since 1.2.0
    … …  
    160160                 *
    161161                 * @param int  $user_id ID of the user. Not used.
    162                  * @param bool $all     Whether or not to return all. Not used.
    163                  * @return bool
     162                 * @param bool $all     Whether to return all. Not used.
     163                 * @return false
    164164                 */
    165165                function get_blogs_of_user( $user_id, $all = false ) {
    … …  
    168168        }
    169169
    170         if ( !function_exists( 'update_blog_status' ) ) {
     170        if ( ! function_exists( 'update_blog_status' ) ) {
    171171
    172172                /**
    … …  
    181181                 * @param string $value      Value. Not used.
    182182                 * @param null   $deprecated Whether or not deprecated. Not used.
    183                  * @return bool
     183                 * @return true
    184184                 */
    185185                function update_blog_status( $blog_id, $pref, $value, $deprecated = null ) {
    … …  
    188188        }
    189189
    190         if ( !function_exists( 'is_subdomain_install' ) ) {
     190        if ( ! function_exists( 'is_subdomain_install' ) ) {
    191191
    192192                /**
    … …  
    200200                 */
    201201                function is_subdomain_install() {
    202                         if ( ( defined( 'VHOST' ) && 'yes' == VHOST ) || ( defined( 'SUBDOMAIN_INSTALL' ) && SUBDOMAIN_INSTALL ) )
    203                                 return true;
    204 
    205                         return false;
     202                        return ( ( defined( 'VHOST' ) && 'yes' === VHOST ) || ( defined( 'SUBDOMAIN_INSTALL' ) && SUBDOMAIN_INSTALL ) );
    206203                }
    207204        }
    … …  
    218215 */
    219216function bp_core_get_status_sql( $prefix = false ) {
    220         if ( !is_multisite() )
     217        if ( ! is_multisite() ) {
    221218                return "{$prefix}user_status = 0";
    222         else
    223                 return "{$prefix}spam = 0 AND {$prefix}deleted = 0 AND {$prefix}user_status = 0";
     219        }
     220
     221        return "{$prefix}spam = 0 AND {$prefix}deleted = 0 AND {$prefix}user_status = 0";
    224222}
    225223
    … …  
    232230 * Borrowed from MediaWiki, under the GPLv2. Thanks!
    233231 */
    234 if ( !function_exists( 'mb_strlen' ) ) {
     232if ( ! function_exists( 'mb_strlen' ) ) {
    235233
    236234        /**
    … …  
    243241        function mb_strlen( $str, $enc = '' ) {
    244242                $counts = count_chars( $str );
    245                 $total = 0;
     243                $total  = 0;
    246244
    247245                // Count ASCII bytes.
    248                 for( $i = 0; $i < 0x80; $i++ ) {
    249                         $total += $counts[$i];
     246                for ( $i = 0; $i < 0x80; $i++ ) {
     247                        $total += $counts[ $i ];
    250248                }
    251249
    252250                // Count multibyte sequence heads.
    253                 for( $i = 0xc0; $i < 0xff; $i++ ) {
    254                         $total += $counts[$i];
     251                for ( $i = 0xc0; $i < 0xff; $i++ ) {
     252                        $total += $counts[ $i ];
    255253                }
    256254                return $total;
    … …  
    258256}
    259257
    260 if ( !function_exists( 'mb_strpos' ) ) {
     258if ( ! function_exists( 'mb_strpos' ) ) {
    261259
    262260        /**
    … …  
    275273                preg_match( '/' . $needle . '/u', $haystack, $ar, PREG_OFFSET_CAPTURE, $offset );
    276274
    277                 if( isset( $ar[0][1] ) ) {
     275                if ( isset( $ar[0][1] ) ) {
    278276                        return $ar[0][1];
    279                 } else {
    280                         return false;
    281                 }
    282         }
    283 }
    284 
    285 if ( !function_exists( 'mb_strrpos' ) ) {
     277                }
     278
     279                return false;
     280        }
     281}
     282
     283if ( ! function_exists( 'mb_strrpos' ) ) {
    286284
    287285        /**
    … …  
    300298                preg_match_all( '/' . $needle . '/u', $haystack, $ar, PREG_OFFSET_CAPTURE, $offset );
    301299
    302                 if( isset( $ar[0] ) && count( $ar[0] ) > 0 &&
    303                         isset( $ar[0][count( $ar[0] ) - 1][1] ) ) {
    304                         return $ar[0][count( $ar[0] ) - 1][1];
    305                 } else {
    306                         return false;
    307                 }
     300                if ( isset( $ar[0] ) && count( $ar[0] ) > 0 &&
     301                        isset( $ar[0][ count( $ar[0] ) - 1 ][1] ) ) {
     302                        return $ar[0][ count( $ar[0] ) - 1 ][1];
     303                }
     304
     305                return false;
    308306        }
    309307}
    … …  
    313311 *
    314312 * @since 6.0.0
     313 *
     314 * @param WP_Error|null $errors The WP_Error object.
     315 * @param array         $data   Associative array of complete site data. See {@see wp_insert_site()}.
    315316 */
    316317function bp_catch_site_data( $errors = null, $data = array() ) {
    … …  
    390391function bp_delete_site_no_tables_drop( $site ) {
    391392        if ( isset( $site->deleted ) && 1 === (int) $site->deleted ) {
    392                 return bp_delete_site( $site->id, false );
    393         }
    394 }
    395 add_action( 'wp_update_site', 'bp_delete_site_no_tables_drop', 10, 1 );
     393                bp_delete_site( $site->id );
     394        }
     395}
     396add_action( 'wp_update_site', 'bp_delete_site_no_tables_drop' );
    396397
    397398/**
Note: See TracChangeset for help on using the changeset viewer.