From 2f25655c9e8bc65410118b6fa416a1930efa6b83 Mon Sep 17 00:00:00 2001
From: Antoine Augusti <antoine.augusti@transport.data.gouv.fr>
Date: Mon, 20 Nov 2023 14:12:28 +0100
Subject: [PATCH] =?UTF-8?q?Backoffice=20dataset=20form=20:=20g=C3=A8re=20c?=
 =?UTF-8?q?as=20o=C3=B9=20org=20est=20nil=20(#3614)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../controllers/backoffice/page_controller.ex              | 7 +++++++
 .../templates/backoffice/page/form_dataset.html.heex       | 5 ++---
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/apps/transport/lib/transport_web/controllers/backoffice/page_controller.ex b/apps/transport/lib/transport_web/controllers/backoffice/page_controller.ex
index 6e8d5f3594..574651d53a 100644
--- a/apps/transport/lib/transport_web/controllers/backoffice/page_controller.ex
+++ b/apps/transport/lib/transport_web/controllers/backoffice/page_controller.ex
@@ -169,6 +169,7 @@ defmodule TransportWeb.Backoffice.PageController do
     |> assign(:notifications_last_nb_days, notifications_last_nb_days())
     |> assign(:resources_with_history, DB.Dataset.last_resource_history(dataset_id))
     |> assign(:contacts_datalist, contacts_datalist())
+    |> assign(:contacts_in_org, contacts_in_org(conn.assigns[:dataset]))
     |> assign(
       :import_logs,
       LogsImport
@@ -179,6 +180,12 @@ defmodule TransportWeb.Backoffice.PageController do
     |> render("form_dataset.html")
   end
 
+  defp contacts_in_org(%DB.Dataset{organization_object: organization_object}) do
+    Enum.sort_by(organization_object.contacts, &DB.Contact.display_name/1)
+  end
+
+  defp contacts_in_org(_), do: []
+
   defp contacts_datalist do
     DB.Contact.base_query()
     |> select([contact: c], [:first_name, :last_name, :mailing_list_title, :organization, :id])
diff --git a/apps/transport/lib/transport_web/templates/backoffice/page/form_dataset.html.heex b/apps/transport/lib/transport_web/templates/backoffice/page/form_dataset.html.heex
index d9e5aef7f1..843a9e8756 100644
--- a/apps/transport/lib/transport_web/templates/backoffice/page/form_dataset.html.heex
+++ b/apps/transport/lib/transport_web/templates/backoffice/page/form_dataset.html.heex
@@ -4,7 +4,6 @@
   ) %>
 
   <%= unless is_nil(@dataset) do %>
-    <% contacts_in_org = Enum.sort_by(@dataset.organization_object.contacts, &DB.Contact.display_name/1) %>
     <div class="is-centered mt-48">
       <%= dgettext("backoffice", "Other actions on the dataset") %>
     </div>
@@ -84,12 +83,12 @@
               <i class="fas fa-external-link"></i> Créer un contact
             </a>
           </div>
-          <div :if={Enum.count(contacts_in_org) > 0}>
+          <div :if={Enum.count(@contacts_in_org) > 0}>
             <p>
               Contacts dans l'organisation s'étant déjà connectés au PAN
             </p>
             <ul>
-              <%= for contact <- contacts_in_org do %>
+              <%= for contact <- @contacts_in_org do %>
                 <% onclick_fn = ~s{getElementById("contact_id").value =} <> to_string(contact.id) %>
                 <li class="pb-6">
                   <a href={backoffice_contact_path(@conn, :edit, contact.id)} target="_blank">