Changeset 42343 for trunk/src/wp-includes/class-wp-post.php

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:

• trunk/src/wp-includes/class-wp-post.php (modified) (9 diffs)

trunk/src/wp-includes/class-wp-post.php

@@ -18 +18 @@
  * @property-read int    $post_category
  * @property-read string $tag_input
- *
  */
 final class WP_Post {
@@ -244 +243 @@
             $_post = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->posts WHERE ID = %d LIMIT 1", $post_id ) );
 
-            if ( ! $_post )
+            if ( ! $_post ) {
                 return false;
+            }
 
             $_post = sanitize_post( $_post, 'raw' );
@@ -264 +264 @@
      */
     public function __construct( $post ) {
-        foreach ( get_object_vars( $post ) as $key => $value )
+        foreach ( get_object_vars( $post ) as $key => $value ) {
             $this->$key = $value;
+        }
     }
 
@@ -277 +278 @@
      */
     public function __isset( $key ) {
-        if ( 'ancestors' == $key )
+        if ( 'ancestors' == $key ) {
             return true;
-
-        if ( 'page_template' == $key )
+        }
+
+        if ( 'page_template' == $key ) {
             return true;
-
-        if ( 'post_category' == $key )
+        }
+
+        if ( 'post_category' == $key ) {
             return true;
-
-        if ( 'tags_input' == $key )
+        }
+
+        if ( 'tags_input' == $key ) {
             return true;
+        }
 
         return metadata_exists( 'post', $this->ID, $key );
@@ -306 +311 @@
 
         if ( 'post_category' == $key ) {
-            if ( is_object_in_taxonomy( $this->post_type, 'category' ) )
+            if ( is_object_in_taxonomy( $this->post_type, 'category' ) ) {
                 $terms = get_the_terms( $this, 'category' );
-
-            if ( empty( $terms ) )
+            }
+
+            if ( empty( $terms ) ) {
                 return array();
+            }
 
             return wp_list_pluck( $terms, 'term_id' );
@@ -316 +323 @@
 
         if ( 'tags_input' == $key ) {
-            if ( is_object_in_taxonomy( $this->post_type, 'post_tag' ) )
+            if ( is_object_in_taxonomy( $this->post_type, 'post_tag' ) ) {
                 $terms = get_the_terms( $this, 'post_tag' );
-
-            if ( empty( $terms ) )
+            }
+
+            if ( empty( $terms ) ) {
                 return array();
+            }
 
             return wp_list_pluck( $terms, 'name' );
@@ -326 +335 @@
 
         // Rest of the values need filtering.
-        if ( 'ancestors' == $key )
+        if ( 'ancestors' == $key ) {
             $value = get_post_ancestors( $this );
-        else
+        } else {
             $value = get_post_meta( $this->ID, $key, true );
-
-        if ( $this->filter )
+        }
+
+        if ( $this->filter ) {
             $value = sanitize_post_field( $key, $value, $this->ID, $this->filter );
+        }
 
         return $value;
@@ -346 +357 @@
      */
     public function filter( $filter ) {
-        if ( $this->filter == $filter )
+        if ( $this->filter == $filter ) {
             return $this;
-
-        if ( $filter == 'raw' )
+        }
+
+        if ( $filter == 'raw' ) {
             return self::get_instance( $this->ID );
+        }
 
         return sanitize_post( $this, $filter );
@@ -366 +379 @@
 
         foreach ( array( 'ancestors', 'page_template', 'post_category', 'tags_input' ) as $key ) {
-            if ( $this->__isset( $key ) )
+            if ( $this->__isset( $key ) ) {
                 $post[ $key ] = $this->__get( $key );
+            }
         }