Merge branch 'tweak/stats-clients' into 'dev'

Only send realtime stats when clients connected

See merge request crafty-controller/crafty-4!488
This commit is contained in:
Iain Powrie
2022-10-22 19:30:45 +00:00
2 changed files with 58 additions and 52 deletions

View File

@ -12,8 +12,7 @@
### Tweaks
- Fix sidebar to not move when scrolling ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/481))
- Add the rest of CSS predefined colors to themes ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/477))
### Lang
TBD
- Only send realtime stats when clients connected ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/488))
<br><br>
## --- [4.0.15] - 2022/10/02

View File

@ -1223,6 +1223,9 @@ class ServerInstance:
# **********************************************************************************
def realtime_stats(self):
# only get stats if clients are connected.
# no point in burning cpu
if len(self.helper.websocket_helper.clients) > 0:
total_players = 0
max_players = 0
servers_ping = []
@ -1276,7 +1279,9 @@ class ServerInstance:
"version": raw_ping_result.get("version"),
"icon": raw_ping_result.get("icon"),
"crashed": self.is_crashed,
"created": datetime.datetime.now().strftime("%Y/%m/%d, %H:%M:%S"),
"created": datetime.datetime.now().strftime(
"%Y/%m/%d, %H:%M:%S"
),
},
)
total_players += int(raw_ping_result.get("online"))
@ -1284,7 +1289,9 @@ class ServerInstance:
self.record_server_stats()
if (len(servers_ping) > 0) & (len(self.helper.websocket_helper.clients) > 0):
if (len(servers_ping) > 0) & (
len(self.helper.websocket_helper.clients) > 0
):
try:
self.helper.websocket_helper.broadcast_page(
"/panel/dashboard", "update_server_status", servers_ping