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

Timestamp:

03/26/2014 06:21:45 PM (11 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() ) )