GTFS diff: améliorations de surface

apps/transport/lib/transport_web/live/gtfs_diff_select_live.ex

@@ -161,4 +161,13 @@ defmodule TransportWeb.Live.GTFSDiffSelectLive do
   defp error_to_string(:too_large), do: "File is too large, must be <#{@max_file_size_mb}MB"
   defp error_to_string(:too_many_files), do: "You must select 2 files"
   defp error_to_string(:not_accepted), do: "You have selected an unacceptable file type"
+
+  def translate_target(target, n) do
+    case target do
+      "file" -> dngettext("validations", "file", "files", n)
+      "row" -> dngettext("validations", "row", "rows", n)
+      "column" -> dngettext("validations", "column", "columns", n)
+      _ -> "#{n} #{target}#{if n > 1, do: "s"}"
+    end
+  end
 end

apps/transport/lib/transport_web/live/gtfs_diff_select_live.html.heex

@@ -90,33 +90,17 @@
             gtfs_original_file_name_2: @gtfs_original_file_name_2
           ) %>
           <div class="summary">
-            <div :if={@diff_summary["add"]}>
-              <ul>
-                <%= for {{file, "add", target}, n} <- @diff_summary["add"] do %>
-                  <li>
-                    <span class="green"><%= dgettext("validations", "added") %> &nbsp;</span><%= "#{n} #{target}#{if n > 1, do: "s"} - #{file}" %>
-                  </li>
-                <% end %>
-              </ul>
-            </div>
-            <div :if={@diff_summary["delete"]}>
-              <ul>
-                <%= for {{file, "delete", target}, n} <- @diff_summary["delete"] do %>
-                  <li>
-                    <span class="red"><%= dgettext("validations", "deleted") %> &nbsp;</span><%= "#{n} #{target}#{if n > 1, do: "s"} - #{file}" %>
-                  </li>
-                <% end %>
-              </ul>
-            </div>
-            <div :if={@diff_summary["update"]}>
-              <ul>
-                <%= for {{file, "update", target}, n} <- @diff_summary["update"] do %>
-                  <li>
-                    <span class="orange"><%= dgettext("validations", "updated") %> &nbsp;</span><%= "#{n} #{target}#{if n > 1, do: "s"} - #{file}" %>
-                  </li>
-                <% end %>
-              </ul>
-            </div>
+            <%= for {diff_nature, translation, css_class} <- [{"add", dgettext("validations", "added"), "green"}, {"update", dgettext("validations", "updated"), "orange"}, {"delete", dgettext("validations", "deleted"), "red"}] do %>
+              <div :if={@diff_summary[diff_nature]}>
+                <ul>
+                  <%= for {{file, ^diff_nature, target}, n} <- @diff_summary[diff_nature] do %>
+                    <li>
+                      <span class={css_class}><%= translation %> &nbsp;</span><%= "#{translate_target(target, n)} - #{file}" %>
+                    </li>
+                  <% end %>
+                </ul>
+              </div>
+            <% end %>
           </div>
         </div>
       <% else %>

apps/transport/priv/gettext/en/LC_MESSAGES/validations.po

@@ -437,3 +437,21 @@ msgstr ""
 #, elixir-autogen, elixir-format
 msgid "enRoute Chouette Valid"
 msgstr "enRoute Chouette Valid validator"
+
+#, elixir-autogen, elixir-format
+msgid "column"
+msgid_plural "columns"
+msgstr[0] ""
+msgstr[1] ""
+
+#, elixir-autogen, elixir-format
+msgid "file"
+msgid_plural "files"
+msgstr[0] ""
+msgstr[1] ""
+
+#, elixir-autogen, elixir-format
+msgid "row"
+msgid_plural "rows"
+msgstr[0] ""
+msgstr[1] ""

apps/transport/priv/gettext/fr/LC_MESSAGES/validations.po

@@ -437,3 +437,21 @@ msgstr "le validateur GTFS du <abbr title=\"Point d’Accès National\">PAN</abb
 #, elixir-autogen, elixir-format
 msgid "enRoute Chouette Valid"
 msgstr "le validateur NeTEx d’enRoute"
+
+#, elixir-autogen, elixir-format
+msgid "column"
+msgid_plural "columns"
+msgstr[0] "colonne"
+msgstr[1] "colonnes"
+
+#, elixir-autogen, elixir-format
+msgid "file"
+msgid_plural "files"
+msgstr[0] "fichier"
+msgstr[1] "fichiers"
+
+#, elixir-autogen, elixir-format
+msgid "row"
+msgid_plural "rows"
+msgstr[0] "ligne"
+msgstr[1] "lignes"

apps/transport/priv/gettext/validations.pot

@@ -435,3 +435,21 @@ msgstr ""
 #, elixir-autogen, elixir-format
 msgid "enRoute Chouette Valid"
 msgstr ""
+
+#, elixir-autogen, elixir-format
+msgid "column"
+msgid_plural "columns"
+msgstr[0] ""
+msgstr[1] ""
+
+#, elixir-autogen, elixir-format
+msgid "file"
+msgid_plural "files"
+msgstr[0] ""
+msgstr[1] ""
+
+#, elixir-autogen, elixir-format
+msgid "row"
+msgid_plural "rows"
+msgstr[0] ""
+msgstr[1] ""