Display removed scrapers on the stats page

Earlopain · Oct 11, 2023 · 84ee8dc · 84ee8dc · faucetlol · Oct 13, 2023
1 parent 17532f4
commit 84ee8dc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/app/controllers/stats_controller.rb b/app/controllers/stats_controller.rb
@@ -14,7 +14,7 @@ def index
       "count(*) as file_count",
     ).joins(artist_submission: { artist_url: :artist }).group(:site_type).index_by(&:site_type)
     @counts.transform_keys! { |id| ArtistUrl.site_types.invert[id] }
-    @definitions = (Sites.scraper_definitions + [Sites.from_enum("manual")]).sort_by(&:display_name)
+    @definitions = Sites.definitions.sort_by(&:display_name)
   end
 
   private

diff --git a/app/views/stats/index.html.erb b/app/views/stats/index.html.erb
@@ -14,6 +14,7 @@
   <tbody>
     <% @definitions.each do |definition| %>
       <% entry = @counts[definition.site_type] || {} %>
+      <% next if definition.instance_of?(Sites::SimpleDefinition) && !entry["file_count"] %>
       <tr>
         <td><%= link_to_if(definition.site_type != "manual", definition.display_name, config_path(definition.site_type)) %></td>
         <td><%= link_to(entry["artist_count"] || 0, artists_path(search: { site_type: definition.site_type })) %></td>
@@ -26,6 +27,8 @@
           <% elsif definition.missing_config_keys.any? %>
             Missing config <%= "key".pluralize(definition.missing_config_keys.count) %>:<br>
             <%= definition.missing_config_keys.join(", ") %>
+          <% elsif definition.instance_of?(Sites::SimpleDefinition) %>
+            Removed
           <% else %>
             Enabled
           <% end %>