From 0b66c8e1be4e248ebba670bb288b27997bf316b9 Mon Sep 17 00:00:00 2001
From: johnnyq <johnny@pittpc.com>
Date: Thu, 29 May 2025 13:29:27 -0400
Subject: [PATCH] Better naming of Indetity Providers instead of Integrations
 to make way for more Indentity Provider like Google and custom SSO for the
 future

---
 ...gration.php => admin_identity_provider.php | 21 ++++++++++++++++---
 includes/admin_side_nav.php                   |  8 +++----
 ...ration.php => admin_identity_provider.php} |  6 +++---
 3 files changed, 25 insertions(+), 10 deletions(-)
 rename admin_settings_integration.php => admin_identity_provider.php (59%)
 rename post/admin/{admin_settings_integration.php => admin_identity_provider.php} (71%)

diff --git a/admin_settings_integration.php b/admin_identity_provider.php
similarity index 59%
rename from admin_settings_integration.php
rename to admin_identity_provider.php
index 408a61ef..2149815d 100644
--- a/admin_settings_integration.php
+++ b/admin_identity_provider.php
@@ -4,13 +4,29 @@ require_once "includes/inc_all_admin.php";
 
 <div class="card card-dark">
     <div class="card-header py-3">
-        <h3 class="card-title"><i class="fas fa-fw fa-plug mr-2"></i>Integration Settings</h3>
+        <h3 class="card-title"><i class="fas fa-fw fa-fingerprint mr-2"></i>Identity Providers</h3>
     </div>
     <div class="card-body">
         <form action="post.php" method="post" autocomplete="off">
             <input type="hidden" name="csrf_token" value="<?php echo $_SESSION['csrf_token'] ?>">
 
             <h4>Client Portal SSO via Microsoft Entra</h4>
+
+            <div class="form-group">
+                <label>Identity Provider <small class='text-secondary'>(Currently only works with Microsft Entra)</small></label>
+                <div class="input-group">
+                    <div class="input-group-prepend">
+                        <span class="input-group-text"><i class="fa fa-fw fa-fingerprint"></i></span>
+                    </div>
+                    <select class="form-control select2" readonly>
+                        <option <?php if (empty($config_azure_client_id)) { echo "selected"; } ?>>Disabled</option>
+                        <option <?php if ($config_azure_client_id) { echo "selected"; } ?>>Microsoft Entra</option>
+                        <option>Google (WIP)</option>
+                        <option>Custom SSO (WIP)</option>
+                    </select>
+                </div>
+            </div>
+
             <div class="form-group">
                 <label>MS Entra OAuth App (Client) ID</label>
                 <div class="input-group">
@@ -33,11 +49,10 @@ require_once "includes/inc_all_admin.php";
 
             <hr>
 
-            <button type="submit" name="edit_integrations_settings" class="btn btn-primary text-bold"><i class="fa fa-check mr-2"></i>Save</button>
+            <button type="submit" name="edit_identity_provider" class="btn btn-primary text-bold"><i class="fa fa-check mr-2"></i>Save</button>
 
         </form>
     </div>
 </div>
 
 <?php require_once "includes/footer.php";
-
diff --git a/includes/admin_side_nav.php b/includes/admin_side_nav.php
index 02e83152..f1b870e0 100644
--- a/includes/admin_side_nav.php
+++ b/includes/admin_side_nav.php
@@ -145,7 +145,7 @@
                 </li>
 
                 <!-- SETTINGS Section -->
-                <li class="nav-item has-treeview mt-2 <?php echo (in_array(basename($_SERVER['PHP_SELF']), ['admin_settings_company.php', 'admin_settings_localization.php', 'admin_settings_theme.php', 'admin_settings_security.php', 'admin_settings_mail.php', 'admin_settings_notification.php', 'admin_settings_default.php', 'admin_settings_invoice.php', 'admin_settings_quote.php', 'admin_settings_online_payment.php', 'admin_settings_online_payment_clients.php', 'admin_settings_project.php', 'admin_settings_ticket.php', 'admin_settings_ai.php', 'admin_settings_integration.php', 'admin_settings_telemetry.php', 'admin_settings_module.php']) ? 'menu-open' : ''); ?>">
+                <li class="nav-item has-treeview mt-2 <?php echo (in_array(basename($_SERVER['PHP_SELF']), ['admin_settings_company.php', 'admin_settings_localization.php', 'admin_settings_theme.php', 'admin_settings_security.php', 'admin_settings_mail.php', 'admin_settings_notification.php', 'admin_settings_default.php', 'admin_settings_invoice.php', 'admin_settings_quote.php', 'admin_settings_online_payment.php', 'admin_settings_online_payment_clients.php', 'admin_settings_project.php', 'admin_settings_ticket.php', 'admin_settings_ai.php', 'admin_identity_provider.php', 'admin_settings_telemetry.php', 'admin_settings_module.php']) ? 'menu-open' : ''); ?>">
                     <a href="#" class="nav-link">
                         <p>
                             SETTINGS
@@ -246,9 +246,9 @@
                         <!-- Currently the only integration is the client portal SSO -->
                         <?php if ($config_client_portal_enable) { ?>
                             <li class="nav-item">
-                                <a href="admin_settings_integration.php" class="nav-link <?php echo (basename($_SERVER['PHP_SELF']) == 'admin_settings_integration.php' ? 'active' : ''); ?>">
-                                    <i class="nav-icon fas fa-plug"></i>
-                                    <p>Integrations</p>
+                                <a href="admin_identity_provider.php" class="nav-link <?php echo (basename($_SERVER['PHP_SELF']) == 'admin_identity_provider.php' ? 'active' : ''); ?>">
+                                    <i class="nav-icon fas fa-fingerprint"></i>
+                                    <p>Identity Provider</p>
                                 </a>
                             </li>
                         <?php } ?>
diff --git a/post/admin/admin_settings_integration.php b/post/admin/admin_identity_provider.php
similarity index 71%
rename from post/admin/admin_settings_integration.php
rename to post/admin/admin_identity_provider.php
index c2f5d363..331754a4 100644
--- a/post/admin/admin_settings_integration.php
+++ b/post/admin/admin_identity_provider.php
@@ -2,7 +2,7 @@
 
 defined('FROM_POST_HANDLER') || die("Direct file access is not allowed");
 
-if (isset($_POST['edit_integrations_settings'])) {
+if (isset($_POST['edit_identity_provider'])) {
 
     validateCSRFToken($_POST['csrf_token']);
 
@@ -12,9 +12,9 @@ if (isset($_POST['edit_integrations_settings'])) {
     mysqli_query($mysqli,"UPDATE settings SET config_azure_client_id = '$azure_client_id', config_azure_client_secret = '$azure_client_secret' WHERE company_id = 1");
 
     // Logging
-    logAction("Settings", "Edit", "$session_name edited integrations settings");
+    logAction("Settings", "Edit", "$session_name edited identity provider settings");
 
-    $_SESSION['alert_message'] = "Integrations Settings updated";
+    $_SESSION['alert_message'] = "Identity Provider Settings updated";
 
     header("Location: " . $_SERVER["HTTP_REFERER"]);