From bed86b7fd9e904ec10c7241b220b5106d0ce60a6 Mon Sep 17 00:00:00 2001
From: Averi <github@j3rzy.dev>
Date: Mon, 14 Oct 2024 10:04:49 +0200
Subject: [PATCH 1/2] Fixed "Most Damage" being present and `NaN` when player
 didnt do any damage to dragons and empty tooltips being present

---
 views/sections/stats/misc/end_island.ejs | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/views/sections/stats/misc/end_island.ejs b/views/sections/stats/misc/end_island.ejs
index 74b3042f11..939b7fc9ec 100644
--- a/views/sections/stats/misc/end_island.ejs
+++ b/views/sections/stats/misc/end_island.ejs
@@ -9,7 +9,7 @@
 
   <p class="stat-raw-values">
     <!-- Most Damage -->
-    <% if (calculated.misc.dragons.most_damage !== undefined) { %>
+    <% if (calculated.misc.dragons.most_damage !== undefined && Object.keys(calculated.misc.dragons.most_damage).length) { %>
       <%  let tooltip = "";
       for (const key in calculated.misc.dragons.most_damage) {
         if (key === "best") {
@@ -19,7 +19,7 @@
         tooltip += `<span class="stat-name">${helper.capitalizeFirstLetter(key)}: </span><span class="stat-value">${Math.floor(calculated.misc.dragons.most_damage[key]).toLocaleString()}</span><br>`;
       } %>
 
-      <span data-tippy-content='<%= tooltip %>'>
+      <% if (tooltip) { %><span data-tippy-content='<%= tooltip %>'><% } else { %><span><% } %>
         <span class="stat-name">Most Damage: </span>
         <span class="stat-value"><%= Math.floor(calculated.misc.dragons.most_damage.best).toLocaleString() %></span>
       </span>
@@ -27,7 +27,7 @@
     <% } %>
 
     <!-- Fastest Kill -->
-    <% if (calculated.misc.dragons.fastest_kill !== undefined) { %>
+    <% if (calculated.misc.dragons.fastest_kill !== undefined && Object.keys(calculated.misc.dragons.fastest_kill).length) { %>
       <%  tooltip = "";
       for (const key in calculated.misc.dragons.fastest_kill) {
         if (key === "best") {
@@ -37,7 +37,7 @@
         tooltip += `<span class="stat-name">${helper.capitalizeFirstLetter(key)}: </span><span class="stat-value">${moment.duration(calculated.misc.dragons.fastest_kill[key], "milliseconds").format("m:ss.SSS")}s</span><br>`;
       } %>
 
-      <span data-tippy-content='<%= tooltip %>'>
+        <% if (tooltip) { %><span data-tippy-content='<%= tooltip %>'><% } else { %><span><% } %>
         <span class="stat-name">Fastest Kill: </span>
         <span class="stat-value"><%= moment.duration(calculated.misc.dragons.fastest_kill.best, "milliseconds").format("m:ss.SSS") %>s</span>
       </span>
@@ -46,7 +46,7 @@
 
 
     <!-- Last Hits (Kills) -->
-    <% if (calculated.misc.dragons.kills !== undefined) { %>
+    <% if (calculated.misc.dragons.kills !== undefined && Object.keys(calculated.misc.dragons.kills).length) { %>
       <% tooltip = "";
       for (const key in calculated.misc.dragons.kills) {
         if (key === "total") {
@@ -56,7 +56,7 @@
         tooltip += `<span class="stat-name">${helper.capitalizeFirstLetter(key)}: </span><span class="stat-value">${calculated.misc.dragons.kills[key].toLocaleString()}</span><br>`;
       } %>
 
-      <span data-tippy-content='<%= tooltip %>'>
+      <% if (tooltip) { %><span data-tippy-content='<%= tooltip %>'><% } else { %><span><% } %>
         <span class="stat-name">Last Hits: </span>
         <span class="stat-value"><%= calculated.misc.dragons.kills.total.toLocaleString() %></span>
       </span>
@@ -64,7 +64,7 @@
     <% } %>
 
     <!-- Deaths -->
-    <% if (calculated.misc.dragons.deaths !== undefined) { %>
+    <% if (calculated.misc.dragons.deaths !== undefined && Object.keys(calculated.misc.dragons.deaths).length) { %>
       <% tooltip = "";
       for (const key in calculated.misc.dragons.deaths) {
         if (key === "total") {
@@ -73,7 +73,8 @@
 
         tooltip += `<span class="stat-name">${helper.capitalizeFirstLetter(key)}: </span><span class="stat-value">${calculated.misc.dragons.deaths[key].toLocaleString()}</span><br>`;
       } %>
-      <span data-tippy-content='<%= tooltip %>'>
+
+      <% if (tooltip) { %><span data-tippy-content='<%= tooltip %>'><% } else { %><span><% } %>
         <span class="stat-name">Deaths: </span>
         <span class="stat-value"><%= calculated.misc.dragons.deaths.total.toLocaleString() %></span>
       </span>
@@ -99,4 +100,4 @@
     <span class="stat-value"><%= calculated.misc.endstone_protector.deaths.toLocaleString() %></span>
     <br>
   </p>
-<% } %>
\ No newline at end of file
+<% } %>

From 8e9de74f746313a18c621d2a7af1ce89de94d9a0 Mon Sep 17 00:00:00 2001
From: Averi <github@j3rzy.dev>
Date: Mon, 14 Oct 2024 10:08:35 +0200
Subject: [PATCH 2/2] fixed time below 1s displaying as full seconds instead
 split second (SkyCryptWebsite/SkyCrypt#2250)

---
 views/sections/stats/misc/end_island.ejs | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/views/sections/stats/misc/end_island.ejs b/views/sections/stats/misc/end_island.ejs
index 939b7fc9ec..0febfa8fdd 100644
--- a/views/sections/stats/misc/end_island.ejs
+++ b/views/sections/stats/misc/end_island.ejs
@@ -29,17 +29,20 @@
     <!-- Fastest Kill -->
     <% if (calculated.misc.dragons.fastest_kill !== undefined && Object.keys(calculated.misc.dragons.fastest_kill).length) { %>
       <%  tooltip = "";
+
+      const formatTime = (ms) => (ms < 1000 ? `0.${ms}` : moment.duration(ms, "milliseconds").format("m:ss.SSS")) + "s";
+
       for (const key in calculated.misc.dragons.fastest_kill) {
         if (key === "best") {
           continue;
         }
 
-        tooltip += `<span class="stat-name">${helper.capitalizeFirstLetter(key)}: </span><span class="stat-value">${moment.duration(calculated.misc.dragons.fastest_kill[key], "milliseconds").format("m:ss.SSS")}s</span><br>`;
+        tooltip += `<span class="stat-name">${helper.capitalizeFirstLetter(key)}: </span><span class="stat-value">${formatTime(calculated.misc.dragons.fastest_kill[key])}</span><br>`;
       } %>
 
-        <% if (tooltip) { %><span data-tippy-content='<%= tooltip %>'><% } else { %><span><% } %>
+      <% if (tooltip) { %><span data-tippy-content='<%= tooltip %>'><% } else { %><span><% } %>
         <span class="stat-name">Fastest Kill: </span>
-        <span class="stat-value"><%= moment.duration(calculated.misc.dragons.fastest_kill.best, "milliseconds").format("m:ss.SSS") %>s</span>
+        <span class="stat-value"><%= formatTime(calculated.misc.dragons.fastest_kill.best) %></span>
       </span>
       <br>
     <% } %>