lint: run phpcbf

Author: theodesp

plugins/wpgraphql-logging/src/Admin/View/List/List_Table.php

@@ -128,7 +128,7 @@ public function process_bulk_action(): void {
 
 		// Nonce action WordPress uses for bulk actions is 'bulk-' . $this->_args['plural']
 		$nonce_action = 'bulk-' . $this->_args['plural'];
-		$nonce_value = $_REQUEST['_wpnonce'] ?? '';
+		$nonce_value  = $_REQUEST['_wpnonce'] ?? '';
 
 		// Ensure nonce is a string for wp_verify_nonce
 		$nonce = is_string( $nonce_value ) ? $nonce_value : '';
@@ -162,7 +162,7 @@ public function process_bulk_action(): void {
 		if ( $deleted_count > 0 ) {
 			// Preserve filters during redirect
 			$preserved_filters = [];
-			$filter_keys = [ 'level_filter', 'start_date', 'end_date' ];
+			$filter_keys       = [ 'level_filter', 'start_date', 'end_date' ];
 
 			foreach ( $filter_keys as $key ) {
 				$value = $_REQUEST[ $key ] ?? null;
@@ -488,10 +488,12 @@ protected function render_custom_filters(): void {
 			__DIR__ . '/../Templates/wpgraphql-logger-filters.php'
 		);
 
-		if ( file_exists( $template ) ) {
-			echo '<div class="alignleft actions">';
-			require $template; // phpcs:ignore WordPressVIPMinimum.Files.IncludingFile.UsingVariable
-			echo '</div>';
+		if ( ! file_exists( $template ) ) {
+			return;
 		}
+
+		echo '<div class="alignleft actions">';
+		require $template; // phpcs:ignore WordPressVIPMinimum.Files.IncludingFile.UsingVariable
+		echo '</div>';
 	}
 }

plugins/wpgraphql-logging/src/Admin/View/Templates/wpgraphql-logger-filters.php

@@ -1,8 +1,9 @@
 <?php
+
 declare(strict_types=1);
 
 if ( ! defined( 'ABSPATH' ) ) {
-    exit;
+	exit;
 }
 
 // Current filter values.
@@ -14,28 +15,28 @@
 ?>
 
 <div class="alignleft actions" style="display: inline-flex; align-items: center; gap: 8px; margin-right: 10px;">
-    <input type="text" 
-           name="start_date" 
-           class="wpgraphql-logging-datepicker" 
-           placeholder="Start Date" 
-           value="<?php echo esc_attr( $current_start_date ); ?>" 
-           style="width: 120px;" />
-    
-    <input type="text" 
-           name="end_date" 
-           class="wpgraphql-logging-datepicker" 
-           placeholder="End Date" 
-           value="<?php echo esc_attr( $current_end_date ); ?>" 
-           style="width: 120px;" />
+	<input type="text" 
+			name="start_date" 
+			class="wpgraphql-logging-datepicker" 
+			placeholder="Start Date" 
+			value="<?php echo esc_attr( $current_start_date ); ?>" 
+			style="width: 120px;" />
+	
+	<input type="text" 
+			name="end_date" 
+			class="wpgraphql-logging-datepicker" 
+			placeholder="End Date" 
+			value="<?php echo esc_attr( $current_end_date ); ?>" 
+			style="width: 120px;" />
 
-    <select name="level_filter" style="width: 120px;">
-        <option value="">All Levels</option>
-        <?php foreach ( $log_levels as $level ) : ?>
-            <option value="<?php echo esc_attr( $level ); ?>" <?php selected( $current_level, $level ); ?>>
-                <?php echo esc_html( ucfirst( $level ) ); ?>
-            </option>
-        <?php endforeach; ?>
-    </select>
+	<select name="level_filter" style="width: 120px;">
+		<option value="">All Levels</option>
+		<?php foreach ( $log_levels as $level ) : ?>
+			<option value="<?php echo esc_attr( $level ); ?>" <?php selected( $current_level, $level ); ?>>
+				<?php echo esc_html( ucfirst( $level ) ); ?>
+			</option>
+		<?php endforeach; ?>
+	</select>
 
-    <?php submit_button( __( 'Filter', 'wpgraphql-logging' ), 'secondary', '', false, [ 'style' => 'margin: 0;' ] ); ?>
-</div>
+	<?php submit_button( __( 'Filter', 'wpgraphql-logging' ), 'secondary', '', false, [ 'style' => 'margin: 0;' ] ); ?>
+</div>

plugins/wpgraphql-logging/src/Logger/Database/LogsRepository.php

@@ -111,8 +111,6 @@ public function delete(int $id): bool {
 
 	/**
 	 * Delete all log entries.
-	 *
-	 * @return void
 	 */
 	public function delete_all(): void {
 		global $wpdb;

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

@@ -50,9 +50,9 @@ protected function is_logging_enabled( array $config, ?string $query_string = nu
 		}
 
 		// Check if the query is an introspection query and skip logging if it is.
-        if ( $is_enabled && $this->is_introspection_query( $query_string ) ) {
-            $is_enabled = false;
-        }
+		if ( $is_enabled && $this->is_introspection_query( $query_string ) ) {
+			$is_enabled = false;
+		}
 
 		/**
 		 * Filter the final decision on whether to log a request.
@@ -64,16 +64,15 @@ protected function is_logging_enabled( array $config, ?string $query_string = nu
 	}
 
 	/**
-     * Checks if a query is an introspection query.
-     *
-     * @param string|null $query_string The GraphQL query string.
-     * @return bool
-     */
-    protected function is_introspection_query( ?string $query_string ): bool {
-        if ( null === $query_string ) {
-            return false;
-        }
+	 * Checks if a query is an introspection query.
+	 *
+	 * @param string|null $query_string The GraphQL query string.
+	 */
+	protected function is_introspection_query( ?string $query_string ): bool {
+		if ( null === $query_string ) {
+			return false;
+		}
 
-        return strpos( $query_string, '__schema' ) !== false;
-    }
+		return strpos( $query_string, '__schema' ) !== false;
+	}
 }