WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (7 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-includes/revision.php

    r42228 r42343  
    3232        // Allow these to be versioned
    3333        $fields = array(
    34             'post_title' => __( 'Title' ),
     34            'post_title'   => __( 'Title' ),
    3535            'post_content' => __( 'Content' ),
    3636            'post_excerpt' => __( 'Excerpt' ),
    … …  
    6060        unset( $fields[ $protect ] );
    6161    }
    62 
    6362
    6463    return $fields;
    … …  
    111110 */
    112111function wp_save_post_revision( $post_id ) {
    113     if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE )
     112    if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) {
    114113        return;
    115 
    116     if ( ! $post = get_post( $post_id ) )
     114    }
     115
     116    if ( ! $post = get_post( $post_id ) ) {
    117117        return;
    118 
    119     if ( ! post_type_supports( $post->post_type, 'revisions' ) )
     118    }
     119
     120    if ( ! post_type_supports( $post->post_type, 'revisions' ) ) {
    120121        return;
    121 
    122     if ( 'auto-draft' == $post->post_status )
     122    }
     123
     124    if ( 'auto-draft' == $post->post_status ) {
    123125        return;
    124 
    125     if ( ! wp_revisions_enabled( $post ) )
     126    }
     127
     128    if ( ! wp_revisions_enabled( $post ) ) {
    126129        return;
     130    }
    127131
    128132    // Compare the proposed update with the last stored revision verifying that
    … …  
    150154         * @param WP_Post $last_revision     The last revision post object.
    151155         * @param WP_Post $post              The post object.
    152          *
    153156         */
    154157        if ( isset( $last_revision ) && apply_filters( 'wp_save_post_revision_check_for_changes', $check_for_changes = true, $last_revision, $post ) ) {
    … …  
    173176             * @param WP_Post $last_revision    The last revision post object.
    174177             * @param WP_Post $post             The post object.
    175              *
    176178             */
    177179            $post_has_changed = (bool) apply_filters( 'wp_save_post_revision_post_has_changed', $post_has_changed, $last_revision, $post );
    … …  
    190192    $revisions_to_keep = wp_revisions_to_keep( $post );
    191193
    192     if ( $revisions_to_keep < 0 )
     194    if ( $revisions_to_keep < 0 ) {
    193195        return $return;
     196    }
    194197
    195198    $revisions = wp_get_post_revisions( $post_id, array( 'order' => 'ASC' ) );
    196199
    197     $delete = count($revisions) - $revisions_to_keep;
    198 
    199     if ( $delete < 1 )
     200    $delete = count( $revisions ) - $revisions_to_keep;
     201
     202    if ( $delete < 1 ) {
    200203        return $return;
     204    }
    201205
    202206    $revisions = array_slice( $revisions, 0, $delete );
    203207
    204     for ( $i = 0; isset( $revisions[$i] ); $i++ ) {
    205         if ( false !== strpos( $revisions[ $i ]->post_name, 'autosave' ) )
     208    for ( $i = 0; isset( $revisions[ $i ] ); $i++ ) {
     209        if ( false !== strpos( $revisions[ $i ]->post_name, 'autosave' ) ) {
    206210            continue;
     211        }
    207212
    208213        wp_delete_post_revision( $revisions[ $i ]->ID );
    … …  
    230235    foreach ( $revisions as $revision ) {
    231236        if ( false !== strpos( $revision->post_name, "{$post_id}-autosave" ) ) {
    232             if ( $user_id && $user_id != $revision->post_author )
     237            if ( $user_id && $user_id != $revision->post_author ) {
    233238                continue;
     239            }
    234240
    235241            return $revision;
    … …  
    249255 */
    250256function wp_is_post_revision( $post ) {
    251     if ( !$post = wp_get_post_revision( $post ) )
     257    if ( ! $post = wp_get_post_revision( $post ) ) {
    252258        return false;
     259    }
    253260
    254261    return (int) $post->post_parent;
    … …  
    264271 */
    265272function wp_is_post_autosave( $post ) {
    266     if ( !$post = wp_get_post_revision( $post ) )
     273    if ( ! $post = wp_get_post_revision( $post ) ) {
    267274        return false;
    268 
    269     if ( false !== strpos( $post->post_name, "{$post->post_parent}-autosave" ) )
     275    }
     276
     277    if ( false !== strpos( $post->post_name, "{$post->post_parent}-autosave" ) ) {
    270278        return (int) $post->post_parent;
     279    }
    271280
    272281    return false;
    … …  
    284293 */
    285294function _wp_put_post_revision( $post = null, $autosave = false ) {
    286     if ( is_object($post) )
     295    if ( is_object( $post ) ) {
    287296        $post = get_object_vars( $post );
    288     elseif ( !is_array($post) )
    289         $post = get_post($post, ARRAY_A);
    290 
    291     if ( ! $post || empty($post['ID']) )
     297    } elseif ( ! is_array( $post ) ) {
     298        $post = get_post( $post, ARRAY_A );
     299    }
     300
     301    if ( ! $post || empty( $post['ID'] ) ) {
    292302        return new WP_Error( 'invalid_post', __( 'Invalid post ID.' ) );
    293 
    294     if ( isset($post['post_type']) && 'revision' == $post['post_type'] )
     303    }
     304
     305    if ( isset( $post['post_type'] ) && 'revision' == $post['post_type'] ) {
    295306        return new WP_Error( 'post_type', __( 'Cannot create a revision of a revision' ) );
     307    }
    296308
    297309    $post = _wp_post_revision_data( $post, $autosave );
    298     $post = wp_slash($post); //since data is from db
     310    $post = wp_slash( $post ); //since data is from db
    299311
    300312    $revision_id = wp_insert_post( $post );
    301     if ( is_wp_error($revision_id) )
     313    if ( is_wp_error( $revision_id ) ) {
    302314        return $revision_id;
     315    }
    303316
    304317    if ( $revision_id ) {
    … …  
    327340 * @return WP_Post|array|null WP_Post (or array) on success, or null on failure.
    328341 */
    329 function wp_get_post_revision(&$post, $output = OBJECT, $filter = 'raw') {
    330     if ( !$revision = get_post( $post, OBJECT, $filter ) )
     342function wp_get_post_revision( &$post, $output = OBJECT, $filter = 'raw' ) {
     343    if ( ! $revision = get_post( $post, OBJECT, $filter ) ) {
    331344        return $revision;
    332     if ( 'revision' !== $revision->post_type )
     345    }
     346    if ( 'revision' !== $revision->post_type ) {
    333347        return null;
     348    }
    334349
    335350    if ( $output == OBJECT ) {
    336351        return $revision;
    337352    } elseif ( $output == ARRAY_A ) {
    338         $_revision = get_object_vars($revision);
     353        $_revision = get_object_vars( $revision );
    339354        return $_revision;
    340355    } elseif ( $output == ARRAY_N ) {
    341         $_revision = array_values(get_object_vars($revision));
     356        $_revision = array_values( get_object_vars( $revision ) );
    342357        return $_revision;
    343358    }
    … …  
    358373 */
    359374function wp_restore_post_revision( $revision_id, $fields = null ) {
    360     if ( !$revision = wp_get_post_revision( $revision_id, ARRAY_A ) )
     375    if ( ! $revision = wp_get_post_revision( $revision_id, ARRAY_A ) ) {
    361376        return $revision;
    362 
    363     if ( !is_array( $fields ) )
     377    }
     378
     379    if ( ! is_array( $fields ) ) {
    364380        $fields = array_keys( _wp_post_revision_fields( $revision ) );
     381    }
    365382
    366383    $update = array();
    367384    foreach ( array_intersect( array_keys( $revision ), $fields ) as $field ) {
    368         $update[$field] = $revision[$field];
    369     }
    370 
    371     if ( !$update )
     385        $update[ $field ] = $revision[ $field ];
     386    }
     387
     388    if ( ! $update ) {
    372389        return false;
     390    }
    373391
    374392    $update['ID'] = $revision['post_parent'];
    … …  
    377395
    378396    $post_id = wp_update_post( $update );
    379     if ( ! $post_id || is_wp_error( $post_id ) )
     397    if ( ! $post_id || is_wp_error( $post_id ) ) {
    380398        return $post_id;
     399    }
    381400
    382401    // Update last edit user
    … …  
    440459function wp_get_post_revisions( $post_id = 0, $args = null ) {
    441460    $post = get_post( $post_id );
    442     if ( ! $post || empty( $post->ID ) )
     461    if ( ! $post || empty( $post->ID ) ) {
    443462        return array();
    444 
    445     $defaults = array( 'order' => 'DESC', 'orderby' => 'date ID', 'check_enabled' => true );
    446     $args = wp_parse_args( $args, $defaults );
    447 
    448     if ( $args['check_enabled'] && ! wp_revisions_enabled( $post ) )
     463    }
     464
     465    $defaults = array(
     466        'order'         => 'DESC',
     467        'orderby'       => 'date ID',
     468        'check_enabled' => true,
     469    );
     470    $args     = wp_parse_args( $args, $defaults );
     471
     472    if ( $args['check_enabled'] && ! wp_revisions_enabled( $post ) ) {
    449473        return array();
    450 
    451     $args = array_merge( $args, array( 'post_parent' => $post->ID, 'post_type' => 'revision', 'post_status' => 'inherit' ) );
    452 
    453     if ( ! $revisions = get_children( $args ) )
     474    }
     475
     476    $args = array_merge(
     477        $args, array(
     478            'post_parent' => $post->ID,
     479            'post_type'   => 'revision',
     480            'post_status' => 'inherit',
     481        )
     482    );
     483
     484    if ( ! $revisions = get_children( $args ) ) {
    454485        return array();
     486    }
    455487
    456488    return $revisions;
    … …  
    485517    $num = WP_POST_REVISIONS;
    486518
    487     if ( true === $num )
     519    if ( true === $num ) {
    488520        $num = -1;
    489     else
     521    } else {
    490522        $num = intval( $num );
    491 
    492     if ( ! post_type_supports( $post->post_type, 'revisions' ) )
     523    }
     524
     525    if ( ! post_type_supports( $post->post_type, 'revisions' ) ) {
    493526        $num = 0;
     527    }
    494528
    495529    /**
    … …  
    528562
    529563    $post->post_content = $preview->post_content;
    530     $post->post_title = $preview->post_title;
     564    $post->post_title   = $preview->post_title;
    531565    $post->post_excerpt = $preview->post_excerpt;
    532566
    … …  
    544578 */
    545579function _show_post_preview() {
    546     if ( isset($_GET['preview_id']) && isset($_GET['preview_nonce']) ) {
     580    if ( isset( $_GET['preview_id'] ) && isset( $_GET['preview_nonce'] ) ) {
    547581        $id = (int) $_GET['preview_id'];
    548582
    549         if ( false === wp_verify_nonce( $_GET['preview_nonce'], 'post_preview_' . $id ) )
    550             wp_die( __('Sorry, you are not allowed to preview drafts.') );
    551 
    552         add_filter('the_preview', '_set_preview');
     583        if ( false === wp_verify_nonce( $_GET['preview_nonce'], 'post_preview_' . $id ) ) {
     584            wp_die( __( 'Sorry, you are not allowed to preview drafts.' ) );
     585        }
     586
     587        add_filter( 'the_preview', '_set_preview' );
    553588    }
    554589}
    … …  
    566601 */
    567602function _wp_preview_terms_filter( $terms, $post_id, $taxonomy ) {
    568     if ( ! $post = get_post() )
     603    if ( ! $post = get_post() ) {
    569604        return $terms;
    570 
    571     if ( empty( $_REQUEST['post_format'] ) || $post->ID != $post_id || 'post_format' != $taxonomy || 'revision' == $post->post_type )
     605    }
     606
     607    if ( empty( $_REQUEST['post_format'] ) || $post->ID != $post_id || 'post_format' != $taxonomy || 'revision' == $post->post_type ) {
    572608        return $terms;
    573 
    574     if ( 'standard' == $_REQUEST['post_format'] )
     609    }
     610
     611    if ( 'standard' == $_REQUEST['post_format'] ) {
    575612        $terms = array();
    576     elseif ( $term = get_term_by( 'slug', 'post-format-' . sanitize_key( $_REQUEST['post_format'] ), 'post_format' ) )
     613    } elseif ( $term = get_term_by( 'slug', 'post-format-' . sanitize_key( $_REQUEST['post_format'] ), 'post_format' ) ) {
    577614        $terms = array( $term ); // Can only have one post format
     615    }
    578616
    579617    return $terms;
    … …  
    624662 */
    625663function _wp_get_post_revision_version( $revision ) {
    626     if ( is_object( $revision ) )
     664    if ( is_object( $revision ) ) {
    627665        $revision = get_object_vars( $revision );
    628     elseif ( !is_array( $revision ) )
     666    } elseif ( ! is_array( $revision ) ) {
    629667        return false;
    630 
    631     if ( preg_match( '/^\d+-(?:autosave|revision)-v(\d+)$/', $revision['post_name'], $matches ) )
     668    }
     669
     670    if ( preg_match( '/^\d+-(?:autosave|revision)-v(\d+)$/', $revision['post_name'], $matches ) ) {
    632671        return (int) $matches[1];
     672    }
    633673
    634674    return 0;
    … …  
    651691
    652692    // Add post option exclusively
    653     $lock = "revision-upgrade-{$post->ID}";
    654     $now = time();
     693    $lock   = "revision-upgrade-{$post->ID}";
     694    $now    = time();
    655695    $result = $wpdb->query( $wpdb->prepare( "INSERT IGNORE INTO `$wpdb->options` (`option_name`, `option_value`, `autoload`) VALUES (%s, %s, 'no') /* LOCK */", $lock, $now ) );
    656696    if ( ! $result ) {
    … …  
    684724
    685725        // Something terrible happened
    686         if ( false === $this_revision_version )
     726        if ( false === $this_revision_version ) {
    687727            continue;
     728        }
    688729
    689730        // 1 is the latest revision version, so we're already up to date.
    … …  
    706747
    707748            // If the previous revision is already up to date, it no longer has the information we need :(
    708             if ( $prev_revision_version < 1 )
     749            if ( $prev_revision_version < 1 ) {
    709750                $update['post_author'] = $prev_revision->post_author;
     751            }
    710752        }
    711753
    … …  
    713755        $result = $wpdb->update( $wpdb->posts, $update, array( 'ID' => $this_revision->ID ) );
    714756
    715         if ( $result )
     757        if ( $result ) {
    716758            wp_cache_delete( $this_revision->ID, 'posts' );
    717 
     759        }
    718760    } while ( $prev_revision );
    719761
    … …  
    721763
    722764    // Add a copy of the post as latest revision.
    723     if ( $add_last )
     765    if ( $add_last ) {
    724766        wp_save_post_revision( $post->ID );
     767    }
    725768
    726769    return true;
Note: See TracChangeset for help on using the changeset viewer.