Changeset 60995

Timestamp:

10/20/2025 10:29:35 PM (12 months ago)

Author:

jorbin

Message:

Plugins and Themes: Add filters to validation functions.

It is now possible to add additional validation requirements for themes and plugins besides the built in core checks.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/1812 and ​https://github.com/WordPress/wordpress-develop/pull/10361.

Props kraftbj, mukesh27, jorbin, joedolson.
Fixes #54381.

Location:

trunk

Files:

• src/wp-admin/includes/plugin.php (modified) (1 diff)
• src/wp-includes/theme.php (modified) (1 diff)
• tests/phpunit/tests/theme.php (modified) (1 diff)

trunk/src/wp-admin/includes/plugin.php

@@ -1248 +1248 @@
         }
 
-        return true;
+        /**
+         * Filters the plugin requirement validation response.
+         *
+         * If a plugin fails due to a Core-provided validation (incompatible WP, PHP versions), this
+         * filter will not fire. A WP_Error response will already be returned.
+         *
+         * This filter is intended to add additional validation steps by site administrators.
+         *
+         * @since 6.9.0
+         *
+         * @param bool|WP_Error $met_requirements True if the plugin meets requirements, WP_Error if not.
+         * @param string $plugin Path to the plugin file relative to the plugins directory.
+         */
+        return apply_filters( 'validate_plugin_requirements', true, $plugin );
 }
 

trunk/src/wp-includes/theme.php

@@ -1003 +1003 @@
         }
 
-        return true;
+        /**
+         * Filters the theme requirement validation response.
+         *
+         * If a theme fails due to a Core-provided validation (incompatible WP, PHP versions), this
+         * filter will not fire. A WP_Error response will already be returned.
+         *
+         * This filter is intended to add additional validation steps by site administrators.
+         *
+         * @since 6.9.0
+         *
+         * @param bool|WP_Error $met_requirements True if the theme meets requirements, WP_Error if not.
+         * @param string $stylesheet Directory name for the theme.
+         */
+        return apply_filters( 'validate_theme_requirements', true, $stylesheet );
 }
 

trunk/tests/phpunit/tests/theme.php

@@ -1530 +1530 @@
                 $this->assertSame( $template_path, $new_template_path, 'Switching blogs switches the template path' );
         }
+
+        /**
+         * Verify the validate_theme_requirements theme responds as expected for twentyten.
+         *
+         * @ticket 54381
+         */
+        public function test_validate_theme_requirements_filter_default() {
+                // Default expectation since twentyten has the least strict requirements.
+                $this->assertTrue( validate_theme_requirements( 'twentyten' ) );
+        }
+
+        /**
+         * Verify that a filtered failure of validate_theme_requirements returns WP_Error
+         *
+         * @ticket 54381
+         */
+        public function test_validate_theme_requirements_filter_error() {
+                // Adds an extra requirement that always fails.
+                add_filter(
+                        'validate_theme_requirements',
+                        function () {
+                                return new WP_Error( 'theme_test_failed_requirement' );
+                        }
+                );
+
+                $this->assertInstanceOf( 'WP_Error', validate_theme_requirements( 'twentyten' ) );
+        }
+
+        /**
+         * Verify that the theme is passed through to the validate_theme_requirements filter by selectively erroring.
+         *
+         * @ticket 54381
+         */
+        public function test_validate_theme_requirements_filter_selective_failure() {
+                // Adds an extra requirement only for a particular theme.
+                add_filter(
+                        'validate_theme_requirements',
+                        function ( $met_requirements, $stylesheet ) {
+                                if ( 'twentytwenty' === $stylesheet ) {
+                                        return new WP_Error( 'theme_test_failed_requirement' );
+                                }
+                                return $met_requirements;
+                        },
+                        10,
+                        2
+                );
+
+                $this->assertTrue( validate_theme_requirements( 'twentyten' ) );
+                $this->assertInstanceOf( 'WP_Error', validate_theme_requirements( 'twentytwenty' ) );
+        }
 }