fixup: one tag per itinerary

mbta · Feb 14, 2024 · b66ef82 · b66ef82
1 parent c0b311b
commit b66ef82
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/lib/dotcom_web/views/trip_plan_view.ex b/lib/dotcom_web/views/trip_plan_view.ex
@@ -519,19 +519,22 @@ defmodule DotcomWeb.TripPlanView do
         tab_html: tab_html,
         id: index,
         map: itinerary_map(map),
-        tags:
-          Enum.map(i.tags, fn tag ->
-            tag
-            |> Atom.to_string()
-            |> String.replace("_", " ")
-          end),
+        tag: tag_string(i.tag),
         access_html: access_html,
         fares_estimate_html: fares_estimate_html,
         fare_calculator_html: fare_calculator_html
       }
     end
   end
 
+  defp tag_string(nil), do: nil
+
+  defp tag_string(tag),
+    do:
+      tag
+      |> Atom.to_string()
+      |> String.replace("_", " ")
+
   @spec render_react(map) :: HTML.safe()
   def render_react(assigns) do
     Util.log_duration(__MODULE__, :do_render_react, [assigns])

diff --git a/lib/trip_plan/api/open_trip_planner.ex b/lib/trip_plan/api/open_trip_planner.ex
@@ -36,7 +36,7 @@ defmodule TripPlan.Api.OpenTripPlanner do
       stop: parse_time(json["endTime"]),
       legs: Enum.map(json["legs"], &parse_leg/1),
       accessible?: if(score, do: score == 1.0),
-      tag: json["tag"] |> MapSet.to_list()
+      tag: json["tag"]
     }
   end