Updated packages. Fixed some QA issues.

Author: colinmurphy

plugins/wpgraphql-logging/composer.lock

@@ -17,9 +17,9 @@ class CheckboxField extends AbstractSettingsField {
 	/**
 	 * Render the checkbox field.
 	 *
-	 * @param array<string> $option_value The option value.
-	 * @param string        $setting_key  The setting key.
-	 * @param string        $tab_key      The tab key.
+	 * @param array<string, mixed> $option_value The option value.
+	 * @param string               $setting_key  The setting key.
+	 * @param string               $tab_key      The tab key.
 	 *
 	 * @return string The rendered field HTML.
 	 */

plugins/wpgraphql-logging/src/Admin/Settings/Fields/Field/CheckboxField.php

@@ -41,9 +41,9 @@ public function __construct(
 	/**
 	 * Render the select field.
 	 *
-	 * @param array<string> $option_value The option value.
-	 * @param string        $setting_key  The setting key.
-	 * @param string        $tab_key      The tab key.
+	 * @param array<string, mixed> $option_value The option value.
+	 * @param string               $setting_key  The setting key.
+	 * @param string               $tab_key      The tab key.
 	 *
 	 * @return string The rendered field HTML.
 	 */

plugins/wpgraphql-logging/src/Admin/Settings/Fields/Field/SelectField.php

@@ -41,9 +41,9 @@ public function __construct(
 	/**
 	 * Render the text input field.
 	 *
-	 * @param array<string> $option_value The option value.
-	 * @param string        $setting_key  The setting key.
-	 * @param string        $tab_key      The tab key.
+	 * @param array<string, mixed> $option_value The option value.
+	 * @param string               $setting_key  The setting key.
+	 * @param string               $tab_key      The tab key.
 	 *
 	 * @return string The rendered field HTML.
 	 */

plugins/wpgraphql-logging/src/Admin/Settings/Fields/Field/TextInputField.php

@@ -11,61 +11,61 @@
  * Implementing classes must handle rendering, sanitization, and integration with WordPress Settings API.
  *
  * @package WPGraphQL\Logging
+ *
  * @since 0.0.1
  */
 interface SettingsFieldInterface {
+	/**
+	 * Render the settings field.
+	 *
+	 * This should return the HTML for the field. Always escape output using esc_html(), esc_attr(), or wp_kses_post() as appropriate.
+	 *
+	 * @param array<string, mixed> $option_value The option value(s) for this field.
+	 * @param string               $setting_key  The setting key associated with this field.
+	 * @param string               $tab_key      The tab key that this field belongs to.
+	 *
+	 * @return string Rendered HTML for the field.
+	 */
+	public function render_field(array $option_value, string $setting_key, string $tab_key): string;
 
-    /**
-     * Render the settings field.
-     *
-     * This should return the HTML for the field. Always escape output using esc_html(), esc_attr(), or wp_kses_post() as appropriate.
-     *
-     * @param array<string, mixed> $option_value The option value(s) for this field.
-     * @param string               $setting_key  The setting key associated with this field.
-     * @param string               $tab_key      The tab key that this field belongs to.
-     *
-     * @return string Rendered HTML for the field.
-     */
-    public function render_field(array $option_value, string $setting_key, string $tab_key): string;
-
-    /**
-     * Get the unique field ID.
-     *
-     * Must be unique within the tab/page to avoid conflicts.
-     *
-     * @return string The field ID.
-     */
-    public function get_id(): string;
+	/**
+	 * Get the unique field ID.
+	 *
+	 * Must be unique within the tab/page to avoid conflicts.
+	 *
+	 * @return string The field ID.
+	 */
+	public function get_id(): string;
 
-    /**
-     * Determine if the field should render for a specific tab.
-     *
-     * @param string $tab_key The tab key to check.
-     *
-     * @return bool True if the field should render for this tab.
-     */
-    public function should_render_for_tab(string $tab_key): bool;
+	/**
+	 * Determine if the field should render for a specific tab.
+	 *
+	 * @param string $tab_key The tab key to check.
+	 *
+	 * @return bool True if the field should render for this tab.
+	 */
+	public function should_render_for_tab(string $tab_key): bool;
 
-    /**
-     * Add the field to WordPress Settings API.
-     *
-     * Implementing classes should call add_settings_field() internally with proper sanitization callback.
-     *
-     * @param string               $section The settings section ID.
-     * @param string               $page    The settings page ID.
-     * @param array<string, mixed> $args    Additional field arguments.
-     */
-    public function add_settings_field(string $section, string $page, array $args): void;
+	/**
+	 * Add the field to WordPress Settings API.
+	 *
+	 * Implementing classes should call add_settings_field() internally with proper sanitization callback.
+	 *
+	 * @param string               $section The settings section ID.
+	 * @param string               $page    The settings page ID.
+	 * @param array<string, mixed> $args    Additional field arguments.
+	 */
+	public function add_settings_field(string $section, string $page, array $args): void;
 
-    /**
-     * Sanitize the field value.
-     *
-     * Must ensure that all output saved to the database is safe.
-     * For text: use sanitize_text_field(), for HTML: wp_kses_post(), etc.
-     *
-     * @param mixed $value The raw value from user input.
-     *
-     * @return mixed The sanitized value to store in the database.
-     */
-    public function sanitize_field(mixed $value): mixed;
+	/**
+	 * Sanitize the field value.
+	 *
+	 * Must ensure that all output saved to the database is safe.
+	 * For text: use sanitize_text_field(), for HTML: wp_kses_post(), etc.
+	 *
+	 * @param mixed $value The raw value from user input.
+	 *
+	 * @return mixed The sanitized value to store in the database.
+	 */
+	public function sanitize_field(mixed $value): mixed;
 }

plugins/wpgraphql-logging/src/Admin/Settings/Fields/SettingsFieldInterface.php

@@ -4,47 +4,45 @@
 
 namespace WPGraphQL\Logging\Admin\Settings\Fields\Tab;
 
-use WPGraphQL\Logging\Admin\Settings\Fields\SettingsFieldInterface;
-
 /**
  * Interface for settings tabs.
  *
  * Defines the contract for a settings tab that groups related fields.
  * Each tab must provide its metadata and fields for registration in the WordPress Settings API.
  *
  * @package WPGraphQL\Logging
+ *
  * @since 0.0.1
  */
 interface SettingsTabInterface {
+	/**
+	 * Get the settings fields for this tab.
+	 *
+	 * The returned array should be keyed by field ID and contain instances
+	 * implementing SettingsFieldInterface. These fields will be rendered
+	 * and registered automatically in the admin settings page.
+	 *
+	 * @return array<string, \WPGraphQL\Logging\Admin\Settings\Fields\SettingsFieldInterface> Fields keyed by their unique ID.
+	 */
+	public function get_fields(): array;
 
-    /**
-     * Get the settings fields for this tab.
-     *
-     * The returned array should be keyed by field ID and contain instances
-     * implementing SettingsFieldInterface. These fields will be rendered
-     * and registered automatically in the admin settings page.
-     *
-     * @return array<string, SettingsFieldInterface> Fields keyed by their unique ID.
-     */
-    public function get_fields(): array;
-
-    /**
-     * Get the unique name/slug for this tab.
-     *
-     * Must be unique within the plugin to avoid conflicts between tabs.
-     * This name is used in URLs, queries, and as the array key for field storage.
-     *
-     * @return string The tab name/slug.
-     */
-    public static function get_name(): string;
+	/**
+	 * Get the unique name/slug for this tab.
+	 *
+	 * Must be unique within the plugin to avoid conflicts between tabs.
+	 * This name is used in URLs, queries, and as the array key for field storage.
+	 *
+	 * @return string The tab name/slug.
+	 */
+	public static function get_name(): string;
 
-    /**
-     * Get the human-readable label for this tab.
-     *
-     * The label is displayed in the admin UI as the tab title.
-     * Should be internationalized using esc_html__().
-     *
-     * @return string The tab label.
-     */
-    public static function get_label(): string;
+	/**
+	 * Get the human-readable label for this tab.
+	 *
+	 * The label is displayed in the admin UI as the tab title.
+	 * Should be internationalized using esc_html__().
+	 *
+	 * @return string The tab label.
+	 */
+	public static function get_label(): string;
 }

plugins/wpgraphql-logging/src/Admin/Settings/Fields/Tab/SettingsTabInterface.php

@@ -10,7 +10,7 @@
 
 // Include the WP_List_Table class if not already loaded.
 if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php'; // @phpstan-ignore-line
+	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**

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

@@ -363,7 +363,7 @@ public static function get_schema(): string {
 	 * Creates the logging table in the database.
 	 */
 	public static function create_table(): void {
-		require_once ABSPATH . 'wp-admin/includes/upgrade.php'; // @phpstan-ignore-line
+		require_once ABSPATH . 'wp-admin/includes/upgrade.php';
 		dbDelta( self::get_schema() );
 	}
 
@@ -373,7 +373,7 @@ public static function create_table(): void {
 	public static function drop_table(): void {
 		global $wpdb;
 		$table_name = self::get_table_name();
-		$wpdb->query( "DROP TABLE IF EXISTS {$table_name}" ); // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.PreparedSQL.InterpolatedNotPrepared, WordPress.DB.DirectDatabaseQuery.SchemaChange, WordPress.DB.DirectDatabaseQuery.NoCaching
+		$wpdb->query( $wpdb->prepare( 'DROP TABLE IF EXISTS %i', $table_name ) ); // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching, WordPress.DB.DirectDatabaseQuery.SchemaChange
 	}
 
 	/**