From e9c74be057201f1b04180f758237c462602d8423 Mon Sep 17 00:00:00 2001
From: amcmanu3 <mcdeweykp@gmail.com>
Date: Sun, 4 Sep 2022 16:21:33 -0400
Subject: [PATCH 1/2] Add ability to change metrics scale

---
 app/classes/controllers/servers_controller.py    |  5 ++---
 app/classes/models/server_stats.py               |  4 ++--
 app/classes/web/panel_handler.py                 | 15 ++++++++++++++-
 app/frontend/templates/panel/server_metrics.html | 15 +++++++++++++++
 4 files changed, 33 insertions(+), 6 deletions(-)

diff --git a/app/classes/controllers/servers_controller.py b/app/classes/controllers/servers_controller.py
index c9434b13..1bf81564 100644
--- a/app/classes/controllers/servers_controller.py
+++ b/app/classes/controllers/servers_controller.py
@@ -105,10 +105,9 @@ class ServersController(metaclass=Singleton):
         server_instance.update_server_instance()
         return ret
 
-    def get_history_stats(self, server_id):
-        now = datetime.datetime.now()
+    def get_history_stats(self, server_id, days):
         srv = ServersController().get_server_instance_by_id(server_id)
-        return srv.stats_helper.get_history_stats(server_id)
+        return srv.stats_helper.get_history_stats(server_id, days)
 
     @staticmethod
     def update_unloaded_server(server_obj):
diff --git a/app/classes/models/server_stats.py b/app/classes/models/server_stats.py
index d6c6efdc..29fdb856 100644
--- a/app/classes/models/server_stats.py
+++ b/app/classes/models/server_stats.py
@@ -138,8 +138,8 @@ class HelperServerStats:
             )
         return server_data
 
-    def get_history_stats(self, server_id):
-        max_age = datetime.datetime.now() - timedelta(days=1)
+    def get_history_stats(self, server_id, num_days):
+        max_age = datetime.datetime.now() - timedelta(days=num_days)
         return (
             ServerStats.select()
             .where(ServerStats.created > max_age)
diff --git a/app/classes/web/panel_handler.py b/app/classes/web/panel_handler.py
index a8bac6e2..a30f7d29 100644
--- a/app/classes/web/panel_handler.py
+++ b/app/classes/web/panel_handler.py
@@ -756,8 +756,21 @@ class PanelHandler(BaseHandler):
                 page_data["backup_path"] = Helpers.wtol_path(server_info["backup_path"])
 
             if subpage == "metrics":
+                try:
+                    days = int(self.get_argument("days", "1"))
+                except ValueError as e:
+                    self.redirect(
+                        f"/panel/error?error=Type error: Argument must be an int {e}"
+                    )
+                page_data["options"] = [1, 2, 3]
+                if not days in page_data["options"]:
+                    page_data["options"].insert(0, days)
+                else:
+                    page_data["options"].insert(
+                        0, page_data["options"].pop(page_data["options"].index(days))
+                    )
                 page_data["history_stats"] = self.controller.servers.get_history_stats(
-                    server_id
+                    server_id, days
                 )
 
             def get_banned_players_html():
diff --git a/app/frontend/templates/panel/server_metrics.html b/app/frontend/templates/panel/server_metrics.html
index 9bf46941..d10fec51 100644
--- a/app/frontend/templates/panel/server_metrics.html
+++ b/app/frontend/templates/panel/server_metrics.html
@@ -38,6 +38,16 @@
           <span class="d-block d-sm-none">
             {% include "parts/m_server_controls_list.html %}
           </span>
+          <div class="col-2">
+            <div class="form-group">
+              <label for="days">Metric Period</label>
+              <select required class="form-control form-control-lg select-css" id="days" name="days">
+                {% for value in data['options'] %}
+                <option value="{{value}}">{{value}} Day(s)</option>
+                {% end %}
+              </select>
+            </div>
+          </div>
           <button style="float: right; visibility: hidden;" class="btn btn-outline-success reset-button"
             id="reset-button"><i class="fas fa-undo"></i>&nbsp;{{ translate('serverMetrics', 'resetZoom', data['lang'])
             }}</button>
@@ -69,6 +79,7 @@
   }
 </style>
 <script type="text/javascript">
+  const serverId = new URLSearchParams(document.location.search).get('id')
   var zoomed = false;
   //line
   var ctxL = document.getElementById("lineChart").getContext('2d');
@@ -162,6 +173,10 @@
     }
   });
   $(window).ready(function () {
+    $('#days').on("change", function () {
+      let days = $('#days').find(":selected").val();
+      window.location.href = "/panel/server_detail?id=" + serverId + "&days=" + days + "&subpage=metrics"
+    });
     $('body').click(function () {
       $('.hints').popover("hide");
     });

From dd0f609d0ce14205027bed4c03f98ad8655d2680 Mon Sep 17 00:00:00 2001
From: Zedifus <zedifus@zediverse.gg>
Date: Sun, 4 Sep 2022 21:31:10 +0100
Subject: [PATCH 2/2] Update changelog !448

---
 CHANGELOG.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 3ef75323..f47beb06 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,7 +4,7 @@
 - Win Portable Updater will now be included in Windows Package ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/446))
 - Bedrock Server Creator ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/443))
 ### Bug fixes
-- Fix performance issues on server metrics panels (Temporarily setting to 24hr query) ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/440))
+- Fix performance issues on server metrics panels 'with metrics range' ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/440)) ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/448))
 - Fix no id on import3 servers ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/442))
 - Fix functionality of bedrock update ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/441))
 ### Tweaks