From f8f1a5ccedbe5d67a8b635dc2041ace78cd50781 Mon Sep 17 00:00:00 2001
From: Dean Lofts <dean@deanlofts.xyz>
Date: Sun, 15 Sep 2024 00:05:52 +1000
Subject: [PATCH] fix analytics charts

---
 app/views/analytics/index.html.erb | 40 ++++++++++++++++--------------
 1 file changed, 21 insertions(+), 19 deletions(-)

diff --git a/app/views/analytics/index.html.erb b/app/views/analytics/index.html.erb
index b5f1a2b..12f7e22 100644
--- a/app/views/analytics/index.html.erb
+++ b/app/views/analytics/index.html.erb
@@ -100,30 +100,32 @@
     </div>
   </div>
 
-  <!-- Top Visitor Locations (New Section) -->
-  <div class="bg-gray-800 rounded-lg shadow p-4 mb-6">
-    <h2 class="text-xl font-semibold mb-4">Top Visitor Locations</h2>
-    <div class="overflow-x-auto">
-      <table class="w-full text-sm text-center table-auto">
-        <thead class="text-xs uppercase bg-gray-700">
-          <tr>
-            <th scope="col" class="px-4 py-3 rounded-tl-lg">City</th>
-            <th scope="col" class="px-4 py-3">Country</th>
-            <th scope="col" class="px-4 py-3 rounded-tr-lg">Views</th>
-          </tr>
-        </thead>
-        <tbody>
-          <% @location_data.each_with_index do |location, index| %>
+ <!-- Top Visitor Locations (New Section) -->
+<div class="bg-gray-800 rounded-lg shadow p-4 mb-6">
+  <h2 class="text-xl font-semibold mb-4">Top Visitor Locations</h2>
+  <div class="overflow-x-auto">
+    <table class="w-full text-sm text-center table-auto">
+      <thead class="text-xs uppercase bg-gray-700">
+        <tr>
+          <th scope="col" class="px-4 py-3 rounded-tl-lg">City</th>
+          <th scope="col" class="px-4 py-3">Country</th>
+          <th scope="col" class="px-4 py-3 rounded-tr-lg">Views</th>
+        </tr>
+      </thead>
+      <tbody>
+        <% @location_data.each_with_index do |location, index| %>
+          <% unless location[:city] == 'Unknown' && location[:country] == 'Unknown' %>
             <tr class="<%= index.even? ? 'bg-gray-800' : 'bg-gray-900' %> border-b border-gray-700">
-              <td class="px-4 py-3"><%= location[:city] || 'Unknown' %></td>
-              <td class="px-4 py-3"><%= location[:country] || 'Unknown' %></td>
+              <td class="px-4 py-3"><%= location[:city].present? && location[:city] != 'Unknown' ? location[:city] : '' %></td>
+              <td class="px-4 py-3"><%= location[:country].present? && location[:country] != 'Unknown' ? location[:country] : '' %></td>
               <td class="px-4 py-3"><%= number_with_delimiter(location[:count]) %></td>
             </tr>
           <% end %>
-        </tbody>
-      </table>
-    </div>
+        <% end %>
+      </tbody>
+    </table>
   </div>
+</div>
 
   <!-- Link Analytics Table -->
   <div class="bg-gray-800 rounded-lg shadow p-4 mb-6">