Changeset 29350 for trunk/src/wp-admin/includes/class-wp-plugin-install-list-table.php

Timestamp:

08/01/2014 09:03:48 PM (12 years ago)

Author:

ocean90

Message:

Add some braces to [29225].

see #28673.

File:

• trunk/src/wp-admin/includes/class-wp-plugin-install-list-table.php (modified) (4 diffs)

trunk/src/wp-admin/includes/class-wp-plugin-install-list-table.php

@@ -182 +182 @@
                 ) );
 
-                if ( isset( $api->info['groups'] ) )
+                if ( isset( $api->info['groups'] ) ) {
                         $this->groups = $api->info['groups'];
+                }
         }
 
@@ -294 +295 @@
 
         public function _order_callback( $plugin_a, $plugin_b ) {
-
                 $orderby = $this->orderby;
-                if ( !isset( $plugin_a->$orderby, $plugin_b->$orderby ) )
+                if ( ! isset( $plugin_a->$orderby, $plugin_b->$orderby ) ) {
                         return 0;
+                }
 
                 $a = $plugin_a->$orderby;
                 $b = $plugin_b->$orderby;
 
-                if ( $a == $b )
+                if ( $a == $b ) {
                         return 0;
-
-                if ( 'DESC' == $this->order )
+                }
+
+                if ( 'DESC' == $this->order ) {
                         return ( $a < $b ) ? 1 : -1;
-                else
+                } else {
                         return ( $a < $b ) ? -1 : 1;
-        }
-
+                }
+        }
 
         public function display_rows() {
@@ -330 +332 @@
 
                 foreach ( (array) $this->items as $plugin ) {
-                        if ( is_object( $plugin ) )
+                        if ( is_object( $plugin ) ) {
                                 $plugin = (array) $plugin;
+                        }
 
                         // Display the group heading if there is one
                         if ( isset( $plugin['group'] ) && $plugin['group'] != $group ) {
-                                if ( isset( $this->groups[ $plugin['group'] ] ) )
+                                if ( isset( $this->groups[ $plugin['group'] ] ) ) {
                                         $group_name = translate( $this->groups[ $plugin['group'] ] ); // Does this need context?
-                                else
+                                } else {
                                         $group_name = $plugin['group'];
+                                }
 
                                 // Starting a new group, close off the divs of the last one
@@ -359 +363 @@
                         $name = strip_tags( $title . ' ' . $version );
 
-                        $author = $plugin['author'];
-
+                        $author = wp_kses( $plugin['author'], $plugins_allowedtags );
                         if ( ! empty( $author ) ) {
                                 $author = ' <cite>' . sprintf( __( 'By %s' ), $author ) . '</cite>';
                         }
-
-                        $author = wp_kses( $author, $plugins_allowedtags );
 
                         $action_links = array();