From d280ec197bd667b828e6f56f4ad37a5eb681afa4 Mon Sep 17 00:00:00 2001
From: Marcus Quinn <marcus@agentdesign.co.uk>
Date: Mon, 24 Mar 2025 17:04:22 +0000
Subject: [PATCH] refactor: create Admin Manager class and move menu
 registration

---
 admin/includes/class-admin-manager.php | 35 ++++++++++++++++++++++++++
 admin/settings.php                     | 15 +++++------
 2 files changed, 41 insertions(+), 9 deletions(-)
 create mode 100644 admin/includes/class-admin-manager.php

diff --git a/admin/includes/class-admin-manager.php b/admin/includes/class-admin-manager.php
new file mode 100644
index 0000000..77a7c51
--- /dev/null
+++ b/admin/includes/class-admin-manager.php
@@ -0,0 +1,35 @@
+<?php
+/**
+ * WP ALLSTARS Admin Manager
+ * 
+ * Handles admin-related functionality including menu registration, 
+ * script enqueueing, and admin page rendering.
+ */
+
+if (!defined('ABSPATH')) {
+    exit; // Exit if accessed directly
+}
+
+class WP_Allstars_Admin_Manager {
+    
+    /**
+     * Initialize the class
+     */
+    public static function init() {
+        // Register hooks - we'll add more as we refactor each function
+        add_action('admin_menu', array(__CLASS__, 'register_admin_menu'));
+    }
+    
+    /**
+     * Register the admin menu item
+     */
+    public static function register_admin_menu() {
+        add_options_page(
+            'WP ALLSTARS Settings',
+            'WP ALLSTARS',
+            'manage_options',
+            'wp-allstars',
+            'wp_allstars_settings_page'
+        );
+    }
+}
diff --git a/admin/settings.php b/admin/settings.php
index 5281a88..1ab7682 100644
--- a/admin/settings.php
+++ b/admin/settings.php
@@ -3,17 +3,12 @@
  * Admin settings page
  */
 
-// Add menu item
+// Add menu item - now handled by WP_Allstars_Admin_Manager class, but kept for backward compatibility
 function wp_allstars_admin_menu() {
-    add_options_page(
-        'WP ALLSTARS Settings',
-        'WP ALLSTARS',
-        'manage_options',
-        'wp-allstars',
-        'wp_allstars_settings_page'
-    );
+    // This function now redirects to the Admin Manager class
+    // Kept for backward compatibility
+    WP_Allstars_Admin_Manager::register_admin_menu();
 }
-add_action('admin_menu', 'wp_allstars_admin_menu');
 
 // Register settings
 function wp_allstars_register_settings() {
@@ -49,6 +44,7 @@ require_once dirname(__FILE__) . '/includes/class-tools-manager.php';
 require_once dirname(__FILE__) . '/includes/class-theme-manager.php';
 require_once dirname(__FILE__) . '/includes/class-hosting-manager.php';
 require_once dirname(__FILE__) . '/includes/class-recommended-plugins-manager.php';
+require_once dirname(__FILE__) . '/includes/class-admin-manager.php';
 
 // Initialize the managers
 WP_Allstars_Plugin_Manager::init();
@@ -58,6 +54,7 @@ WP_Allstars_Tools_Manager::init();
 WP_Allstars_Theme_Manager::init();
 WP_Allstars_Hosting_Manager::init();
 WP_Allstars_Recommended_Plugins_Manager::init();
+WP_Allstars_Admin_Manager::init();
 
 
 // Remove the old plugins API filter since we're handling everything in the AJAX endpoint