Changeset 57847

Timestamp:

03/16/2024 01:16:16 PM (2 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Use strict comparison in wp-includes/class-wp-theme.php.

Follow-up to [20029], [20119], [20144].

Props aristath, poena, afercia, SergeyBiryukov.
See #60700.

File:

• trunk/src/wp-includes/class-wp-theme.php (modified) (6 diffs)

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

@@ -353 +353 @@
                         $default_theme_slug = array_search( $this->headers['Name'], self::$default_themes, true );
                         if ( $default_theme_slug ) {
-                                if ( basename( $this->stylesheet ) != $default_theme_slug ) {
+                                if ( basename( $this->stylesheet ) !== $default_theme_slug ) {
                                         $this->headers['Name'] .= '/' . $this->stylesheet;
                                 }
@@ -418 +418 @@
 
                 // If we got our data from cache, we can assume that 'template' is pointing to the right place.
-                if ( ! is_array( $cache ) && $this->template != $this->stylesheet && ! file_exists( $this->theme_root . '/' . $this->template . '/index.php' ) ) {
+                if ( ! is_array( $cache )
+                        && $this->template !== $this->stylesheet
+                        && ! file_exists( $this->theme_root . '/' . $this->template . '/index.php' )
+                ) {
                         /*
                          * If we're in a directory of themes inside /themes, look for the parent nearby.
@@ -426 +429 @@
                         $directories = search_theme_directories();
 
-                        if ( '.' !== $parent_dir && file_exists( $this->theme_root . '/' . $parent_dir . '/' . $this->template . '/index.php' ) ) {
+                        if ( '.' !== $parent_dir
+                                && file_exists( $this->theme_root . '/' . $parent_dir . '/' . $this->template . '/index.php' )
+                        ) {
                                 $this->template = $parent_dir . '/' . $this->template;
                         } elseif ( $directories && isset( $directories[ $this->template ] ) ) {
@@ -461 +466 @@
 
                 // Set the parent, if we're a child theme.
-                if ( $this->template != $this->stylesheet ) {
+                if ( $this->template !== $this->stylesheet ) {
                         // If we are a parent, then there is a problem. Only two generations allowed! Cancel things out.
-                        if ( $_child instanceof WP_Theme && $_child->template == $this->stylesheet ) {
+                        if ( $_child instanceof WP_Theme && $_child->template === $this->stylesheet ) {
                                 $_child->parent = null;
                                 $_child->errors = new WP_Error(
@@ -485 +490 @@
                                 );
                                 // The two themes actually reference each other with the Template header.
-                                if ( $_child->stylesheet == $this->template ) {
+                                if ( $_child->stylesheet === $this->template ) {
                                         $this->errors = new WP_Error(
                                                 'theme_parent_invalid',
@@ -1715 +1720 @@
                 }
 
-                $current = get_current_blog_id() == $blog_id;
+                $current = get_current_blog_id() === $blog_id;
 
                 if ( $current ) {