Allow module can-load.php callbacks to return a WP_Error with more information

admin/load.php

@@ -141,7 +141,7 @@ function perflab_render_modules_page_field( $module_slug, $module_data, $module_
 			<?php echo esc_html( $module_data['name'] ); ?>
 		</legend>
 		<label for="<?php echo esc_attr( "{$base_id}_enabled" ); ?>">
-			<?php if ( $can_load_module && ! $is_standalone_plugin_loaded ) { ?>
+			<?php if ( ! is_wp_error( $can_load_module ) && ! $is_standalone_plugin_loaded ) { ?>
 				<input type="checkbox" id="<?php echo esc_attr( "{$base_id}_enabled" ); ?>" name="<?php echo esc_attr( "{$base_name}[enabled]" ); ?>" aria-describedby="<?php echo esc_attr( "{$base_id}_description" ); ?>" value="1"<?php checked( $enabled ); ?>>
 				<?php
 				if ( $module_data['experimental'] ) {
@@ -167,6 +167,8 @@ function perflab_render_modules_page_field( $module_slug, $module_data, $module_
 				<?php
 				if ( $is_standalone_plugin_loaded ) {
 					esc_html_e( 'The module cannot be managed with Performance Lab since it is already active as a standalone plugin.', 'performance-lab' );
+				} elseif ( is_wp_error( $can_load_module ) ) {
+					echo esc_html( $can_load_module->get_error_message() );
 				} else {
 					printf(
 						/* translators: %s: module name */

load.php

@@ -197,7 +197,8 @@ function perflab_is_valid_module( $module ) {
 	}
 
 	// Do not load module if it cannot be loaded, e.g. if it was already merged and is available in WordPress core.
-	return perflab_can_load_module( $module );
+	$can_load_module = perflab_can_load_module( $module );
+	return is_wp_error( $can_load_module ) ? false : $can_load_module;
 }
 
 /**
@@ -237,7 +238,7 @@ function perflab_render_generator() {
  * @since 1.3.0
  *
  * @param string $module Slug of the module.
- * @return bool Whether the module can be loaded or not.
+ * @return bool|WP_Error True or false based on whether the module can be loaded or not, or an error on failure.
  */
 function perflab_can_load_module( $module ) {
 	$module_load_file = PERFLAB_PLUGIN_DIR_PATH . 'modules/' . $module . '/can-load.php';
@@ -255,6 +256,11 @@ function perflab_can_load_module( $module ) {
 		return true;
 	}
 
+	// Check if can load return an error.
+	if ( is_wp_error( $module() ) ) {
+		return $module();
+	}
+
 	// Call the closure to determine whether the module can be loaded.
 	return (bool) $module();
 }

modules/database/audit-autoloaded-options/can-load.php

@@ -0,0 +1,19 @@
+<?php
+/**
+ * Can load function to determine if Site Health module is supported or not.
+ *
+ * @since   n.e.x.t
+ * @package performance-lab
+ */
+
+if ( ! defined( 'ABSPATH' ) ) {
+	exit; // Exit if accessed directly.
+}
+
+return static function () {
+	if ( ! has_filter( 'user_has_cap', 'wp_maybe_grant_site_health_caps' ) ) {
+		return new WP_Error( 'module_not_loaded', esc_html__( 'The module cannot be loaded with Performance Lab since it is disabled.', 'performance-lab' ) );
+	} else {
+		return true;
+	}
+};

modules/images/webp-support/can-load.php

@@ -0,0 +1,19 @@
+<?php
+/**
+ * Can load function to determine if Site Health module is supported or not.
+ *
+ * @since   n.e.x.t
+ * @package performance-lab
+ */
+
+if ( ! defined( 'ABSPATH' ) ) {
+	exit; // Exit if accessed directly.
+}
+
+return static function () {
+	if ( ! has_filter( 'user_has_cap', 'wp_maybe_grant_site_health_caps' ) ) {
+		return new WP_Error( 'module_not_loaded', esc_html__( 'The module cannot be loaded with Performance Lab since it is disabled.', 'performance-lab' ) );
+	} else {
+		return true;
+	}
+};

modules/js-and-css/audit-enqueued-assets/can-load.php

@@ -0,0 +1,19 @@
+<?php
+/**
+ * Can load function to determine if Site Health module is supported or not.
+ *
+ * @since   n.e.x.t
+ * @package performance-lab
+ */
+
+if ( ! defined( 'ABSPATH' ) ) {
+	exit; // Exit if accessed directly.
+}
+
+return static function () {
+	if ( ! has_filter( 'user_has_cap', 'wp_maybe_grant_site_health_caps' ) ) {
+		return new WP_Error( 'module_not_loaded', esc_html__( 'The module cannot be loaded with Performance Lab since it is disabled.', 'performance-lab' ) );
+	} else {
+		return true;
+	}
+};

tests/admin/load-tests.php

@@ -32,6 +32,13 @@ class Admin_Load_Tests extends WP_UnitTestCase {
 			'focus'        => 'images',
 			'slug'         => 'demo-module-3',
 		),
+		'check-error/demo-module-4' => array(
+			'name'         => 'Demo Module 4',
+			'description'  => 'This is the description for demo module 4.',
+			'experimental' => false,
+			'focus'        => 'check-error',
+			'slug'         => 'demo-module-4',
+		),
 	);
 
 	private static $demo_focus_areas = array(
@@ -157,7 +164,7 @@ public function test_perflab_load_modules_page() {
 			array_keys( $wp_settings_fields[ PERFLAB_MODULES_SCREEN ]['js-and-css'] )
 		);
 		$this->assertEqualSets(
-			array( 'something/demo-module-2' ),
+			array( 'something/demo-module-2', 'check-error/demo-module-4' ),
 			array_keys( $wp_settings_fields[ PERFLAB_MODULES_SCREEN ]['other'] )
 		);
 	}

tests/load-tests.php

@@ -196,6 +196,7 @@ public function data_perflab_is_valid_module() {
 			array( '../tests/testdata/demo-modules/js-and-css/demo-module-1', false ),
 			array( '../tests/testdata/demo-modules/something/demo-module-2', true ),
 			array( '../tests/testdata/demo-modules/images/demo-module-3', true ),
+			array( '../tests/testdata/demo-modules/check-error/demo-module-4', false ),
 		);
 	}
 
@@ -214,6 +215,12 @@ public function data_perflab_can_load_module() {
 		);
 	}
 
+	public function test_perflab_can_load_module_with_not_loaded_module() {
+		$can_load_module = perflab_can_load_module( '../tests/testdata/demo-modules/check-error/demo-module-4' );
+		$this->assertInstanceOf( 'WP_Error', $can_load_module );
+		$this->assertSame( 'module_not_loaded', $can_load_module->get_error_code() );
+	}
+
 	public function test_perflab_activate_module() {
 		perflab_activate_module( __DIR__ . '/testdata/demo-modules/something/demo-module-2' );
 		$this->assertSame( 'activated', get_option( 'test_demo_module_activation_status' ) );

tests/testdata/demo-modules/check-error/demo-module-4/can-load.php

@@ -0,0 +1,10 @@
+<?php
+/**
+ * Can load function to determine if Site Health module is supported or not.
+ *
+ * @package performance-lab
+ */
+
+return static function () {
+    return new WP_Error( 'module_not_loaded', esc_html__( 'The module cannot be loaded with Performance Lab since it is disabled.', 'performance-lab' ) );
+};

tests/testdata/demo-modules/check-error/demo-module-4/load.php

@@ -0,0 +1,10 @@
+<?php
+/**
+ * Module Name: Demo Module 4
+ * Description: This is the description for demo module 4.
+ * Experimental: No
+ *
+ * @package performance-lab
+ */
+
+// This is a demo module and does nothing.