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/post.php

    r45424 r45590  
    454454
    455455    // If the file is relative, prepend upload dir.
    456     if ( $file && 0 !== strpos( $file, '/' ) && ! preg_match( '|^.:\\\|', $file ) && ( ( $uploads = wp_get_upload_dir() ) && false === $uploads['error'] ) ) {
    457         $file = $uploads['basedir'] . "/$file";
     456    if ( $file && 0 !== strpos( $file, '/' ) && ! preg_match( '|^.:\\\|', $file ) ) {
     457        $uploads = wp_get_upload_dir();
     458        if ( false === $uploads['error'] ) {
     459            $file = $uploads['basedir'] . "/$file";
     460        }
    458461    }
    459462
    … …  
    500503    $file = apply_filters( 'update_attached_file', $file, $attachment_id );
    501504
    502     if ( $file = _wp_relative_upload_path( $file ) ) {
     505    $file = _wp_relative_upload_path( $file );
     506    if ( $file ) {
    503507        return update_post_meta( $attachment_id, '_wp_attached_file', $file );
    504508    } else {
    … …  
    756760    $ancestors = array();
    757761
    758     $id = $ancestors[] = $post->post_parent;
     762    $id          = $post->post_parent;
     763    $ancestors[] = $id;
    759764
    760765    while ( $ancestor = get_post( $id ) ) {
    … …  
    764769        }
    765770
    766         $id = $ancestors[] = $ancestor->post_parent;
     771        $id          = $ancestor->post_parent;
     772        $ancestors[] = $id;
    767773    }
    768774
    … …  
    11491155 */
    11501156function get_post_type( $post = null ) {
    1151     if ( $post = get_post( $post ) ) {
     1157    $post = get_post( $post );
     1158    if ( $post ) {
    11521159        return $post->post_type;
    11531160    }
    … …  
    21552162    }
    21562163
    2157     if ( $keys = array_keys( $custom ) ) {
     2164    $keys = array_keys( $custom );
     2165    if ( $keys ) {
    21582166        return $keys;
    21592167    }
    … …  
    40524060    global $wpdb;
    40534061
    4054     if ( ! $post = get_post( $post ) ) {
     4062    $post = get_post( $post );
     4063    if ( ! $post ) {
    40554064        return;
    40564065    }
    … …  
    42284237        $post                        = get_post( $post_ID );
    42294238        $conflicts_with_date_archive = false;
    4230         if ( 'post' === $post_type && ( ! $post || $post->post_name !== $slug ) && preg_match( '/^[0-9]+$/', $slug ) && $slug_num = intval( $slug ) ) {
    4231             $permastructs   = array_values( array_filter( explode( '/', get_option( 'permalink_structure' ) ) ) );
    4232             $postname_index = array_search( '%postname%', $permastructs );
    4233 
    4234             /*
    4235              * Potential date clashes are as follows:
    4236              *
    4237              * - Any integer in the first permastruct position could be a year.
    4238              * - An integer between 1 and 12 that follows 'year' conflicts with 'monthnum'.
    4239              * - An integer between 1 and 31 that follows 'monthnum' conflicts with 'day'.
    4240              */
    4241             if ( 0 === $postname_index ||
    4242                 ( $postname_index && '%year%' === $permastructs[ $postname_index - 1 ] && 13 > $slug_num ) ||
    4243                 ( $postname_index && '%monthnum%' === $permastructs[ $postname_index - 1 ] && 32 > $slug_num )
    4244             ) {
    4245                 $conflicts_with_date_archive = true;
     4239        if ( 'post' === $post_type && ( ! $post || $post->post_name !== $slug ) && preg_match( '/^[0-9]+$/', $slug ) ) {
     4240            $slug_num = intval( $slug );
     4241
     4242            if ( $slug_num ) {
     4243                $permastructs   = array_values( array_filter( explode( '/', get_option( 'permalink_structure' ) ) ) );
     4244                $postname_index = array_search( '%postname%', $permastructs );
     4245
     4246                /*
     4247                * Potential date clashes are as follows:
     4248                *
     4249                * - Any integer in the first permastruct position could be a year.
     4250                * - An integer between 1 and 12 that follows 'year' conflicts with 'monthnum'.
     4251                * - An integer between 1 and 31 that follows 'monthnum' conflicts with 'day'.
     4252                */
     4253                if ( 0 === $postname_index ||
     4254                    ( $postname_index && '%year%' === $permastructs[ $postname_index - 1 ] && 13 > $slug_num ) ||
     4255                    ( $postname_index && '%monthnum%' === $permastructs[ $postname_index - 1 ] && 32 > $slug_num )
     4256                ) {
     4257                    $conflicts_with_date_archive = true;
     4258                }
    42464259            }
    42474260        }
    … …  
    53215334        return true;
    53225335    }
    5323     if ( $id = url_to_postid( $url ) ) {
     5336
     5337    $id = url_to_postid( $url );
     5338    if ( $id ) {
    53245339        $post = get_post( $id );
    53255340        if ( 'attachment' == $post->post_type ) {
    … …  
    55455560function wp_get_attachment_metadata( $attachment_id = 0, $unfiltered = false ) {
    55465561    $attachment_id = (int) $attachment_id;
    5547     if ( ! $post = get_post( $attachment_id ) ) {
     5562    $post          = get_post( $attachment_id );
     5563    if ( ! $post ) {
    55485564        return false;
    55495565    }
    … …  
    55785594function wp_update_attachment_metadata( $attachment_id, $data ) {
    55795595    $attachment_id = (int) $attachment_id;
    5580     if ( ! $post = get_post( $attachment_id ) ) {
     5596    $post          = get_post( $attachment_id );
     5597    if ( ! $post ) {
    55815598        return false;
    55825599    }
    … …  
    55905607     * @param int   $attachment_id Attachment post ID.
    55915608     */
    5592     if ( $data = apply_filters( 'wp_update_attachment_metadata', $data, $post->ID ) ) {
     5609    $data = apply_filters( 'wp_update_attachment_metadata', $data, $post->ID );
     5610    if ( $data ) {
    55935611        return update_post_meta( $post->ID, '_wp_attachment_metadata', $data );
    55945612    } else {
    … …  
    56095627function wp_get_attachment_url( $attachment_id = 0 ) {
    56105628    $attachment_id = (int) $attachment_id;
    5611     if ( ! $post = get_post( $attachment_id ) ) {
     5629    $post          = get_post( $attachment_id );
     5630    if ( ! $post ) {
    56125631        return false;
    56135632    }
    … …  
    56195638    $url = '';
    56205639    // Get attached file.
    5621     if ( $file = get_post_meta( $post->ID, '_wp_attached_file', true ) ) {
     5640    $file = get_post_meta( $post->ID, '_wp_attached_file', true );
     5641    if ( $file ) {
    56225642        // Get upload directory.
    5623         if ( ( $uploads = wp_get_upload_dir() ) && false === $uploads['error'] ) {
     5643        $uploads = wp_get_upload_dir();
     5644        if ( $uploads && false === $uploads['error'] ) {
    56245645            // Check that the upload base exists in the file location.
    56255646            if ( 0 === strpos( $file, $uploads['basedir'] ) ) {
    … …  
    56765697function wp_get_attachment_caption( $post_id = 0 ) {
    56775698    $post_id = (int) $post_id;
    5678     if ( ! $post = get_post( $post_id ) ) {
     5699    $post    = get_post( $post_id );
     5700    if ( ! $post ) {
    56795701        return false;
    56805702    }
    … …  
    57075729function wp_get_attachment_thumb_file( $post_id = 0 ) {
    57085730    $post_id = (int) $post_id;
    5709     if ( ! $post = get_post( $post_id ) ) {
     5731    $post    = get_post( $post_id );
     5732    if ( ! $post ) {
    57105733        return false;
    57115734    }
    5712     if ( ! is_array( $imagedata = wp_get_attachment_metadata( $post->ID ) ) ) {
     5735
     5736    $imagedata = wp_get_attachment_metadata( $post->ID );
     5737    if ( ! is_array( $imagedata ) ) {
    57135738        return false;
    57145739    }
    … …  
    57165741    $file = get_attached_file( $post->ID );
    57175742
    5718     if ( ! empty( $imagedata['thumb'] ) && ( $thumbfile = str_replace( wp_basename( $file ), $imagedata['thumb'], $file ) ) && file_exists( $thumbfile ) ) {
    5719         /**
    5720          * Filters the attachment thumbnail file path.
    5721          *
    5722          * @since 2.1.0
    5723          *
    5724          * @param string $thumbfile File path to the attachment thumbnail.
    5725          * @param int    $post_id   Attachment ID.
    5726          */
    5727         return apply_filters( 'wp_get_attachment_thumb_file', $thumbfile, $post->ID );
     5743    if ( ! empty( $imagedata['thumb'] ) ) {
     5744        $thumbfile = str_replace( wp_basename( $file ), $imagedata['thumb'], $file );
     5745        if ( file_exists( $thumbfile ) ) {
     5746            /**
     5747             * Filters the attachment thumbnail file path.
     5748             *
     5749             * @since 2.1.0
     5750             *
     5751             * @param string $thumbfile File path to the attachment thumbnail.
     5752             * @param int    $post_id   Attachment ID.
     5753             */
     5754            return apply_filters( 'wp_get_attachment_thumb_file', $thumbfile, $post->ID );
     5755        }
    57285756    }
    57295757    return false;
    … …  
    57405768function wp_get_attachment_thumb_url( $post_id = 0 ) {
    57415769    $post_id = (int) $post_id;
    5742     if ( ! $post = get_post( $post_id ) ) {
     5770    $post    = get_post( $post_id );
     5771    if ( ! $post ) {
    57435772        return false;
    57445773    }
    5745     if ( ! $url = wp_get_attachment_url( $post->ID ) ) {
     5774
     5775    $url = wp_get_attachment_url( $post->ID );
     5776    if ( ! $url ) {
    57465777        return false;
    57475778    }
    … …  
    57525783    }
    57535784
    5754     if ( ! $thumb = wp_get_attachment_thumb_file( $post->ID ) ) {
     5785    $thumb = wp_get_attachment_thumb_file( $post->ID );
     5786    if ( ! $thumb ) {
    57555787        return false;
    57565788    }
    … …  
    57795811 */
    57805812function wp_attachment_is( $type, $post = null ) {
    5781     if ( ! $post = get_post( $post ) ) {
     5813    $post = get_post( $post );
     5814    if ( ! $post ) {
    57825815        return false;
    57835816    }
    57845817
    5785     if ( ! $file = get_attached_file( $post->ID ) ) {
     5818    $file = get_attached_file( $post->ID );
     5819    if ( ! $file ) {
    57865820        return false;
    57875821    }
    … …  
    58545888        if ( is_numeric( $mime ) ) {
    58555889            $mime = (int) $mime;
    5856             if ( $post = get_post( $mime ) ) {
     5890            $post = get_post( $mime );
     5891            if ( $post ) {
    58575892                $post_id = (int) $post->ID;
    58585893                $file    = get_attached_file( $post_id );
    … …  
    58605895                if ( ! empty( $ext ) ) {
    58615896                    $post_mimes[] = $ext;
    5862                     if ( $ext_type = wp_ext2type( $ext ) ) {
     5897                    $ext_type     = wp_ext2type( $ext );
     5898                    if ( $ext_type ) {
    58635899                        $post_mimes[] = $ext_type;
    58645900                    }
    … …  
    59065942                $dir  = array_shift( $keys );
    59075943                $uri  = array_shift( $dirs );
    5908                 if ( $dh = opendir( $dir ) ) {
     5944                $dh   = opendir( $dir );
     5945                if ( $dh ) {
    59095946                    while ( false !== $file = readdir( $dh ) ) {
    59105947                        $file = wp_basename( $file );
    … …  
    61136150         * @param string $cap Capability.
    61146151         */
    6115         if ( ! $cap = apply_filters( 'pub_priv_sql_capability', '' ) ) {
     6152        $cap = apply_filters( 'pub_priv_sql_capability', '' );
     6153        if ( ! $cap ) {
    61166154            $cap = current_user_can( $post_type_obj->cap->read_private_posts );
    61176155        }
    … …  
    66436681
    66446682    // Now look for larger loops.
    6645     if ( ! $loop = wp_find_hierarchy_loop( 'wp_get_post_parent_id', $post_ID, $post_parent ) ) {
     6683    $loop = wp_find_hierarchy_loop( 'wp_get_post_parent_id', $post_ID, $post_parent );
     6684    if ( ! $loop ) {
    66466685        return $post_parent; // No loop
    66476686    }
    … …  
    67296768 */
    67306769function wp_queue_posts_for_term_meta_lazyload( $posts ) {
    6731     $post_type_taxonomies = $term_ids = array();
     6770    $post_type_taxonomies = array();
     6771    $term_ids             = array();
    67326772    foreach ( $posts as $post ) {
    67336773        if ( ! ( $post instanceof WP_Post ) ) {
Note: See TracChangeset for help on using the changeset viewer.