From 2405f98eb8abcf860794da2f9801bcb686672518 Mon Sep 17 00:00:00 2001
From: Marcus Quinn <marcus@agentdesign.co.uk>
Date: Mon, 24 Mar 2025 16:17:31 +0000
Subject: [PATCH] refactor: move Hosting tab functionality to
 WP_Allstars_Hosting_Manager class

---
 admin/includes/class-hosting-manager.php | 68 ++++++++++++++++++++++++
 admin/settings.php                       | 29 ++--------
 2 files changed, 71 insertions(+), 26 deletions(-)
 create mode 100644 admin/includes/class-hosting-manager.php

diff --git a/admin/includes/class-hosting-manager.php b/admin/includes/class-hosting-manager.php
new file mode 100644
index 0000000..241944a
--- /dev/null
+++ b/admin/includes/class-hosting-manager.php
@@ -0,0 +1,68 @@
+<?php
+/**
+ * WP ALLSTARS Hosting Manager
+ * 
+ * Manages the hosting providers tab and functionality
+ */
+
+if (!defined('ABSPATH')) {
+    exit; // Exit if accessed directly
+}
+
+class WP_Allstars_Hosting_Manager {
+    
+    /**
+     * Initialize the class
+     */
+    public static function init() {
+        // No specific initialization needed currently
+    }
+    
+    /**
+     * Display the hosting tab content
+     */
+    public static function display_tab_content() {
+        ?>
+        <div class="wpa-pro-plugins">
+            <?php
+            $hosting_providers = self::get_hosting_providers();
+            // Sort providers alphabetically by name
+            uasort($hosting_providers, function($a, $b) {
+                return strcasecmp($a['name'], $b['name']);
+            });
+            foreach ($hosting_providers as $provider) {
+                ?>
+                <div class="wpa-pro-plugin">
+                    <h3><?php echo esc_html($provider['name']); ?></h3>
+                    <p><?php echo esc_html($provider['description']); ?></p>
+                    <?php if (isset($provider['button_group'])): ?>
+                        <div class="button-group">
+                            <?php foreach ($provider['button_group'] as $button): ?>
+                                <a href="<?php echo esc_url($button['url']); ?>" target="_blank" class="button <?php echo isset($button['primary']) && $button['primary'] ? 'button-primary' : ''; ?>">
+                                    <?php echo esc_html($button['text']); ?>
+                                </a>
+                            <?php endforeach; ?>
+                        </div>
+                    <?php endif; ?>
+                </div>
+                <?php
+            }
+            ?>
+        </div>
+        <?php
+    }
+    
+    /**
+     * Get the list of hosting providers
+     * 
+     * @return array Array of hosting providers
+     */
+    public static function get_hosting_providers() {
+        // Define the providers data if it hasn't been included yet
+        if (!function_exists('wp_allstars_get_hosting_providers')) {
+            require_once dirname(dirname(__FILE__)) . '/data/hosting-providers.php';
+        }
+        
+        return wp_allstars_get_hosting_providers();
+    }
+}
diff --git a/admin/settings.php b/admin/settings.php
index 43e3078..80b1d59 100644
--- a/admin/settings.php
+++ b/admin/settings.php
@@ -47,6 +47,7 @@ require_once dirname(__FILE__) . '/includes/class-pro-plugins-manager.php';
 require_once dirname(__FILE__) . '/includes/class-settings-manager.php';
 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';
 
 // Initialize the managers
 WP_Allstars_Plugin_Manager::init();
@@ -54,6 +55,7 @@ WP_Allstars_Pro_Plugins_Manager::init();
 WP_Allstars_Settings_Manager::init();
 WP_Allstars_Tools_Manager::init();
 WP_Allstars_Theme_Manager::init();
+WP_Allstars_Hosting_Manager::init();
 
 
 // Remove the old plugins API filter since we're handling everything in the AJAX endpoint
@@ -328,32 +330,7 @@ function wp_allstars_settings_page() {
                     </div>
                 <?php elseif ($active_tab == 'hosting'): ?>
                      <div class="tab-content" id="hosting">
-                        <div class="wpa-pro-plugins">
-                            <?php
-                            $hosting_providers = wp_allstars_get_hosting_providers();
-                            // Sort providers alphabetically by name
-                            uasort($hosting_providers, function($a, $b) {
-                                return strcasecmp($a['name'], $b['name']);
-                            });
-                            foreach ($hosting_providers as $provider) {
-                                ?>
-                                <div class="wpa-pro-plugin">
-                                    <h3><?php echo esc_html($provider['name']); ?></h3>
-                                    <p><?php echo esc_html($provider['description']); ?></p>
-                                    <?php if (isset($provider['button_group'])): ?>
-                                        <div class="button-group">
-                                            <?php foreach ($provider['button_group'] as $button): ?>
-                                                <a href="<?php echo esc_url($button['url']); ?>" target="_blank" class="button <?php echo isset($button['primary']) && $button['primary'] ? 'button-primary' : ''; ?>">
-                                                    <?php echo esc_html($button['text']); ?>
-                                                </a>
-                                            <?php endforeach; ?>
-                                        </div>
-                                    <?php endif; ?>
-                                </div>
-                                <?php
-                            }
-                            ?>
-                        </div>
+                        <?php WP_Allstars_Hosting_Manager::display_tab_content(); ?>
                     </div>
                 <?php elseif ($active_tab == 'recommended'): ?>
                     <div class="tab-content" id="recommended">