hide GPX links if there is none

breunigs · Oct 14, 2023 · 3161928 · 3161928
1 parent c0bca93
commit 3161928
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 8 deletions.
diff --git a/lib/article/decorators.ex b/lib/article/decorators.ex
@@ -70,16 +70,22 @@ defmodule Article.Decorators do
     end
   end
 
-  @spec gpx_links(module()) :: Phoenix.LiveView.Rendered.t()
+  @spec gpx_links(module()) :: [Phoenix.LiveView.Rendered.t()]
   def gpx_links(art) when is_module(art) do
     assigns = %{name: art.name()}
 
-    ~H"""
-    Route im
-    <a href={"/geo/#{@name}.gpx"} download={"#{@name}.gpx"}>GPX-Format</a>
-    bzw.
-    <a href={"/geo/#{@name}.kml"} download={"#{@name}.kml"}>KML-Format</a>
-    """
+    if length(art.tracks()) > 0 do
+      [
+        ~H"""
+        Route im
+        <a href={"/geo/#{@name}.gpx"} download={"#{@name}.gpx"}>GPX-Format</a>
+        bzw.
+        <a href={"/geo/#{@name}.kml"} download={"#{@name}.kml"}>KML-Format</a>
+        """
+      ]
+    else
+      []
+    end
   end
 
   @doc """

diff --git a/lib/components/tag_helpers.ex b/lib/components/tag_helpers.ex
@@ -390,7 +390,7 @@ defmodule Components.TagHelpers do
           other
       end)
 
-    links = if assigns[:gpx], do: links ++ [Article.Decorators.gpx_links(art)], else: links
+    links = if assigns[:gpx], do: links ++ Article.Decorators.gpx_links(art), else: links
     assigns = %{links: links}
 
     case length(links) do