Fixed formatting issues. Temporarily removed QueryLifecycle class from QA as this is a POC and not actually ready.

Author: colinmurphy

plugins/wpgraphql-logging/deactivation.php

@@ -1,7 +1,4 @@
 <?php
-
-declare(strict_types=1);
-
 /**
  * Deactivation Hook
  *
@@ -10,6 +7,8 @@
  * @since 0.0.1
  */
 
+declare(strict_types=1);
+
 use WPGraphQL\Logging\Plugin;
 
 /**

plugins/wpgraphql-logging/phpcs.xml

@@ -15,6 +15,7 @@
 	<exclude-pattern>*/phpunit.xml*</exclude-pattern>
 	<exclude-pattern>**/tests/**</exclude-pattern>
 	<exclude-pattern>*/vendor/*</exclude-pattern>
+	<exclude-pattern>*/src/Events/QueryEventLifecycle.php</exclude-pattern>
 
 	<!-- ===================================== -->
 	<!-- CLI Arguments & Version Config       -->

plugins/wpgraphql-logging/phpstan.neon.dist

@@ -30,3 +30,5 @@ parameters:
     paths:
         - wpgraphql-logging.php
         - src/
+    excludePaths:
+        - src/Events/QueryEventLifecycle.php

plugins/wpgraphql-logging/psalm.xml

@@ -15,8 +15,12 @@
 		<file name="deactivation.php"/>
 		<file name="vendor/autoload.php"/>
 		<directory name="src"/>
+		<ignoreFiles>
+			<file name="src/Events/QueryEventLifecycle.php"/>
+		</ignoreFiles>
 	</projectFiles>
 
+
 	<plugins>
 		<pluginClass class="PsalmWordPress\Plugin"/>
 	</plugins>

plugins/wpgraphql-logging/src/Logger/LoggerService.php

@@ -44,7 +44,7 @@ class LoggerService {
 	/**
 	 * The instance of the logger based off the channel name.
 	 *
-	 * @var array<LoggerService>
+	 * @var array<\WPGraphQL\Logging\Logger\LoggerService>
 	 */
 	protected static array $instances = [];
 
@@ -90,16 +90,16 @@ public static function get_instance(
 		?array $processors = null,
 		?array $default_context = null
 	): LoggerService {
-		if ( isset(self::$instances[$channel]) ) {
-			return self::$instances[$channel];
+		if ( isset( self::$instances[ $channel ] ) ) {
+			return self::$instances[ $channel ];
 		}
 
 		$processors      = $processors ?? self::get_default_processors();
 		$handlers        = $handlers ?? self::get_default_handlers();
 		$default_context = $default_context ?? self::get_default_context();
 
-		self::$instances[$channel] = new self( $channel, $handlers, $processors, $default_context );
-		return self::$instances[$channel];
+		self::$instances[ $channel ] = new self( $channel, $handlers, $processors, $default_context );
+		return self::$instances[ $channel ];
 	}
 
 	/**

plugins/wpgraphql-logging/src/Plugin.php

@@ -54,6 +54,21 @@ public function setup(): void {
 		QueryEventLifecycle::init();
 	}
 
+	/**
+	 * Activation callback for the plugin.
+	 */
+	public static function activate(): void {
+		DatabaseEntity::create_table();
+	}
+
+	/**
+	 * Deactivation callback for the plugin.
+	 */
+	public static function deactivate(): void {
+		// @TODO: Add configuration to determine if the table should be dropped on deactivation.
+		DatabaseEntity::drop_table();
+	}
+
 	/**
 	 * Throw error on object clone.
 	 * The whole idea of the singleton design pattern is that there is a single object
@@ -77,19 +92,4 @@ public function __wakeup(): void {
 		// De-serializing instances of the class is forbidden.
 		_doing_it_wrong( __METHOD__, 'De-serializing instances of the plugin Main class is not allowed.', '0.0.1' );
 	}
-
-	/**
-	 * Activation callback for the plugin.
-	 */
-	public static function activate(): void {
-		DatabaseEntity::create_table();
-	}
-
-	/**
-	 * Deactivation callback for the plugin.
-	 */
-	public static function deactivate(): void {
-		// @TODO: Add configuration to determine if the table should be dropped on deactivation.
-		DatabaseEntity::drop_table();
-	}
 }

plugins/wpgraphql-logging/wpgraphql-logging.php

@@ -140,6 +140,6 @@ function wpgraphql_logging_load_textdomain(): void {
 add_action( 'init', 'wpgraphql_logging_load_textdomain', 1, 0 );
 
 /** @psalm-suppress HookNotFound */
-add_action( 'plugins_loaded', function() : void {
+add_action( 'plugins_loaded', static function (): void {
 	wpgraphql_logging_init();
- }, 10, 0 );
+}, 10, 0 );