Changeset 47219 for trunk/src/wp-includes/class-wp-admin-bar.php

Timestamp:

02/09/2020 04:52:28 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Use Yoda conditions where appropriate.

See #49222.

File:

• trunk/src/wp-includes/class-wp-admin-bar.php (modified) (8 diffs)

trunk/src/wp-includes/class-wp-admin-bar.php

@@ -319 +319 @@
 
         foreach ( $this->_get_nodes() as $node ) {
-            if ( 'root' == $node->id ) {
+            if ( 'root' === $node->id ) {
                 continue;
             }
@@ -330 +330 @@
 
             // Generate the group class (we distinguish between top level and other level groups).
-            $group_class = ( $node->parent == 'root' ) ? 'ab-top-menu' : 'ab-submenu';
-
-            if ( $node->type == 'group' ) {
+            $group_class = ( 'root' === $node->parent ) ? 'ab-top-menu' : 'ab-submenu';
+
+            if ( 'group' === $node->type ) {
                 if ( empty( $node->meta['class'] ) ) {
                     $node->meta['class'] = $group_class;
@@ -341 +341 @@
 
             // Items in items aren't allowed. Wrap nested items in 'default' groups.
-            if ( $parent->type == 'item' && $node->type == 'item' ) {
+            if ( 'item' === $parent->type && 'item' === $node->type ) {
                 $default_id = $parent->id . '-default';
                 $default    = $this->_get_node( $default_id );
@@ -370 +370 @@
                 // Groups in groups aren't allowed. Add a special 'container' node.
                 // The container will invisibly wrap both groups.
-            } elseif ( $parent->type == 'group' && $node->type == 'group' ) {
+            } elseif ( 'group' === $parent->type && 'group' === $node->type ) {
                 $container_id = $parent->id . '-container';
                 $container    = $this->_get_node( $container_id );
@@ -399 +399 @@
 
                         $index = array_search( $parent, $grandparent->children, true );
-                        if ( $index === false ) {
+                        if ( false === $index ) {
                             $grandparent->children[] = $container;
                         } else {
@@ -470 +470 @@
      */
     final protected function _render_container( $node ) {
-        if ( $node->type != 'container' || empty( $node->children ) ) {
+        if ( 'container' !== $node->type || empty( $node->children ) ) {
             return;
         }
@@ -485 +485 @@
      */
     final protected function _render_group( $node ) {
-        if ( $node->type == 'container' ) {
+        if ( 'container' === $node->type ) {
             $this->_render_container( $node );
             return;
         }
-        if ( $node->type != 'group' || empty( $node->children ) ) {
+        if ( 'group' !== $node->type || empty( $node->children ) ) {
             return;
         }
@@ -510 +510 @@
      */
     final protected function _render_item( $node ) {
-        if ( $node->type != 'item' ) {
+        if ( 'item' !== $node->type ) {
             return;
         }