WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
07/02/2019 11:41:16 PM (5 years ago)
Author:
pento
Message:

Coding Standards: Fix the Squiz.PHP.DisallowMultipleAssignments violations in wp-includes.

See #47632.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-includes/class-wp-xmlrpc-server.php

    r45554 r45590  
    669669        $password = $args[1];
    670670
    671         if ( ! $user = $this->login( $username, $password ) ) {
     671        $user = $this->login( $username, $password );
     672        if ( ! $user ) {
    672673            return $this->error;
    673674        }
    … …  
    12711272        $content_struct = $args[3];
    12721273
    1273         if ( ! $user = $this->login( $username, $password ) ) {
     1274        $user = $this->login( $username, $password );
     1275        if ( ! $user ) {
    12741276            return $this->error;
    12751277        }
    … …  
    16731675        $content_struct = $args[4];
    16741676
    1675         if ( ! $user = $this->login( $username, $password ) ) {
     1677        $user = $this->login( $username, $password );
     1678        if ( ! $user ) {
    16761679            return $this->error;
    16771680        }
    … …  
    17451748        $post_id  = (int) $args[3];
    17461749
    1747         if ( ! $user = $this->login( $username, $password ) ) {
     1750        $user = $this->login( $username, $password );
     1751        if ( ! $user ) {
    17481752            return $this->error;
    17491753        }
    … …  
    18441848        }
    18451849
    1846         if ( ! $user = $this->login( $username, $password ) ) {
     1850        $user = $this->login( $username, $password );
     1851        if ( ! $user ) {
    18471852            return $this->error;
    18481853        }
    … …  
    19041909        }
    19051910
    1906         if ( ! $user = $this->login( $username, $password ) ) {
     1911        $user = $this->login( $username, $password );
     1912        if ( ! $user ) {
    19071913            return $this->error;
    19081914        }
    … …  
    20022008        $content_struct = $args[3];
    20032009
    2004         if ( ! $user = $this->login( $username, $password ) ) {
     2010        $user = $this->login( $username, $password );
     2011        if ( ! $user ) {
    20052012            return $this->error;
    20062013        }
    … …  
    21062113        $content_struct = $args[4];
    21072114
    2108         if ( ! $user = $this->login( $username, $password ) ) {
     2115        $user = $this->login( $username, $password );
     2116        if ( ! $user ) {
    21092117            return $this->error;
    21102118        }
    … …  
    22212229        $term_id  = (int) $args[4];
    22222230
    2223         if ( ! $user = $this->login( $username, $password ) ) {
     2231        $user = $this->login( $username, $password );
     2232        if ( ! $user ) {
    22242233            return $this->error;
    22252234        }
    … …  
    22992308        $term_id  = (int) $args[4];
    23002309
    2301         if ( ! $user = $this->login( $username, $password ) ) {
     2310        $user = $this->login( $username, $password );
     2311        if ( ! $user ) {
    23022312            return $this->error;
    23032313        }
    … …  
    23632373        $filter   = isset( $args[4] ) ? $args[4] : array();
    23642374
    2365         if ( ! $user = $this->login( $username, $password ) ) {
     2375        $user = $this->login( $username, $password );
     2376        if ( ! $user ) {
    23662377            return $this->error;
    23672378        }
    … …  
    24682479        }
    24692480
    2470         if ( ! $user = $this->login( $username, $password ) ) {
     2481        $user = $this->login( $username, $password );
     2482        if ( ! $user ) {
    24712483            return $this->error;
    24722484        }
    … …  
    25252537        }
    25262538
    2527         if ( ! $user = $this->login( $username, $password ) ) {
     2539        $user = $this->login( $username, $password );
     2540        if ( ! $user ) {
    25282541            return $this->error;
    25292542        }
    … …  
    26102623        }
    26112624
    2612         if ( ! $user = $this->login( $username, $password ) ) {
     2625        $user = $this->login( $username, $password );
     2626        if ( ! $user ) {
    26132627            return $this->error;
    26142628        }
    … …  
    26722686        }
    26732687
    2674         if ( ! $user = $this->login( $username, $password ) ) {
     2688        $user = $this->login( $username, $password );
     2689        if ( ! $user ) {
    26752690            return $this->error;
    26762691        }
    … …  
    27512766        }
    27522767
    2753         if ( ! $user = $this->login( $username, $password ) ) {
     2768        $user = $this->login( $username, $password );
     2769        if ( ! $user ) {
    27542770            return $this->error;
    27552771        }
    … …  
    28002816        $content_struct = $args[3];
    28012817
    2802         if ( ! $user = $this->login( $username, $password ) ) {
     2818        $user = $this->login( $username, $password );
     2819        if ( ! $user ) {
    28032820            return $this->error;
    28042821        }
    … …  
    28792896        $password = $args[3];
    28802897
    2881         if ( ! $user = $this->login( $username, $password ) ) {
     2898        $user = $this->login( $username, $password );
     2899        if ( ! $user ) {
    28822900            return $this->error;
    28832901        }
    … …  
    29262944        $num_pages = isset( $args[3] ) ? (int) $args[3] : 10;
    29272945
    2928         if ( ! $user = $this->login( $username, $password ) ) {
     2946        $user = $this->login( $username, $password );
     2947        if ( ! $user ) {
    29292948            return $this->error;
    29302949        }
    … …  
    29843003        $password = $this->escape( $args[2] );
    29853004
    2986         if ( ! $user = $this->login( $username, $password ) ) {
     3005        $user = $this->login( $username, $password );
     3006        if ( ! $user ) {
    29873007            return $this->error;
    29883008        }
    … …  
    30203040        $page_id  = (int) $args[3];
    30213041
    3022         if ( ! $user = $this->login( $username, $password ) ) {
     3042        $user = $this->login( $username, $password );
     3043        if ( ! $user ) {
    30233044            return $this->error;
    30243045        }
    … …  
    30863107        $escaped_password = $this->escape( $password );
    30873108
    3088         if ( ! $user = $this->login( $escaped_username, $escaped_password ) ) {
     3109        $user = $this->login( $escaped_username, $escaped_password );
     3110        if ( ! $user ) {
    30893111            return $this->error;
    30903112        }
    … …  
    31443166        $password = $args[2];
    31453167
    3146         if ( ! $user = $this->login( $username, $password ) ) {
     3168        $user = $this->login( $username, $password );
     3169        if ( ! $user ) {
    31473170            return $this->error;
    31483171        }
    … …  
    32043227        $password = $args[2];
    32053228
    3206         if ( ! $user = $this->login( $username, $password ) ) {
     3229        $user = $this->login( $username, $password );
     3230        if ( ! $user ) {
    32073231            return $this->error;
    32083232        }
    … …  
    32473271        $password = $args[2];
    32483272
    3249         if ( ! $user = $this->login( $username, $password ) ) {
     3273        $user = $this->login( $username, $password );
     3274        if ( ! $user ) {
    32503275            return $this->error;
    32513276        }
    … …  
    32603285        $tags = array();
    32613286
    3262         if ( $all_tags = get_tags() ) {
     3287        $all_tags = get_tags();
     3288        if ( $all_tags ) {
    32633289            foreach ( (array) $all_tags as $tag ) {
    32643290                $struct             = array();
    … …  
    32993325        $category = $args[3];
    33003326
    3301         if ( ! $user = $this->login( $username, $password ) ) {
     3327        $user = $this->login( $username, $password );
     3328        if ( ! $user ) {
    33023329            return $this->error;
    33033330        }
    … …  
    33813408        $category_id = (int) $args[3];
    33823409
    3383         if ( ! $user = $this->login( $username, $password ) ) {
     3410        $user = $this->login( $username, $password );
     3411        if ( ! $user ) {
    33843412            return $this->error;
    33853413        }
    … …  
    34333461        $max_results = (int) $args[4];
    34343462
    3435         if ( ! $user = $this->login( $username, $password ) ) {
     3463        $user = $this->login( $username, $password );
     3464        if ( ! $user ) {
    34363465            return $this->error;
    34373466        }
    … …  
    34823511        $comment_id = (int) $args[3];
    34833512
    3484         if ( ! $user = $this->login( $username, $password ) ) {
     3513        $user = $this->login( $username, $password );
     3514        if ( ! $user ) {
    34853515            return $this->error;
    34863516        }
    … …  
    34893519        do_action( 'xmlrpc_call', 'wp.getComment' );
    34903520
    3491         if ( ! $comment = get_comment( $comment_id ) ) {
     3521        $comment = get_comment( $comment_id );
     3522        if ( ! $comment ) {
    34923523            return new IXR_Error( 404, __( 'Invalid comment ID.' ) );
    34933524        }
    … …  
    35333564        $struct   = isset( $args[3] ) ? $args[3] : array();
    35343565
    3535         if ( ! $user = $this->login( $username, $password ) ) {
     3566        $user = $this->login( $username, $password );
     3567        if ( ! $user ) {
    35363568            return $this->error;
    35373569        }
    … …  
    36193651        $comment_ID = (int) $args[3];
    36203652
    3621         if ( ! $user = $this->login( $username, $password ) ) {
     3653        $user = $this->login( $username, $password );
     3654        if ( ! $user ) {
    36223655            return $this->error;
    36233656        }
    … …  
    36863719        $content_struct = $args[4];
    36873720
    3688         if ( ! $user = $this->login( $username, $password ) ) {
     3721        $user = $this->login( $username, $password );
     3722        if ( ! $user ) {
    36893723            return $this->error;
    36903724        }
    … …  
    39183952        $password = $args[2];
    39193953
    3920         if ( ! $user = $this->login( $username, $password ) ) {
     3954        $user = $this->login( $username, $password );
     3955        if ( ! $user ) {
    39213956            return $this->error;
    39223957        }
    … …  
    39543989        $post_id  = (int) $args[3];
    39553990
    3956         if ( ! $user = $this->login( $username, $password ) ) {
     3991        $user = $this->login( $username, $password );
     3992        if ( ! $user ) {
    39573993            return $this->error;
    39583994        }
    … …  
    40004036        $password = $args[2];
    40014037
    4002         if ( ! $user = $this->login( $username, $password ) ) {
     4038        $user = $this->login( $username, $password );
     4039        if ( ! $user ) {
    40034040            return $this->error;
    40044041        }
    … …  
    40344071        $password = $args[2];
    40354072
    4036         if ( ! $user = $this->login( $username, $password ) ) {
     4073        $user = $this->login( $username, $password );
     4074        if ( ! $user ) {
    40374075            return $this->error;
    40384076        }
    … …  
    40684106        $password = $args[2];
    40694107
    4070         if ( ! $user = $this->login( $username, $password ) ) {
     4108        $user = $this->login( $username, $password );
     4109        if ( ! $user ) {
    40714110            return $this->error;
    40724111        }
    … …  
    41044143        $options  = isset( $args[3] ) ? (array) $args[3] : array();
    41054144
    4106         if ( ! $user = $this->login( $username, $password ) ) {
     4145        $user = $this->login( $username, $password );
     4146        if ( ! $user ) {
    41074147            return $this->error;
    41084148        }
    … …  
    41674207        $options  = (array) $args[3];
    41684208
    4169         if ( ! $user = $this->login( $username, $password ) ) {
     4209        $user = $this->login( $username, $password );
     4210        if ( ! $user ) {
    41704211            return $this->error;
    41714212        }
    … …  
    42234264        $attachment_id = (int) $args[3];
    42244265
    4225         if ( ! $user = $this->login( $username, $password ) ) {
     4266        $user = $this->login( $username, $password );
     4267        if ( ! $user ) {
    42264268            return $this->error;
    42274269        }
    … …  
    42344276        do_action( 'xmlrpc_call', 'wp.getMediaItem' );
    42354277
    4236         if ( ! $attachment = get_post( $attachment_id ) ) {
     4278        $attachment = get_post( $attachment_id );
     4279        if ( ! $attachment ) {
    42374280            return new IXR_Error( 404, __( 'Invalid attachment ID.' ) );
    42384281        }
    … …  
    42744317        $struct   = isset( $args[3] ) ? $args[3] : array();
    42754318
    4276         if ( ! $user = $this->login( $username, $password ) ) {
     4319        $user = $this->login( $username, $password );
     4320        if ( ! $user ) {
    42774321            return $this->error;
    42784322        }
    … …  
    43294373        $password = $args[2];
    43304374
    4331         if ( ! $user = $this->login( $username, $password ) ) {
     4375        $user = $this->login( $username, $password );
     4376        if ( ! $user ) {
    43324377            return $this->error;
    43334378        }
    … …  
    44124457        }
    44134458
    4414         if ( ! $user = $this->login( $username, $password ) ) {
     4459        $user = $this->login( $username, $password );
     4460        if ( ! $user ) {
    44154461            return $this->error;
    44164462        }
    … …  
    44684514        }
    44694515
    4470         if ( ! $user = $this->login( $username, $password ) ) {
     4516        $user = $this->login( $username, $password );
     4517        if ( ! $user ) {
    44714518            return $this->error;
    44724519        }
    … …  
    45374584        }
    45384585
    4539         if ( ! $user = $this->login( $username, $password ) ) {
     4586        $user = $this->login( $username, $password );
     4587        if ( ! $user ) {
    45404588            return $this->error;
    45414589        }
    … …  
    45444592        do_action( 'xmlrpc_call', 'wp.getRevisions' );
    45454593
    4546         if ( ! $post = get_post( $post_id ) ) {
     4594        $post = get_post( $post_id );
     4595        if ( ! $post ) {
    45474596            return new IXR_Error( 404, __( 'Invalid post ID.' ) );
    45484597        }
    … …  
    46094658        $revision_id = (int) $args[3];
    46104659
    4611         if ( ! $user = $this->login( $username, $password ) ) {
     4660        $user = $this->login( $username, $password );
     4661        if ( ! $user ) {
    46124662            return $this->error;
    46134663        }
    … …  
    46164666        do_action( 'xmlrpc_call', 'wp.restoreRevision' );
    46174667
    4618         if ( ! $revision = wp_get_post_revision( $revision_id ) ) {
     4668        $revision = wp_get_post_revision( $revision_id );
     4669        if ( ! $revision ) {
    46194670            return new IXR_Error( 404, __( 'Invalid post ID.' ) );
    46204671        }
    … …  
    46244675        }
    46254676
    4626         if ( ! $post = get_post( $revision->post_parent ) ) {
     4677        $post = get_post( $revision->post_parent );
     4678        if ( ! $post ) {
    46274679            return new IXR_Error( 404, __( 'Invalid post ID.' ) );
    46284680        }
    … …  
    46764728        $password = $args[2];
    46774729
    4678         if ( ! $user = $this->login( $username, $password ) ) {
     4730        $user = $this->login( $username, $password );
     4731        if ( ! $user ) {
    46794732            return $this->error;
    46804733        }
    … …  
    47574810        $password = $args[2];
    47584811
    4759         if ( ! $user = $this->login( $username, $password ) ) {
     4812        $user = $this->login( $username, $password );
     4813        if ( ! $user ) {
    47604814            return $this->error;
    47614815        }
    … …  
    48014855        $password = $args[3];
    48024856
    4803         if ( ! $user = $this->login( $username, $password ) ) {
     4857        $user = $this->login( $username, $password );
     4858        if ( ! $user ) {
    48044859            return $this->error;
    48054860        }
    … …  
    48624917        }
    48634918
    4864         if ( ! $user = $this->login( $username, $password ) ) {
     4919        $user = $this->login( $username, $password );
     4920        if ( ! $user ) {
    48654921            return $this->error;
    48664922        }
    … …  
    49555011        $publish  = $args[5];
    49565012
    4957         if ( ! $user = $this->login( $username, $password ) ) {
     5013        $user = $this->login( $username, $password );
     5014        if ( ! $user ) {
    49585015            return $this->error;
    49595016        }
    … …  
    50315088        $publish  = $args[5];
    50325089
    5033         if ( ! $user = $this->login( $username, $password ) ) {
     5090        $user = $this->login( $username, $password );
     5091        if ( ! $user ) {
    50345092            return $this->error;
    50355093        }
    … …  
    51045162        $password = $args[3];
    51055163
    5106         if ( ! $user = $this->login( $username, $password ) ) {
     5164        $user = $this->login( $username, $password );
     5165        if ( ! $user ) {
    51075166            return $this->error;
    51085167        }
    … …  
    51905249        $publish        = isset( $args[4] ) ? $args[4] : 0;
    51915250
    5192         if ( ! $user = $this->login( $username, $password ) ) {
     5251        $user = $this->login( $username, $password );
     5252        if ( ! $user ) {
    51935253            return $this->error;
    51945254        }
    … …  
    54325492        $postdata = compact( 'post_author', 'post_date', 'post_date_gmt', 'post_content', 'post_title', 'post_category', 'post_status', 'post_excerpt', 'comment_status', 'ping_status', 'to_ping', 'post_type', 'post_name', 'post_password', 'post_parent', 'menu_order', 'tags_input', 'page_template' );
    54335493
    5434         $post_ID = $postdata['ID'] = get_default_post_to_edit( $post_type, true )->ID;
     5494        $post_ID        = get_default_post_to_edit( $post_type, true )->ID;
     5495        $postdata['ID'] = $post_ID;
    54355496
    54365497        // Only posts can be sticky
    … …  
    55005561    public function add_enclosure_if_new( $post_ID, $enclosure ) {
    55015562        if ( is_array( $enclosure ) && isset( $enclosure['url'] ) && isset( $enclosure['length'] ) && isset( $enclosure['type'] ) ) {
    5502             $encstring = $enclosure['url'] . "\n" . $enclosure['length'] . "\n" . $enclosure['type'] . "\n";
    5503             $found     = false;
    5504             if ( $enclosures = get_post_meta( $post_ID, 'enclosure' ) ) {
     5563            $encstring  = $enclosure['url'] . "\n" . $enclosure['length'] . "\n" . $enclosure['type'] . "\n";
     5564            $found      = false;
     5565            $enclosures = get_post_meta( $post_ID, 'enclosure' );
     5566            if ( $enclosures ) {
    55055567                foreach ( $enclosures as $enc ) {
    55065568                    // This method used to omit the trailing new line. #23219
    … …  
    55665628        $publish        = isset( $args[4] ) ? $args[4] : 0;
    55675629
    5568         if ( ! $user = $this->login( $username, $password ) ) {
     5630        $user = $this->login( $username, $password );
     5631        if ( ! $user ) {
    55695632            return $this->error;
    55705633        }
    … …  
    58945957        $password = $args[2];
    58955958
    5896         if ( ! $user = $this->login( $username, $password ) ) {
     5959        $user = $this->login( $username, $password );
     5960        if ( ! $user ) {
    58975961            return $this->error;
    58985962        }
    … …  
    60396103        }
    60406104
    6041         if ( ! $user = $this->login( $username, $password ) ) {
     6105        $user = $this->login( $username, $password );
     6106        if ( ! $user ) {
    60426107            return $this->error;
    60436108        }
    … …  
    61596224        $password = $args[2];
    61606225
    6161         if ( ! $user = $this->login( $username, $password ) ) {
     6226        $user = $this->login( $username, $password );
     6227        if ( ! $user ) {
    61626228            return $this->error;
    61636229        }
    … …  
    61726238        $categories_struct = array();
    61736239
    6174         if ( $cats = get_categories( array( 'get' => 'all' ) ) ) {
     6240        $cats = get_categories( array( 'get' => 'all' ) );
     6241        if ( $cats ) {
    61756242            foreach ( $cats as $cat ) {
    61766243                $struct                        = array();
    … …  
    62226289        $bits = $data['bits'];
    62236290
    6224         if ( ! $user = $this->login( $username, $password ) ) {
     6291        $user = $this->login( $username, $password );
     6292        if ( ! $user ) {
    62256293            return $this->error;
    62266294        }
    … …  
    62566324         * @param bool $error Whether to pre-empt the media upload. Default false.
    62576325         */
    6258         if ( $upload_err = apply_filters( 'pre_upload_error', false ) ) {
     6326        $upload_err = apply_filters( 'pre_upload_error', false );
     6327        if ( $upload_err ) {
    62596328            return new IXR_Error( 500, $upload_err );
    62606329        }
    … …  
    63386407        }
    63396408
    6340         if ( ! $user = $this->login( $username, $password ) ) {
     6409        $user = $this->login( $username, $password );
     6410        if ( ! $user ) {
    63416411            return $this->error;
    63426412        }
    … …  
    63956465        $password = $args[2];
    63966466
    6397         if ( ! $user = $this->login( $username, $password ) ) {
     6467        $user = $this->login( $username, $password );
     6468        if ( ! $user ) {
    63986469            return $this->error;
    63996470        }
    … …  
    64086479        $categories_struct = array();
    64096480
    6410         if ( $cats = get_categories(
     6481        $cats = get_categories(
    64116482            array(
    64126483                'hide_empty'   => 0,
    64136484                'hierarchical' => 0,
    64146485            )
    6415         ) ) {
     6486        );
     6487        if ( $cats ) {
    64166488            foreach ( $cats as $cat ) {
    64176489                $struct                 = array();
    … …  
    64476519        $password = $args[2];
    64486520
    6449         if ( ! $user = $this->login( $username, $password ) ) {
     6521        $user = $this->login( $username, $password );
     6522        if ( ! $user ) {
    64506523            return $this->error;
    64516524        }
    … …  
    65016574        $categories = $args[3];
    65026575
    6503         if ( ! $user = $this->login( $username, $password ) ) {
     6576        $user = $this->login( $username, $password );
     6577        if ( ! $user ) {
    65046578            return $this->error;
    65056579        }
    … …  
    66246698        $password = $args[2];
    66256699
    6626         if ( ! $user = $this->login( $username, $password ) ) {
     6700        $user = $this->login( $username, $password );
     6701        if ( ! $user ) {
    66276702            return $this->error;
    66286703        }
    … …  
    67036778        //        if so, then let's use it and drop the old code.
    67046779        $urltest = parse_url( $pagelinkedto );
    6705         if ( $post_ID = url_to_postid( $pagelinkedto ) ) {
     6780        $post_ID = url_to_postid( $pagelinkedto );
     6781        if ( $post_ID ) {
    67066782            // $way
    67076783        } elseif ( isset( $urltest['path'] ) && preg_match( '#p/[0-9]{1,}#', $urltest['path'], $match ) ) {
    … …  
    67236799            } elseif ( is_string( $urltest['fragment'] ) ) {
    67246800                // ...or a string #title, a little more complicated
    6725                 $title = preg_replace( '/[^a-z0-9]/i', '.', $urltest['fragment'] );
    6726                 $sql   = $wpdb->prepare( "SELECT ID FROM $wpdb->posts WHERE post_title RLIKE %s", $title );
    6727                 if ( ! ( $post_ID = $wpdb->get_var( $sql ) ) ) {
     6801                $title   = preg_replace( '/[^a-z0-9]/i', '.', $urltest['fragment'] );
     6802                $sql     = $wpdb->prepare( "SELECT ID FROM $wpdb->posts WHERE post_title RLIKE %s", $title );
     6803                $post_ID = $wpdb->get_var( $sql );
     6804                if ( ! $post_ID ) {
    67286805                    // returning unknown error '0' is better than die()ing
    67296806                    return $this->pingback_error( 0, '' );
    … …  
    67756852        );
    67766853
    6777         $request       = wp_safe_remote_get( $pagelinkedfrom, $http_api_args );
    6778         $remote_source = $remote_source_original = wp_remote_retrieve_body( $request );
     6854        $request                = wp_safe_remote_get( $pagelinkedfrom, $http_api_args );
     6855        $remote_source          = wp_remote_retrieve_body( $request );
     6856        $remote_source_original = $remote_source;
    67796857
    67806858        if ( ! $remote_source ) {
Note: See TracChangeset for help on using the changeset viewer.