From 906456782a8d86e56f91f6a37bd7625e97b0a8c6 Mon Sep 17 00:00:00 2001
From: Michael Green <84688932+michael-j-green@users.noreply.github.com>
Date: Mon, 25 Sep 2023 04:22:27 -0700
Subject: [PATCH] Logs page now has paging (#136)

---
 gaseous-server/Controllers/LogsController.cs  |  4 +-
 .../wwwroot/pages/settings/logs.html          | 55 +++++++++++++------
 gaseous-tools/Logging.cs                      | 20 ++++++-
 3 files changed, 58 insertions(+), 21 deletions(-)

diff --git a/gaseous-server/Controllers/LogsController.cs b/gaseous-server/Controllers/LogsController.cs
index a9800ea8..04b2af07 100644
--- a/gaseous-server/Controllers/LogsController.cs
+++ b/gaseous-server/Controllers/LogsController.cs
@@ -13,9 +13,9 @@ public class LogsController : Controller
     {
         [HttpGet]
         [ProducesResponseType(StatusCodes.Status200OK)]
-        public List<Logging.LogItem> Logs()
+        public List<Logging.LogItem> Logs(long? StartIndex, int PageNumber = 1, int PageSize = 100)
         {
-            return Logging.GetLogs();
+            return Logging.GetLogs(StartIndex, PageNumber, PageSize);
         }
     }
 }
\ No newline at end of file
diff --git a/gaseous-server/wwwroot/pages/settings/logs.html b/gaseous-server/wwwroot/pages/settings/logs.html
index 69f29248..3e6b25df 100644
--- a/gaseous-server/wwwroot/pages/settings/logs.html
+++ b/gaseous-server/wwwroot/pages/settings/logs.html
@@ -7,30 +7,53 @@ <h1 id="gametitle_label">Logs</h1>
     
 </table>
 
+<div style="width: 960px; text-align: center;">
+    <button value="Load More" onclick="loadLogs(lastStartIndex, currentPage);">Load More</button>
+</div>
+
 <script type="text/javascript">
-    function loadLogs() {
+    var lastStartIndex = 0;
+    var currentPage = 1;
+
+    function loadLogs(StartIndex, PageNumber) {
+        var apiQuery = '';
+
+        if (StartIndex && PageNumber) {
+            currentPage += 1;
+            apiQuery = '?StartIndex=' + StartIndex + '&PageNumber=' + PageNumber;
+        } else {
+            currentPage = 1;
+        }
+
         ajaxCall(
-            '/api/v1/Logs',
+            '/api/v1/Logs' + apiQuery,
             'GET',
             function (result) {
                 var newTable = document.getElementById('settings_events_table');
-                newTable.innerHTML = '';
-                newTable.appendChild(
-                    createTableRow(
-                        true, 
-                        [ 
-                            ['Event Time', 'logs_table_cell_150px'],
-                            ['Severity', 'logs_table_cell_150px'],
-                            'Process',
-                            'Message'
-                        ],
-                        '',
-                        ''
-                        )
-                    );
+                if (currentPage == 1) {
+                    newTable.innerHTML = '';
+                
+                    newTable.appendChild(
+                        createTableRow(
+                            true, 
+                            [
+                                //'Id',
+                                ['Event Time', 'logs_table_cell_150px'],
+                                ['Severity', 'logs_table_cell_150px'],
+                                'Process',
+                                'Message'
+                            ],
+                            '',
+                            ''
+                            )
+                        );
+                }
 
                 for (var i = 0; i < result.length; i++) {
+                    lastStartIndex = result[i].id;
+
                     var newRow = [
+                        //result[i].id,
                         moment(result[i].eventTime).fromNow(),
                         result[i].eventType,
                         result[i].process,
diff --git a/gaseous-tools/Logging.cs b/gaseous-tools/Logging.cs
index 849d1132..e0349ffb 100644
--- a/gaseous-tools/Logging.cs
+++ b/gaseous-tools/Logging.cs
@@ -111,17 +111,30 @@ static void LogToDisk(LogItem logItem, string TraceOutput, Exception? exception)
             File.AppendAllText(Config.LogFilePath, TraceOutput);
         }
 
-        static public List<LogItem> GetLogs() 
+        static public List<LogItem> GetLogs(long? StartIndex, int PageNumber = 1, int PageSize = 100) 
         {
             Database db = new gaseous_tools.Database(Database.databaseType.MySql, Config.DatabaseConfiguration.ConnectionString);
-            string sql = "SELECT * FROM ServerLogs ORDER BY Id DESC";
-            DataTable dataTable = db.ExecuteCMD(sql);
+            string sql = "";
+            if (StartIndex == null)
+            {
+                sql = "SELECT * FROM ServerLogs ORDER BY Id DESC LIMIT @PageSize OFFSET @PageNumber;";
+            }
+            else
+            {
+                sql = "SELECT * FROM ServerLogs WHERE Id < @StartIndex ORDER BY Id DESC LIMIT @PageSize OFFSET @PageNumber;";
+            }
+            Dictionary<string, object> dbDict = new Dictionary<string, object>();
+            dbDict.Add("StartIndex", StartIndex);
+            dbDict.Add("PageNumber", (PageNumber - 1) * PageSize);
+            dbDict.Add("PageSize", PageSize);
+            DataTable dataTable = db.ExecuteCMD(sql, dbDict);
 
             List<LogItem> logs = new List<LogItem>();
             foreach (DataRow row in dataTable.Rows)
             {
                 LogItem log = new LogItem
                 {
+                    Id = (long)row["Id"],
                     EventTime = DateTime.Parse(((DateTime)row["EventTime"]).ToString("yyyy-MM-ddThh:mm:ss") + 'Z'),
                     EventType = (LogType)row["EventType"],
                     Process = (string)row["Process"],
@@ -145,6 +158,7 @@ public enum LogType
 
         public class LogItem
         {
+            public long Id { get; set; }
             public DateTime EventTime { get; set; }
             public LogType? EventType { get; set; }
             public string Process { get; set; } = "";