Make it safer to access information about registered Action Scheduler versions/sources.

The problem with making this information available via ActionScheduler_Versions is that we don't know which version of that class will be loaded at runtime, therefore we can't safely access newly added methods or properties (or, not directly).

Author: barryhughes

classes/ActionScheduler_AdminView.php

@@ -251,8 +251,8 @@ public function add_help_tabs() {
 		}
 
 		$as_version       = ActionScheduler_Versions::instance()->latest_version();
-		$as_source        = ActionScheduler_Versions::instance()->active_source();
-		$as_source_path   = ActionScheduler_Versions::instance()->active_source_path();
+		$as_source        = ActionScheduler_SystemInformation::active_source();
+		$as_source_path   = ActionScheduler_SystemInformation::active_source_path();
 		$as_source_markup = sprintf( '<code>%s</code>', esc_html( $as_source_path ) );
 
 		if ( ! empty( $as_source ) ) {

classes/ActionScheduler_SystemInformation.php

@@ -0,0 +1,89 @@
+<?php
+
+/**
+ * Provides information about active and registered instances of Action Scheduler.
+ */
+class ActionScheduler_SystemInformation {
+	/**
+	 * Returns information about the plugin or theme which contains the current active version
+	 * of Action Scheduler.
+	 *
+	 * If this cannot be determined, or if Action Scheduler is being loaded via some other
+	 * method, then it will return an empty array. Otherwise, if populated, the array will
+	 * look like the following:
+	 *
+	 *     [
+	 *         'type' => 'plugin', # or 'theme'
+	 *         'name' => 'Name',
+	 *     ]
+	 *
+	 * @return array
+	 */
+	public static function active_source(): array {
+		$plugins      = get_plugins();
+		$plugin_files = array_keys( $plugins );
+
+		foreach ( $plugin_files as $plugin_file ) {
+			$plugin_path = trailingslashit( WP_PLUGIN_DIR ) . dirname( $plugin_file );
+			$plugin_file = trailingslashit( WP_PLUGIN_DIR ) . $plugin_file;
+
+			if ( 0 !== strpos( dirname( __DIR__ ), $plugin_path ) ) {
+				continue;
+			}
+
+			$plugin_data = get_plugin_data( $plugin_file );
+
+			if ( ! is_array( $plugin_data ) || empty( $plugin_data['Name'] ) ) {
+				continue;
+			}
+
+			return array(
+				'type' => 'plugin',
+				'name' => $plugin_data['Name'],
+			);
+		}
+
+		$themes = (array) search_theme_directories();
+
+		foreach ( $themes as $slug => $data ) {
+			$needle = trailingslashit( $data['theme_root'] ) . $slug . '/';
+
+			if ( 0 !== strpos( __FILE__, $needle ) ) {
+				continue;
+			}
+
+			$theme = wp_get_theme( $slug );
+
+			if ( ! is_object( $theme ) || ! is_a( $theme, \WP_Theme::class ) ) {
+				continue;
+			}
+
+			return array(
+				'type' => 'theme',
+				// phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
+				'name' => $theme->Name,
+			);
+		}
+
+		return array();
+	}
+
+	/**
+	 * Returns the directory path for the currently active installation of Action Scheduler.
+	 *
+	 * @return string
+	 */
+	public static function active_source_path(): string {
+		return trailingslashit( dirname( __DIR__ ) );
+	}
+
+	/**
+	 * Get registered sources.
+	 *
+	 * @return array<string, string>
+	 */
+	public static function get_sources() {
+		$versions = ActionScheduler_Versions::instance();
+		return method_exists( $versions, 'get_sources' ) ? $versions->get_sources() : array();
+	}
+}

classes/WP_CLI/System_Command.php

@@ -4,6 +4,7 @@
 
 // phpcs:disable WordPress.Security.EscapeOutput.OutputNotEscaped -- Escaping output is not necessary in WP CLI.
 
+use ActionScheduler_SystemInformation;
 use function \WP_CLI\Utils\get_flag_value;
 
 /**
@@ -99,7 +100,7 @@ public function status( array $args, array $assoc_args ) {
 	 */
 	public function version( array $args, array $assoc_args ) {
 		$all    = (bool) get_flag_value( $assoc_args, 'all' );
-		$latest = $this->get_latest_version( $instance );
+		$latest = $this->get_latest_version();
 
 		if ( ! $all ) {
 			echo $latest;
@@ -139,16 +140,15 @@ public function version( array $args, array $assoc_args ) {
 	 *
 	 * @param array $args       Positional args.
 	 * @param array $assoc_args Keyed args.
-	 * @uses \ActionScheduler_Versions::get_sources()
+	 * @uses ActionScheduler_SystemInformation::active_source_path()
 	 * @uses \WP_CLI\Formatter::display_items()
 	 * @uses $this->get_latest_version()
 	 * @return void
 	 */
 	public function source( array $args, array $assoc_args ) {
 		$all      = (bool) get_flag_value( $assoc_args, 'all' );
 		$fullpath = (bool) get_flag_value( $assoc_args, 'fullpath' );
-		$versions = \ActionScheduler_Versions::instance();
-		$source   = $versions->active_source_path();
+		$source   = ActionScheduler_SystemInformation::active_source_path();
 		$path     = $source;
 
 		if ( ! $fullpath ) {
@@ -160,7 +160,7 @@ public function source( array $args, array $assoc_args ) {
 			\WP_CLI::halt( 0 );
 		}
 
-		$sources = $versions->get_sources();
+		$sources = ActionScheduler_SystemInformation::get_sources();
 		$rows    = array();
 
 		foreach ( $sources as $check_source => $version ) {