From d7c4772ad8716ea5dbadd141c6ceccb333f006f3 Mon Sep 17 00:00:00 2001
From: Graham McIntire <graham@mcintire.me>
Date: Thu, 12 Mar 2026 09:01:16 -0500
Subject: [PATCH] Only show subsonic filter when subsonic products exist for
 caliber

---
 lib/ammoprices/catalog.ex                      |  6 ++++++
 lib/ammoprices_web/live/caliber_live/show.ex   |  4 +++-
 .../live/caliber_live/show_test.exs            | 18 ++++++++++++++++--
 3 files changed, 25 insertions(+), 3 deletions(-)

diff --git a/lib/ammoprices/catalog.ex b/lib/ammoprices/catalog.ex
index 93105d7..3d2002d 100644
--- a/lib/ammoprices/catalog.ex
+++ b/lib/ammoprices/catalog.ex
@@ -103,6 +103,12 @@ defmodule Ammoprices.Catalog do
     |> Repo.all()
   end
 
+  def has_subsonic_products?(caliber_id) do
+    Product
+    |> where([p], p.caliber_id == ^caliber_id and p.subsonic == true)
+    |> Repo.exists?()
+  end
+
   def distinct_casings_for_caliber(caliber_id) do
     Product
     |> where([p], p.caliber_id == ^caliber_id)
diff --git a/lib/ammoprices_web/live/caliber_live/show.ex b/lib/ammoprices_web/live/caliber_live/show.ex
index 0ffe7b9..7c207d0 100644
--- a/lib/ammoprices_web/live/caliber_live/show.ex
+++ b/lib/ammoprices_web/live/caliber_live/show.ex
@@ -119,7 +119,8 @@ defmodule AmmopricesWeb.CaliberLive.Show do
 
     assign(socket,
       available_casings: Catalog.distinct_casings_for_caliber(caliber_id),
-      available_grain_weights: Catalog.distinct_grain_weights_for_caliber(caliber_id)
+      available_grain_weights: Catalog.distinct_grain_weights_for_caliber(caliber_id),
+      has_subsonic_products: Catalog.has_subsonic_products?(caliber_id)
     )
   end
 
@@ -310,6 +311,7 @@ defmodule AmmopricesWeb.CaliberLive.Show do
 
           <%!-- Subsonic toggle --%>
           <button
+            :if={@has_subsonic_products}
             id="filter-subsonic"
             phx-click="toggle_subsonic_filter"
             class={[
diff --git a/test/ammoprices_web/live/caliber_live/show_test.exs b/test/ammoprices_web/live/caliber_live/show_test.exs
index 8cb3417..31aaa8e 100644
--- a/test/ammoprices_web/live/caliber_live/show_test.exs
+++ b/test/ammoprices_web/live/caliber_live/show_test.exs
@@ -118,9 +118,13 @@ defmodule AmmopricesWeb.CaliberLive.ShowTest do
       assert has_element?(view, "#filter-grain-147")
     end
 
-    test "renders subsonic filter toggle", %{conn: conn, caliber: caliber, retailer: retailer} do
+    test "renders subsonic filter toggle when subsonic products exist", %{
+      conn: conn,
+      caliber: caliber,
+      retailer: retailer
+    } do
       now = DateTime.truncate(DateTime.utc_now(), :second)
-      product = product_fixture(%{caliber: caliber, retailer: retailer, in_stock: true})
+      product = product_fixture(%{caliber: caliber, retailer: retailer, in_stock: true, subsonic: true})
       snapshot_fixture(%{product: product, price_per_round_cents: 25, in_stock: true, recorded_at: now})
 
       {:ok, view, _html} = live(conn, "/calibers/#{caliber.slug}")
@@ -128,6 +132,16 @@ defmodule AmmopricesWeb.CaliberLive.ShowTest do
       assert has_element?(view, "#filter-subsonic")
     end
 
+    test "hides subsonic filter when no subsonic products", %{conn: conn, caliber: caliber, retailer: retailer} do
+      now = DateTime.truncate(DateTime.utc_now(), :second)
+      product = product_fixture(%{caliber: caliber, retailer: retailer, in_stock: true, subsonic: false})
+      snapshot_fixture(%{product: product, price_per_round_cents: 25, in_stock: true, recorded_at: now})
+
+      {:ok, view, _html} = live(conn, "/calibers/#{caliber.slug}")
+
+      refute has_element?(view, "#filter-subsonic")
+    end
+
     test "displays grain weight and brand in product table", %{conn: conn, caliber: caliber, retailer: retailer} do
       now = DateTime.truncate(DateTime.utc_now(), :second)