Changeset 27745 for trunk/src/wp-includes/class-wp-theme.php

Timestamp:

03/26/2014 06:21:45 PM (12 years ago)

Author:

nacin

Message:

WP_Theme: Return false from the display() method when get() fails.

Fixes a potential warning on the themes page when the active theme is deleted.

fixes #26873.

File:

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

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

@@ -536 +536 @@
          *
          * @param string $header Theme header. Name, Description, Author, Version, ThemeURI, AuthorURI, Status, Tags.
-         * @return string String on success, false on failure.
+         * @return string|bool String on success, false on failure.
          */
         public function get( $header ) {
@@ -572 +572 @@
          * @param bool $markup Optional. Whether to mark up the header. Defaults to true.
          * @param bool $translate Optional. Whether to translate the header. Defaults to true.
-         * @return string Processed header, false on failure.
+         * @return string|bool Processed header, false on failure.
          */
         public function display( $header, $markup = true, $translate = true ) {
                 $value = $this->get( $header );
+                if ( false === $value ) {
+                        return false;
+                }
 
                 if ( $translate && ( empty( $value ) || ! $this->load_textdomain() ) )