Changeset 52940

Timestamp:

03/16/2022 06:28:46 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Themes: Allow block themes to be activated without index.php.

This removes the requirement for block themes to have an unused index.php template just for activation, as they use a templates/index.html file instead.

The updated requirements are as follows:

• Standalone themes need to have a templates/index.html or index.php template file.
• Child themes need to have a Template header in the style.css stylesheet.

Follow-up to [52069], [52247].

Props poena, sabbirshouvo, scruffian, manfcarlo, overclokk, andraganescu, SergeyBiryukov.
Fixes #54272.

Location:

trunk/src/wp-includes

Files:

• class-wp-theme.php (modified) (1 diff)
• theme.php (modified) (3 diffs)

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

@@ -339 +339 @@
                 if ( ! $this->template ) {
                         $this->template = $this->stylesheet;
-                        if ( ! file_exists( $this->theme_root . '/' . $this->stylesheet . '/index.php' ) ) {
+                        $theme_path     = $this->theme_root . '/' . $this->stylesheet;
+
+                        if ( ! file_exists( $theme_path . '/templates/index.html' )
+                                && ! file_exists( $theme_path . '/index.php' )
+                        ) {
                                 $error_message = sprintf(
-                                        /* translators: 1: index.php, 2: Documentation URL, 3: style.css */
-                                        __( 'Template is missing. Standalone themes need to have a %1$s template file. <a href="%2$s">Child themes</a> need to have a Template header in the %3$s stylesheet.' ),
+                                        /* translators: 1: templates/index.html, 2: index.php, 3: Documentation URL, 4: style.css */
+                                        __( 'Template is missing. Standalone themes need to have a %1$s or %2$s template file. <a href="%3$s">Child themes</a> need to have a Template header in the %4$s stylesheet.' ),
+                                        '<code>templates/index.html</code>',
                                         '<code>index.php</code>',
                                         __( 'https://developer.wordpress.org/themes/advanced-topics/child-themes/' ),

trunk/src/wp-includes/theme.php

@@ -831 +831 @@
 
 /**
- * Checks that the active theme has 'index.php' and 'style.css' files.
+ * Checks that the active theme has the required files.
+ *
+ * Standalone themes need to have a `templates/index.html` or `index.php` template file.
+ * Child themes need to have a `Template` header in the `style.css` stylesheet.
  *
  * Does not initially check the default theme, which is the fallback and should always exist.
@@ -841 +844 @@
  *
  * @since 1.5.0
+ * @since 6.0.0 Removed the requirement for block themes to have an `index.php` template.
  *
  * @see WP_DEFAULT_THEME
@@ -858 +862 @@
         }
 
-        if ( ! file_exists( get_template_directory() . '/index.php' ) ) {
+        if ( ! file_exists( get_template_directory() . '/templates/index.html' )
+                && ! file_exists( get_template_directory() . '/index.php' )
+        ) {
                 // Invalid.
         } elseif ( ! file_exists( get_template_directory() . '/style.css' ) ) {