From 7c1ec4f95709820edf715eaa559e9e39115ca1c0 Mon Sep 17 00:00:00 2001
From: Andrew Bauer <zonexpertconsulting@outlook.com>
Date: Sat, 19 Sep 2020 10:44:05 -0500
Subject: [PATCH] convert logout modal to ajax request

---
 .../classic/views => ajax/modals}/logout.php   |  4 ++++
 web/skins/classic/includes/functions.php       |  2 --
 web/skins/classic/js/skin.js                   | 18 ++++++++++++++++++
 3 files changed, 22 insertions(+), 2 deletions(-)
 rename web/{skins/classic/views => ajax/modals}/logout.php (91%)

diff --git a/web/skins/classic/views/logout.php b/web/ajax/modals/logout.php
similarity index 91%
rename from web/skins/classic/views/logout.php
rename to web/ajax/modals/logout.php
index 86e07605a..565e80cc2 100644
--- a/web/skins/classic/views/logout.php
+++ b/web/ajax/modals/logout.php
@@ -33,6 +33,10 @@ global $CLANG;
       </div>
       <div class="modal-footer">
         <form name="logoutForm" id="logoutForm" method="post" action="?">
+          <?php
+          // We have to manually insert the csrf key into the form when using a modal generated via ajax call
+          echo getCSRFinputHTML();
+          ?>
           <input type="hidden" name="view" value="logout"/>
           <button type="submit" name="action" value="logout"><?php echo translate('Logout') ?></button>
           <?php if ( ZM_USER_SELF_EDIT ) echo '<button type="submit" name="action" value="config">'.translate('Config').'</button>'.PHP_EOL; ?>
diff --git a/web/skins/classic/includes/functions.php b/web/skins/classic/includes/functions.php
index d61c6aaee..584924c2d 100644
--- a/web/skins/classic/includes/functions.php
+++ b/web/skins/classic/includes/functions.php
@@ -702,8 +702,6 @@ function getHeaderFlipHTML() {
 
 // Returns the html representing the logged in user name and avatar
 function getAcctCircleHTML($skin, $user=null) {
-  // Include Logout modal
-  include("skins/$skin/views/logout.php");
   $result = '';
   
   if ( ZM_OPT_USE_AUTH and $user ) {
diff --git a/web/skins/classic/js/skin.js b/web/skins/classic/js/skin.js
index 78399280e..264ddfaff 100644
--- a/web/skins/classic/js/skin.js
+++ b/web/skins/classic/js/skin.js
@@ -312,6 +312,9 @@ if ( currentView != 'none' && currentView != 'login' ) {
   $j.ajaxSetup({timeout: AJAX_TIMEOUT}); //sets timeout for all getJSON.
 
   $j(document).ready(function() {
+    // Load the lgoout modal into the dom
+    getLogoutModal();
+
     // Trigger autorefresh of the widget bar stats on the navbar
     if ( $j('.navbar').length ) {
       setInterval(getNavBar, navBarRefresh);
@@ -746,3 +749,18 @@ function enoperm() {
         console.log("Response Text: " + jqxhr.responseText);
       });
 }
+
+function getLogoutModal() {
+  $j.getJSON(thisUrl + '?request=modal&modal=logout')
+      .done(function(data) {
+        if ( $j('#modalLogout').length ) {
+          $j('#modalLogout').replaceWith(data.html);
+        } else {
+          $j("body").append(data.html);
+        }
+      })
+      .fail(function(jqxhr, textStatus, error) {
+        console.log("Request Failed: " + textStatus + ", " + error);
+        console.log("Response Text: " + jqxhr.responseText);
+      });
+}