From 5352b383489a13a16aa023dc7cf22211b21a19ac Mon Sep 17 00:00:00 2001
From: Marcus Quinn <marcus@agentdesign.co.uk>
Date: Mon, 24 Mar 2025 16:31:46 +0000
Subject: [PATCH] refactor: move Recommended Plugins tab to
 WP_Allstars_Recommended_Plugins_Manager class

---
 .../class-recommended-plugins-manager.php     | 100 ++++++++++++++++++
 admin/settings.php                            |  59 +----------
 2 files changed, 103 insertions(+), 56 deletions(-)
 create mode 100644 admin/includes/class-recommended-plugins-manager.php

diff --git a/admin/includes/class-recommended-plugins-manager.php b/admin/includes/class-recommended-plugins-manager.php
new file mode 100644
index 0000000..b509ed0
--- /dev/null
+++ b/admin/includes/class-recommended-plugins-manager.php
@@ -0,0 +1,100 @@
+<?php
+/**
+ * WP ALLSTARS Recommended Plugins Manager
+ * 
+ * Manages the recommended plugins tab and functionality
+ */
+
+if (!defined('ABSPATH')) {
+    exit; // Exit if accessed directly
+}
+
+class WP_Allstars_Recommended_Plugins_Manager {
+    
+    /**
+     * Initialize the class
+     */
+    public static function init() {
+        // No specific initialization needed currently
+    }
+    
+    /**
+     * Display the recommended plugins tab content
+     */
+    public static function display_tab_content() {
+        // Get the active category from query params or default to 'minimal'
+        $active_category = isset($_GET['category']) ? sanitize_text_field($_GET['category']) : 'minimal';
+        ?>
+        <div id="wpa-plugin-filters" class="wp-filter">
+            <ul class="filter-links">
+                <li><a href="#" data-category="minimal" class="<?php echo $active_category == 'minimal' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Minimal', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="admin" class="<?php echo $active_category == 'admin' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Admin', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="ai" class="<?php echo $active_category == 'ai' ? 'current' : ''; ?>">
+                    <?php esc_html_e('AI', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="cms" class="<?php echo $active_category == 'cms' ? 'current' : ''; ?>">
+                    <?php esc_html_e('CMS', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="compliance" class="<?php echo $active_category == 'compliance' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Compliance', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="crm" class="<?php echo $active_category == 'crm' ? 'current' : ''; ?>">
+                    <?php esc_html_e('CRM', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="ecommerce" class="<?php echo $active_category == 'ecommerce' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Ecommerce', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="lms" class="<?php echo $active_category == 'lms' ? 'current' : ''; ?>">
+                    <?php esc_html_e('LMS', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="media" class="<?php echo $active_category == 'media' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Media', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="seo" class="<?php echo $active_category == 'seo' ? 'current' : ''; ?>">
+                    <?php esc_html_e('SEO', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="setup" class="<?php echo $active_category == 'setup' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Setup', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="social" class="<?php echo $active_category == 'social' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Social', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="speed" class="<?php echo $active_category == 'speed' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Speed', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="translation" class="<?php echo $active_category == 'translation' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Translation', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="advanced" class="<?php echo $active_category == 'advanced' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Advanced', 'wp-allstars'); ?>
+                </a></li>
+                <li><a href="#" data-category="debug" class="<?php echo $active_category == 'debug' ? 'current' : ''; ?>">
+                    <?php esc_html_e('Debug', 'wp-allstars'); ?>
+                </a></li>
+            </ul>
+        </div>
+
+        <div class="wp-allstars-plugin-browser">
+            <div id="wpa-plugin-list"></div>
+        </div>
+        <?php
+    }
+    
+    /**
+     * Get the list of recommended plugins
+     * 
+     * @return array Array of recommended plugins
+     */
+    public static function get_recommended_plugins() {
+        // Define the plugins data if it hasn't been included yet
+        if (!function_exists('wp_allstars_get_recommended_plugins')) {
+            require_once dirname(dirname(__FILE__)) . '/data/recommended-plugins.php';
+        }
+        
+        return wp_allstars_get_recommended_plugins();
+    }
+}
diff --git a/admin/settings.php b/admin/settings.php
index 80b1d59..5281a88 100644
--- a/admin/settings.php
+++ b/admin/settings.php
@@ -48,6 +48,7 @@ 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';
+require_once dirname(__FILE__) . '/includes/class-recommended-plugins-manager.php';
 
 // Initialize the managers
 WP_Allstars_Plugin_Manager::init();
@@ -56,6 +57,7 @@ WP_Allstars_Settings_Manager::init();
 WP_Allstars_Tools_Manager::init();
 WP_Allstars_Theme_Manager::init();
 WP_Allstars_Hosting_Manager::init();
+WP_Allstars_Recommended_Plugins_Manager::init();
 
 
 // Remove the old plugins API filter since we're handling everything in the AJAX endpoint
@@ -334,62 +336,7 @@ function wp_allstars_settings_page() {
                     </div>
                 <?php elseif ($active_tab == 'recommended'): ?>
                     <div class="tab-content" id="recommended">
-                        <div id="wpa-plugin-filters" class="wp-filter">
-                            <ul class="filter-links">
-                                <li><a href="#" data-category="minimal" class="<?php echo $active_category == 'minimal' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Minimal', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="admin" class="<?php echo $active_category == 'admin' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Admin', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="ai" class="<?php echo $active_category == 'ai' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('AI', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="cms" class="<?php echo $active_category == 'cms' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('CMS', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="compliance" class="<?php echo $active_category == 'compliance' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Compliance', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="crm" class="<?php echo $active_category == 'crm' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('CRM', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="ecommerce" class="<?php echo $active_category == 'ecommerce' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Ecommerce', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="lms" class="<?php echo $active_category == 'lms' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('LMS', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="media" class="<?php echo $active_category == 'media' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Media', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="seo" class="<?php echo $active_category == 'seo' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('SEO', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="setup" class="<?php echo $active_category == 'setup' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Setup', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="social" class="<?php echo $active_category == 'social' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Social', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="speed" class="<?php echo $active_category == 'speed' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Speed', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="translation" class="<?php echo $active_category == 'translation' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Translation', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="advanced" class="<?php echo $active_category == 'advanced' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Advanced', 'wp-allstars'); ?>
-                                </a></li>
-                                <li><a href="#" data-category="debug" class="<?php echo $active_category == 'debug' ? 'current' : ''; ?>">
-                                    <?php esc_html_e('Debug', 'wp-allstars'); ?>
-                                </a></li>
-                            </ul>
-                        </div>
-
-                        <div class="wp-allstars-plugin-browser">
-                            <div id="wpa-plugin-list"></div>
-                        </div>
+                        <?php WP_Allstars_Recommended_Plugins_Manager::display_tab_content(); ?>
                     </div>
                 <?php elseif ($active_tab == 'pro'): ?>
                     <div class="tab-content" id="pro">