Make modalities form block generic

Move FranceConnect management within related definitions
etalab · Jan 22, 2025 · 309e8b1 · 309e8b1
1 parent dc0a123
commit 309e8b1
Show file tree

Hide file tree

Showing 3 changed files with 50 additions and 30 deletions.
diff --git a/app/views/authorization_request_forms/blocks/api_impot_particulier/_modalities.html.erb b/app/views/authorization_request_forms/blocks/api_impot_particulier/_modalities.html.erb
@@ -0,0 +1 @@
+../api_impot_particulier_sandbox/_modalities.html.erb
diff --git a/...ews/authorization_request_forms/blocks/api_impot_particulier_sandbox/_modalities.html.erb b/...ews/authorization_request_forms/blocks/api_impot_particulier_sandbox/_modalities.html.erb
@@ -0,0 +1,44 @@
+<% if @authorization_request.display_prefilled_banner_for_each_block? && @authorization_request.prefilled_data?(%i[modalities france_connect_authorization_id]) %>
+  <%= render partial: "authorization_request_forms/shared/prefilled_banner" %>
+<% end %>
+
+<%= f.info_for(:modalities) %>
+
+<div data-controller="choose-modalite-impot-particulier">
+  <div class="fr-col-lg-9">
+    <%= 
+      f.dsfr_radio_buttons :modalities, 
+        @authorization_request.available_modalities, 
+        required: true,
+        multiple: true,
+        radio_group_class: "fr-radio-rich", 
+        fieldset_element_class: "small",
+        checked: ->(value) { @authorization_request.modalities.include?(value) },
+        input_options: { 
+          "data-action": "click->choose-modalite-impot-particulier#trigger",
+          "data-choose-modalite-impot-particulier-target": "modality"
+        }
+    %>
+  </div>
+
+  <div class="fr-col-lg-9 fr-mb-5w <%= 'fr-hidden' unless @authorization_request.with_france_connect? %>" data-choose-modalite-impot-particulier-target="franceConnectContainer">
+    <% france_connect_authorizations = current_organization.valid_authorizations_of(AuthorizationRequest::FranceConnect).map(&:decorate) %>
+
+    <% if france_connect_authorizations.empty? %>
+      <div class="fr-callout red-callout">
+        <h3 class="fr-callout__title">
+          <%= t("authorization_request_forms.api_impot_particulier_sandbox.modalities.callout.title") %>
+        </h3>
+        <p class="fr-callout__text fr-my-3w">
+          <%= t("authorization_request_forms.api_impot_particulier_sandbox.modalities.callout.content") %>
+        </p>
+        <%= link_to  t("authorization_request_forms.api_impot_particulier_sandbox.modalities.callout.link"), new_authorization_request_form_path("france-connect"), class: "fr-link fr-btn--icon-right fr-icon-arrow-right-line" %>
+      </div>
+
+    <% else %>
+      <% options = france_connect_authorizations.map{ |authorization| [authorization.name_for_select, authorization.id] } %>
+      <% selected_option = @authorization_request.france_connect_authorization_id || france_connect_authorizations.first.id %>
+      <%= f.dsfr_select :france_connect_authorization_id, options_for_select(options, selected_option), class: %w[fr-select], include_blank: "Sélectionner une option", required: true, input_options: { "data-choose-modalite-impot-particulier-target": "franceConnectSelector" } %>
+    <% end %>
+  </div>
+</div>
diff --git a/app/views/authorization_request_forms/blocks/default/_modalities.html.erb b/app/views/authorization_request_forms/blocks/default/_modalities.html.erb
@@ -4,41 +4,16 @@
 
 <%= f.info_for(:modalities) %>
 
-<div data-controller="choose-modalite-impot-particulier">
+<div>
   <div class="fr-col-lg-9">
-    <%= 
-      f.dsfr_radio_buttons :modalities, 
-        @authorization_request.available_modalities, 
+    <%=
+      f.dsfr_radio_buttons :modalities,
+        @authorization_request.available_modalities,
         required: true,
         multiple: true,
-        radio_group_class: "fr-radio-rich", 
+        radio_group_class: "fr-radio-rich",
         fieldset_element_class: "small",
         checked: ->(value) { @authorization_request.modalities.include?(value) },
-        input_options: { 
-          "data-action": "click->choose-modalite-impot-particulier#trigger",
-          "data-choose-modalite-impot-particulier-target": "modality"
-        }
     %>
   </div>
-
-  <div class="fr-col-lg-9 fr-mb-5w <%= 'fr-hidden' unless @authorization_request.with_france_connect? %>" data-choose-modalite-impot-particulier-target="franceConnectContainer">
-    <% france_connect_authorizations = current_organization.valid_authorizations_of(AuthorizationRequest::FranceConnect).map(&:decorate) %>
-
-    <% if france_connect_authorizations.empty? %>
-      <div class="fr-callout red-callout">
-        <h3 class="fr-callout__title">
-          <%= t("authorization_request_forms.api_impot_particulier_sandbox.modalities.callout.title") %>
-        </h3>
-        <p class="fr-callout__text fr-my-3w">
-          <%= t("authorization_request_forms.api_impot_particulier_sandbox.modalities.callout.content") %>
-        </p>
-        <%= link_to  t("authorization_request_forms.api_impot_particulier_sandbox.modalities.callout.link"), new_authorization_request_form_path("france-connect"), class: "fr-link fr-btn--icon-right fr-icon-arrow-right-line" %>
-      </div>
-
-    <% else %>
-      <% options = france_connect_authorizations.map{ |authorization| [authorization.name_for_select, authorization.id] } %>
-      <% selected_option = @authorization_request.france_connect_authorization_id || france_connect_authorizations.first.id %>
-      <%= f.dsfr_select :france_connect_authorization_id, options_for_select(options, selected_option), class: %w[fr-select], include_blank: "Sélectionner une option", required: true, input_options: { "data-choose-modalite-impot-particulier-target": "franceConnectSelector" } %>
-    <% end %>
-  </div>
 </div>
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		../api_impot_particulier_sandbox/_modalities.html.erb