From a54f0223537e8aead79b6029969fa9e6ac75a8ca Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Tue, 10 Dec 2024 15:50:08 +0100
Subject: [PATCH 1/6] fix(dossier/footer): don't say user can submit dossier
 when he can't

---
 .../autosave_footer_component.en.yml          |  3 +-
 .../autosave_footer_component.fr.yml          |  3 +-
 .../autosave_footer_component.html.haml       |  2 +
 .../autosave_footer_component_spec.rb         | 44 +++++++++++++++++++
 4 files changed, 50 insertions(+), 2 deletions(-)
 create mode 100644 spec/components/dossiers/autosave_footer_component_spec.rb

diff --git a/app/components/dossiers/autosave_footer_component/autosave_footer_component.en.yml b/app/components/dossiers/autosave_footer_component/autosave_footer_component.en.yml
index 21ac1dd6e..d03a8dd98 100644
--- a/app/components/dossiers/autosave_footer_component/autosave_footer_component.en.yml
+++ b/app/components/dossiers/autosave_footer_component/autosave_footer_component.en.yml
@@ -5,7 +5,8 @@ en:
     confirmation: Draft saved
     error: Impossible to save the draft
   en_construction:
-    explanation: Your modifications are automatically saved. Submit them when you’re done.
+    explanation: Your modifications are automatically saved.
+    submit_them: Submit them when you’re done.
     confirmation: Modifications saved
     error: Impossible to save the modifications.
   annotations:
diff --git a/app/components/dossiers/autosave_footer_component/autosave_footer_component.fr.yml b/app/components/dossiers/autosave_footer_component/autosave_footer_component.fr.yml
index bbee04894..6d962ca27 100644
--- a/app/components/dossiers/autosave_footer_component/autosave_footer_component.fr.yml
+++ b/app/components/dossiers/autosave_footer_component/autosave_footer_component.fr.yml
@@ -5,7 +5,8 @@ fr:
     confirmation: Brouillon enregistré
     error: Impossible d’enregistrer le brouillon
   en_construction:
-    explanation: Vos modifications sont automatiquement enregistrées. Déposez-les quand vous aurez terminé.
+    explanation: Vos modifications sont automatiquement enregistrées.
+    submit_them: Déposez-les quand vous aurez terminé.
     confirmation: Modifications enregistrées.
     error: Impossible d’enregistrer les modifications
   annotations:
diff --git a/app/components/dossiers/autosave_footer_component/autosave_footer_component.html.haml b/app/components/dossiers/autosave_footer_component/autosave_footer_component.html.haml
index 28039d90c..879e3a349 100644
--- a/app/components/dossiers/autosave_footer_component/autosave_footer_component.html.haml
+++ b/app/components/dossiers/autosave_footer_component/autosave_footer_component.html.haml
@@ -5,6 +5,8 @@
         = t('.annotations.explanation')
       - elsif dossier.editing_fork?
         = t('.en_construction.explanation')
+        - if dossier.can_passer_en_construction?
+          = t('.en_construction.submit_them')
       - else
         = t('.brouillon.explanation')
     - if !annotation?
diff --git a/spec/components/dossiers/autosave_footer_component_spec.rb b/spec/components/dossiers/autosave_footer_component_spec.rb
new file mode 100644
index 000000000..84d7cd76b
--- /dev/null
+++ b/spec/components/dossiers/autosave_footer_component_spec.rb
@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Dossiers::AutosaveFooterComponent, type: :component do
+  subject(:component) { render_inline(described_class.new(dossier:, annotation:)) }
+
+  let(:dossier) { create(:dossier) }
+  let(:annotation) { false }
+
+  context 'when showing brouillon state (default state)' do
+    it 'displays brouillon explanation' do
+      expect(component).to have_text("Votre brouillon")
+    end
+  end
+
+  context 'when editing fork and can pass en construction' do
+    let(:dossier) { create(:dossier, :en_construction).find_or_create_editing_fork(create(:user)) }
+
+    it 'displays en construction explanation' do
+      expect(component).to have_text("Vos modifications")
+      expect(component).to have_text("Déposez-les")
+    end
+
+    context 'when dossier is not eligible' do
+      before do
+        allow(dossier).to receive(:can_passer_en_construction?).and_return(false)
+      end
+
+      it 'displays en construction explanation' do
+        expect(component).to have_text("Vos modifications")
+        expect(component).not_to have_text("Déposez-les")
+      end
+    end
+  end
+
+  context 'when showing annotations' do
+    let(:annotation) { true }
+
+    it 'displays annotations explanation' do
+      expect(component).to have_text("Vos annotations")
+    end
+  end
+end

From f6366a9cd3a8dbc2734b878fcf7dbd6bf14c12c9 Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Tue, 10 Dec 2024 15:52:17 +0100
Subject: [PATCH 2/6] refactor(dossier): predicates methods must return boolean

---
 app/models/concerns/dossier_clone_concern.rb | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/models/concerns/dossier_clone_concern.rb b/app/models/concerns/dossier_clone_concern.rb
index e147c9a20..84b9ed481 100644
--- a/app/models/concerns/dossier_clone_concern.rb
+++ b/app/models/concerns/dossier_clone_concern.rb
@@ -38,15 +38,15 @@ module DossierCloneConcern
   end
 
   def forked_with_changes?
-    if forked_diff.present?
-      forked_diff.values.any?(&:present?) || forked_groupe_instructeur_changed?
-    end
+    return false if forked_diff.blank?
+
+    forked_diff.values.any?(&:present?) || forked_groupe_instructeur_changed?
   end
 
   def champ_forked_with_changes?(champ)
-    if forked_diff.present?
-      forked_diff.values.any? { |champs| champs.any? { _1.public_id == champ.public_id } }
-    end
+    return false if forked_diff.blank?
+
+    forked_diff.values.any? { |champs| champs.any? { _1.public_id == champ.public_id } }
   end
 
   def make_diff(editing_fork)

From eb682f1e64b35e583c69a29324658fe5ecb399ed Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Tue, 10 Dec 2024 16:59:56 +0100
Subject: [PATCH 3/6] refactor(admin): ineligibilite submit feedback

---
 .../pending_republish_component.html.haml                      | 2 +-
 .../administrateurs/ineligibilite_rules_controller.rb          | 3 ++-
 .../administrateurs/ineligibilite_rules_controller_spec.rb     | 3 ++-
 3 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/app/components/procedure/pending_republish_component/pending_republish_component.html.haml b/app/components/procedure/pending_republish_component/pending_republish_component.html.haml
index eab7f62fc..276955ce1 100644
--- a/app/components/procedure/pending_republish_component/pending_republish_component.html.haml
+++ b/app/components/procedure/pending_republish_component/pending_republish_component.html.haml
@@ -1,3 +1,3 @@
-= render Dsfr::AlertComponent.new(state: :warning) do |c|
+= render Dsfr::AlertComponent.new(state: :warning, extra_class_names: "fr-mb-3w") do |c|
   - c.with_body do
     = t('.pending_republish_html', href: admin_procedure_path(@procedure.id))
diff --git a/app/controllers/administrateurs/ineligibilite_rules_controller.rb b/app/controllers/administrateurs/ineligibilite_rules_controller.rb
index b21e6a2c2..18db66a07 100644
--- a/app/controllers/administrateurs/ineligibilite_rules_controller.rb
+++ b/app/controllers/administrateurs/ineligibilite_rules_controller.rb
@@ -11,7 +11,8 @@ module Administrateurs
       draft_revision.assign_attributes(procedure_revision_params)
 
       if draft_revision.validate(:ineligibilite_rules_editor) && draft_revision.save
-        redirect_to edit_admin_procedure_ineligibilite_rules_path(@procedure)
+        flash[:notice] = "Les conditions d‘inéligibilité ont été modifiées."
+        redirect_to [:admin, @procedure]
       else
         flash[:alert] = draft_revision.errors.full_messages
         render :edit
diff --git a/spec/controllers/administrateurs/ineligibilite_rules_controller_spec.rb b/spec/controllers/administrateurs/ineligibilite_rules_controller_spec.rb
index 92ab88497..f1125384b 100644
--- a/spec/controllers/administrateurs/ineligibilite_rules_controller_spec.rb
+++ b/spec/controllers/administrateurs/ineligibilite_rules_controller_spec.rb
@@ -243,7 +243,8 @@ describe Administrateurs::IneligibiliteRulesController, type: :controller do
         draft_revision = procedure.reload.draft_revision
         expect(draft_revision.ineligibilite_message).to eq('panpan')
         expect(draft_revision.ineligibilite_enabled).to eq(true)
-        expect(response).to redirect_to(edit_admin_procedure_ineligibilite_rules_path(procedure))
+        expect(response).to redirect_to(admin_procedure_path(procedure))
+        expect(flash.notice).not_to be_empty
       end
     end
   end

From 009e426c31354b3f097754dfa7060e6c0384831a Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Tue, 10 Dec 2024 19:13:01 +0100
Subject: [PATCH 4/6] fix(dossier): show ineligibilite message on update

---
 .../invalid_ineligibilite_rules_component.rb  | 18 ++++++++++---
 ...id_ineligibilite_rules_component.html.haml | 12 ++++++---
 app/controllers/users/dossiers_controller.rb  |  4 +--
 .../ineligibilite_rules_match_controller.ts   | 19 --------------
 app/models/dossier.rb                         | 12 ---------
 .../users/dossiers/update.turbo_stream.haml   |  7 ++---
 .../users/dossiers_controller_spec.rb         | 26 +++++++++++++------
 .../users/dossier_ineligibilite_spec.rb       | 19 +++++++-------
 .../shared/dossiers/_edit.html.haml_spec.rb   |  1 +
 9 files changed, 55 insertions(+), 63 deletions(-)
 delete mode 100644 app/javascript/controllers/ineligibilite_rules_match_controller.ts

diff --git a/app/components/dossiers/invalid_ineligibilite_rules_component.rb b/app/components/dossiers/invalid_ineligibilite_rules_component.rb
index 78047042f..3de971b68 100644
--- a/app/components/dossiers/invalid_ineligibilite_rules_component.rb
+++ b/app/components/dossiers/invalid_ineligibilite_rules_component.rb
@@ -1,18 +1,28 @@
 # frozen_string_literal: true
 
 class Dossiers::InvalidIneligibiliteRulesComponent < ApplicationComponent
-  delegate :can_passer_en_construction?, to: :@dossier
+  delegate :can_passer_en_construction?, to: :dossier
 
-  def initialize(dossier:)
+  def initialize(dossier:, wrapped: true)
     @dossier = dossier
     @revision = dossier.revision
+
+    @opened = !dossier.can_passer_en_construction?
+    @wrapped = wrapped
   end
 
+  private
+
+  attr_reader :dossier
+
   def render?
-    !can_passer_en_construction?
+    dossier.revision.ineligibilite_enabled?
   end
 
   def error_message
-    @dossier.revision.ineligibilite_message
+    dossier.revision.ineligibilite_message
   end
+
+  def opened? = @opened
+  def wrapped? = @wrapped
 end
diff --git a/app/components/dossiers/invalid_ineligibilite_rules_component/invalid_ineligibilite_rules_component.html.haml b/app/components/dossiers/invalid_ineligibilite_rules_component/invalid_ineligibilite_rules_component.html.haml
index dd39925cd..061b44c06 100644
--- a/app/components/dossiers/invalid_ineligibilite_rules_component/invalid_ineligibilite_rules_component.html.haml
+++ b/app/components/dossiers/invalid_ineligibilite_rules_component/invalid_ineligibilite_rules_component.html.haml
@@ -1,8 +1,8 @@
-%div{ id: dom_id(@dossier, :ineligibilite_rules_broken), data: { controller: 'ineligibilite-rules-match', turbo_force: :server } }
-  %button.fr-sr-only{ aria: {controls: 'modal-eligibilite-rules-dialog' }, data: {'fr-opened': "false" } }
+- modal_content = capture do
+  %button.fr-sr-only{ aria: { controls: 'modal-eligibilite-rules-dialog' }, data: { 'fr-opened': opened?.to_s } }
     show modal
 
-  %dialog.fr-modal{ "aria-labelledby" => "fr-modal-title-modal-1", role: "dialog", id: 'modal-eligibilite-rules-dialog', data: { 'ineligibilite-rules-match-target' => 'dialog' } }
+  %dialog.fr-modal{ "aria-labelledby" => "fr-modal-title-modal-1", role: "dialog", id: 'modal-eligibilite-rules-dialog', data: { 'turbo-permanent' => true } }
     .fr-container.fr-container--fluid.fr-container-md
       .fr-grid-row.fr-grid-row--center
         .fr-col-12.fr-col-md-8.fr-col-lg-6
@@ -14,3 +14,9 @@
                 %span.fr-icon-arrow-right-line.fr-icon--lg>
                 = t('.modal.title')
               %p= error_message
+
+- if wrapped?
+  %div{ id: dom_id(@dossier, :ineligibilite_rules_broken) }
+    = modal_content
+- else
+  = modal_content
diff --git a/app/controllers/users/dossiers_controller.rb b/app/controllers/users/dossiers_controller.rb
index 626cb0692..00160b984 100644
--- a/app/controllers/users/dossiers_controller.rb
+++ b/app/controllers/users/dossiers_controller.rb
@@ -280,9 +280,7 @@ module Users
     def update
       @dossier = dossier.en_construction? ? dossier.find_editing_fork(dossier.user) : dossier
       @dossier = dossier_with_champs(pj_template: false)
-      @can_passer_en_construction_was, @can_passer_en_construction_is = dossier.track_can_passer_en_construction do
-        update_dossier_and_compute_errors
-      end
+      update_dossier_and_compute_errors
 
       respond_to do |format|
         format.turbo_stream do
diff --git a/app/javascript/controllers/ineligibilite_rules_match_controller.ts b/app/javascript/controllers/ineligibilite_rules_match_controller.ts
deleted file mode 100644
index 5b47d79b5..000000000
--- a/app/javascript/controllers/ineligibilite_rules_match_controller.ts
+++ /dev/null
@@ -1,19 +0,0 @@
-import { ApplicationController } from './application_controller';
-declare interface modal {
-  disclose: () => void;
-}
-declare interface dsfr {
-  modal: modal;
-}
-declare const window: Window &
-  typeof globalThis & { dsfr: (elem: HTMLElement) => dsfr };
-
-export class InvalidIneligibiliteRulesController extends ApplicationController {
-  static targets = ['dialog'];
-
-  declare dialogTarget: HTMLElement;
-
-  connect() {
-    setTimeout(() => window.dsfr(this.dialogTarget).modal.disclose(), 100);
-  }
-}
diff --git a/app/models/dossier.rb b/app/models/dossier.rb
index 197702963..cad644dfd 100644
--- a/app/models/dossier.rb
+++ b/app/models/dossier.rb
@@ -1025,18 +1025,6 @@ class Dossier < ApplicationRecord
     procedure.accuse_lecture? && termine?
   end
 
-  def track_can_passer_en_construction
-    if !revision.ineligibilite_enabled
-      yield
-      [true, true] # without eligibilite rules, we never reach dossier.champs.visible?, don't cache anything
-    else
-      from = can_passer_en_construction? # with eligibilite rules, self.champ[x].visible is cached by passing thru conditions checks
-      yield
-      champs.map(&:reset_visible) # we must reset self.champs[x].visible?, because an update occurred and we should re-evaluate champs[x] visibility
-      [from, can_passer_en_construction?]
-    end
-  end
-
   private
 
   def build_default_champs
diff --git a/app/views/users/dossiers/update.turbo_stream.haml b/app/views/users/dossiers/update.turbo_stream.haml
index 2cd14be47..0b7c13474 100644
--- a/app/views/users/dossiers/update.turbo_stream.haml
+++ b/app/views/users/dossiers/update.turbo_stream.haml
@@ -1,10 +1,7 @@
 = render partial: 'shared/dossiers/update_champs', locals: { to_show: @to_show, to_hide: @to_hide, to_update: @to_update, dossier: @dossier }
 
-- if !params.key?(:validate)
-  - if @can_passer_en_construction_was && !@can_passer_en_construction_is
-    = turbo_stream.append('contenu', render(Dossiers::InvalidIneligibiliteRulesComponent.new(dossier: @dossier)))
-  - else @ineligibilite_rules_is_computable
-    = turbo_stream.remove(dom_id(@dossier, :ineligibilite_rules_broken))
+- if params[:validate].present? && @dossier.revision.ineligibilite_enabled?
+  = turbo_stream.update dom_id(@dossier, :ineligibilite_rules_broken), render(Dossiers::InvalidIneligibiliteRulesComponent.new(dossier: @dossier, wrapped: false))
 
 - if @update_contact_information
   = turbo_stream.update "contact_information", partial: 'shared/dossiers/update_contact_information', locals: { dossier: @dossier, procedure: @dossier.procedure }
diff --git a/spec/controllers/users/dossiers_controller_spec.rb b/spec/controllers/users/dossiers_controller_spec.rb
index 11ce0e20f..62970f9f5 100644
--- a/spec/controllers/users/dossiers_controller_spec.rb
+++ b/spec/controllers/users/dossiers_controller_spec.rb
@@ -775,9 +775,11 @@ describe Users::DossiersController, type: :controller do
       let(:types_de_champ_public) { [{ type: :text }, { type: :integer_number }] }
       let(:text_champ) { dossier.project_champs_public.first }
       let(:number_champ) { dossier.project_champs_public.last }
+      let(:validate) { "true" }
       let(:submit_payload) do
         {
           id: dossier.id,
+          validate:,
           dossier: {
             groupe_instructeur_id: dossier.groupe_instructeur_id,
             champs_public_attributes: {
@@ -805,28 +807,36 @@ describe Users::DossiersController, type: :controller do
       end
       render_views
 
-      context 'when it switches from true to false' do
+      context 'when it becomes invalid' do
         let(:value) { must_be_greater_than + 1 }
 
         it 'raises popup' do
           subject
           dossier.reload
           expect(dossier.can_passer_en_construction?).to be_falsey
-          expect(assigns(:can_passer_en_construction_was)).to eq(true)
-          expect(assigns(:can_passer_en_construction_is)).to eq(false)
-          expect(response.body).to match(ActionView::RecordIdentifier.dom_id(dossier, :ineligibilite_rules_broken))
+          expect(response.body).to match(/aria-controls='modal-eligibilite-rules-dialog'[^>]*data-fr-opened='true'/)
         end
       end
 
-      context 'when it stays true' do
+      context 'when it says valid' do
         let(:value) { must_be_greater_than - 1 }
         it 'does nothing' do
           subject
           dossier.reload
           expect(dossier.can_passer_en_construction?).to be_truthy
-          expect(assigns(:can_passer_en_construction_was)).to eq(true)
-          expect(assigns(:can_passer_en_construction_is)).to eq(true)
-          expect(response.body).not_to have_selector("##{ActionView::RecordIdentifier.dom_id(dossier, :ineligibilite_rules_broken)}")
+          expect(response.body).to match(/aria-controls='modal-eligibilite-rules-dialog'[^>]*data-fr-opened='false'/)
+        end
+      end
+
+      context 'when not validating' do
+        let(:validate) { nil }
+        let(:value) { must_be_greater_than + 1 }
+
+        it 'does not render invalid ineligible modal' do
+          subject
+          dossier.reload
+          expect(dossier.can_passer_en_construction?).to be_falsey
+          expect(response.body).not_to include("aria-controls='modal-eligibilite-rules-dialog'")
         end
       end
     end
diff --git a/spec/system/users/dossier_ineligibilite_spec.rb b/spec/system/users/dossier_ineligibilite_spec.rb
index ef52ae42c..fd7a32c1b 100644
--- a/spec/system/users/dossier_ineligibilite_spec.rb
+++ b/spec/system/users/dossier_ineligibilite_spec.rb
@@ -28,21 +28,21 @@ describe 'Dossier Inéligibilité', js: true do
       visit brouillon_dossier_path(dossier)
       # no error while dossier is empty
       expect(page).to have_selector(:button, text: "Déposer le dossier", disabled: false)
-      expect(page).not_to have_content("Vous ne pouvez pas déposer votre dossier")
+      expect(page).to have_selector("#modal-eligibilite-rules-dialog", visible: false)
 
-      # does raise error when dossier is filled with condition that does not match
+      # does nothing when dossier is filled with condition that does not match
       within "#champ-1" do
         find("label", text: "Non").click
       end
       expect(page).to have_selector(:button, text: "Déposer le dossier", disabled: false)
-      expect(page).not_to have_content("Vous ne pouvez pas déposer votre dossier")
+      expect(page).to have_selector("#modal-eligibilite-rules-dialog", visible: false)
 
-      # raise error when dossier is filled with condition that matches
+      # open modal when dossier is filled with condition that matches
       within "#champ-1" do
         find("label", text: "Oui").click
       end
       expect(page).to have_selector(:button, text: "Déposer le dossier", disabled: true)
-      expect(page).to have_content("Vous ne pouvez pas déposer votre dossier")
+      expect(page).to have_selector("#modal-eligibilite-rules-dialog", visible: true)
 
       # reload page and see error
       visit brouillon_dossier_path(dossier)
@@ -51,6 +51,7 @@ describe 'Dossier Inéligibilité', js: true do
 
       # modal is closable, and we can change our dossier response to be eligible
       expect(page).to have_selector("#modal-eligibilite-rules-dialog", visible: true)
+      expect(page).to have_text("Vous ne pouvez pas déposer votre dossier")
       within("#modal-eligibilite-rules-dialog") { click_on "Fermer" }
       expect(page).to have_selector("#modal-eligibilite-rules-dialog", visible: false)
 
@@ -78,7 +79,7 @@ describe 'Dossier Inéligibilité', js: true do
       visit brouillon_dossier_path(dossier)
       # no error while dossier is empty
       expect(page).to have_selector(:button, text: "Déposer le dossier", disabled: false)
-      expect(page).not_to have_content("Vous ne pouvez pas déposer votre dossier")
+      expect(page).to have_selector("#modal-eligibilite-rules-dialog", visible: false)
 
       # first condition matches (so ineligible), cannot submit dossier and error message is clear
       within "#champ-#{first_tdc.stable_id}" do
@@ -148,14 +149,14 @@ describe 'Dossier Inéligibilité', js: true do
       visit brouillon_dossier_path(dossier)
       # no error while dossier is empty
       expect(page).to have_selector(:button, text: "Déposer le dossier", disabled: false)
-      expect(page).not_to have_content("Vous ne pouvez pas déposer votre dossier")
+      expect(page).to have_selector("#modal-eligibilite-rules-dialog", visible: false)
 
       # only one condition is matches, can submit dossier
       within "#champ-#{first_tdc.stable_id}" do
         find("label", text: "Oui").click
       end
       expect(page).to have_selector(:button, text: "Déposer le dossier", disabled: false)
-      expect(page).not_to have_content("Vous ne pouvez pas déposer votre dossier")
+      expect(page).to have_selector("#modal-eligibilite-rules-dialog", visible: false)
 
       # Now test dossier modification
       click_on "Déposer le dossier"
@@ -196,7 +197,7 @@ describe 'Dossier Inéligibilité', js: true do
     scenario 'ineligibilite rules without validation on champ ensure to re-process cached champs.visible' do
       visit brouillon_dossier_path(dossier)
       expect(page).to have_selector(:button, text: "Déposer le dossier", disabled: false)
-      expect(page).not_to have_content("Vous ne pouvez pas déposer votre dossier")
+      expect(page).to have_selector("#modal-eligibilite-rules-dialog", visible: false)
 
       within "#champ-1" do
         find("label", text: "Non").click
diff --git a/spec/views/shared/dossiers/_edit.html.haml_spec.rb b/spec/views/shared/dossiers/_edit.html.haml_spec.rb
index 4271d129b..787e9cf07 100644
--- a/spec/views/shared/dossiers/_edit.html.haml_spec.rb
+++ b/spec/views/shared/dossiers/_edit.html.haml_spec.rb
@@ -160,6 +160,7 @@ describe 'shared/dossiers/edit', type: :view do
 
     before do
       allow(dossier).to receive(:can_passer_en_construction?).and_return(false)
+      allow(dossier.revision).to receive(:ineligibilite_enabled?).and_return(true)
     end
 
     it 'renders broken transitions rules dialog' do

From 1c6d012f4472fc376fa7f24237bd66981c9394e5 Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Wed, 11 Dec 2024 15:20:59 +0100
Subject: [PATCH 5/6] fix(autosave): validate select onChangeable et on retry

---
 app/javascript/controllers/autosave_controller.ts | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/app/javascript/controllers/autosave_controller.ts b/app/javascript/controllers/autosave_controller.ts
index 96aca094d..455ed6fed 100644
--- a/app/javascript/controllers/autosave_controller.ts
+++ b/app/javascript/controllers/autosave_controller.ts
@@ -80,11 +80,13 @@ export class AutosaveController extends ApplicationController {
         // Wait next tick so champs having JS can interact
         // with form elements before extracting form data.
         setTimeout(() => {
-          this.enqueueAutosaveRequest();
+          this.enqueueAutosaveWithValidationRequest();
           this.showConditionnalSpinner(target);
         }, 0);
       },
-      inputable: (target) => this.enqueueOnInput(target, true),
+      inputable: (target) => {
+        this.enqueueOnInput(target, true);
+      },
       hidden: (target) => {
         // In comboboxes we dispatch a "change" event on hidden inputs to trigger autosave.
         // We want to debounce them.
@@ -152,7 +154,7 @@ export class AutosaveController extends ApplicationController {
 
   private didRequestRetry() {
     if (this.#needsRetry) {
-      this.enqueueAutosaveRequest();
+      this.enqueueAutosaveWithValidationRequest();
     }
   }
 

From d8ee89db29a0a73733267076281d70c45284fc30 Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Mon, 16 Dec 2024 21:18:59 +0100
Subject: [PATCH 6/6] fix(dossier): ineligibilite modal en construction/fork

---
 .../invalid_ineligibilite_rules_component.html.haml             | 2 +-
 app/views/users/dossiers/update.turbo_stream.haml               | 2 +-
 spec/views/shared/dossiers/_edit.html.haml_spec.rb              | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/components/dossiers/invalid_ineligibilite_rules_component/invalid_ineligibilite_rules_component.html.haml b/app/components/dossiers/invalid_ineligibilite_rules_component/invalid_ineligibilite_rules_component.html.haml
index 061b44c06..46d29160b 100644
--- a/app/components/dossiers/invalid_ineligibilite_rules_component/invalid_ineligibilite_rules_component.html.haml
+++ b/app/components/dossiers/invalid_ineligibilite_rules_component/invalid_ineligibilite_rules_component.html.haml
@@ -16,7 +16,7 @@
               %p= error_message
 
 - if wrapped?
-  %div{ id: dom_id(@dossier, :ineligibilite_rules_broken) }
+  #ineligibilite_rules_modal
     = modal_content
 - else
   = modal_content
diff --git a/app/views/users/dossiers/update.turbo_stream.haml b/app/views/users/dossiers/update.turbo_stream.haml
index 0b7c13474..cec0f8c65 100644
--- a/app/views/users/dossiers/update.turbo_stream.haml
+++ b/app/views/users/dossiers/update.turbo_stream.haml
@@ -1,7 +1,7 @@
 = render partial: 'shared/dossiers/update_champs', locals: { to_show: @to_show, to_hide: @to_hide, to_update: @to_update, dossier: @dossier }
 
 - if params[:validate].present? && @dossier.revision.ineligibilite_enabled?
-  = turbo_stream.update dom_id(@dossier, :ineligibilite_rules_broken), render(Dossiers::InvalidIneligibiliteRulesComponent.new(dossier: @dossier, wrapped: false))
+  = turbo_stream.update :ineligibilite_rules_modal, render(Dossiers::InvalidIneligibiliteRulesComponent.new(dossier: @dossier, wrapped: false))
 
 - if @update_contact_information
   = turbo_stream.update "contact_information", partial: 'shared/dossiers/update_contact_information', locals: { dossier: @dossier, procedure: @dossier.procedure }
diff --git a/spec/views/shared/dossiers/_edit.html.haml_spec.rb b/spec/views/shared/dossiers/_edit.html.haml_spec.rb
index 787e9cf07..047d05c6a 100644
--- a/spec/views/shared/dossiers/_edit.html.haml_spec.rb
+++ b/spec/views/shared/dossiers/_edit.html.haml_spec.rb
@@ -164,7 +164,7 @@ describe 'shared/dossiers/edit', type: :view do
     end
 
     it 'renders broken transitions rules dialog' do
-      expect(subject).to have_selector("##{ActionView::RecordIdentifier.dom_id(dossier, :ineligibilite_rules_broken)}")
+      expect(subject).to have_selector("#ineligibilite_rules_modal [data-fr-opened='true']")
     end
   end
 end