From 096c9be284a5f0d160839e327d3a1d39054a90c3 Mon Sep 17 00:00:00 2001 From: Marcus Quinn Date: Mon, 24 Mar 2025 17:07:41 +0000 Subject: [PATCH] refactor: move settings registration to Admin Manager class --- admin/includes/class-admin-manager.php | 9 +++++++++ admin/settings.php | 7 ++++--- 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/admin/includes/class-admin-manager.php b/admin/includes/class-admin-manager.php index 4c2fa68..d710773 100644 --- a/admin/includes/class-admin-manager.php +++ b/admin/includes/class-admin-manager.php @@ -19,6 +19,15 @@ class WP_Allstars_Admin_Manager { // Register hooks - we'll add more as we refactor each function add_action('admin_menu', array(__CLASS__, 'register_admin_menu')); add_action('wp_ajax_wp_allstars_update_option', array(__CLASS__, 'update_option')); + add_action('admin_init', array(__CLASS__, 'register_settings')); + } + + /** + * Register plugin settings + */ + public static function register_settings() { + // Removed minification settings + // This is a placeholder for future settings registration } /** diff --git a/admin/settings.php b/admin/settings.php index 03c9d63..30624d7 100644 --- a/admin/settings.php +++ b/admin/settings.php @@ -10,11 +10,12 @@ function wp_allstars_admin_menu() { WP_Allstars_Admin_Manager::register_admin_menu(); } -// Register settings +// Register settings - now handled by WP_Allstars_Admin_Manager class, but kept for backward compatibility function wp_allstars_register_settings() { - // Removed minification settings + // This function now redirects to the Admin Manager class + // Kept for backward compatibility + WP_Allstars_Admin_Manager::register_settings(); } -add_action('admin_init', 'wp_allstars_register_settings'); // AJAX handler for settings - now handled by WP_Allstars_Admin_Manager class, but kept for backward compatibility function wp_allstars_update_option() {