diff --git a/app/components/procedure/card/instructeurs_component/instructeurs_component.html.haml b/app/components/procedure/card/instructeurs_component/instructeurs_component.html.haml index 463ed893f..495380710 100644 --- a/app/components/procedure/card/instructeurs_component/instructeurs_component.html.haml +++ b/app/components/procedure/card/instructeurs_component/instructeurs_component.html.haml @@ -1,7 +1,7 @@ .fr-col-6.fr-col-md-4.fr-col-lg-3 = link_to admin_procedure_groupe_instructeurs_path(@procedure), id: 'groupe-instructeurs', class: 'fr-tile fr-enlarge-link' do .fr-tile__body.flex.justify-between - - if @procedure.routee? + - if @procedure.routing_enabled? %div %span.icon.accept %p.fr-tile-status-accept Validé @@ -11,7 +11,7 @@ %p.fr-tile-status-todo À faire %div .line-count.fr-my-1w - - if @procedure.routee? + - if @procedure.routing_enabled? %p.fr-tag= @procedure.groupe_instructeurs.size - else %p.fr-tag= @procedure.instructeurs.size diff --git a/app/controllers/administrateurs/groupe_instructeurs_controller.rb b/app/controllers/administrateurs/groupe_instructeurs_controller.rb index fe73319a2..8ccd47414 100644 --- a/app/controllers/administrateurs/groupe_instructeurs_controller.rb +++ b/app/controllers/administrateurs/groupe_instructeurs_controller.rb @@ -128,7 +128,7 @@ module Administrateurs create_instructeur(instructeur_email) end - if procedure.routee? + if procedure.routing_enabled? instructeurs.each do |instructeur| groupe_instructeur.add(instructeur) end @@ -153,7 +153,7 @@ module Administrateurs end end - if procedure.routee? + if procedure.routing_enabled? redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur) else redirect_to admin_procedure_groupe_instructeurs_path(procedure) @@ -165,7 +165,7 @@ module Administrateurs flash[:alert] = "Suppression impossible : il doit y avoir au moins un instructeur dans le groupe" else instructeur = Instructeur.find(instructeur_id) - if procedure.routee? + if procedure.routing_enabled? if groupe_instructeur.remove(instructeur) flash[:notice] = "L’instructeur « #{instructeur.email} » a été retiré du groupe." GroupeInstructeurMailer @@ -183,7 +183,7 @@ module Administrateurs end end - if procedure.routee? + if procedure.routing_enabled? redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur) else redirect_to admin_procedure_groupe_instructeurs_path(procedure) diff --git a/app/controllers/users/dossiers_controller.rb b/app/controllers/users/dossiers_controller.rb index 329761f19..16545c391 100644 --- a/app/controllers/users/dossiers_controller.rb +++ b/app/controllers/users/dossiers_controller.rb @@ -432,7 +432,7 @@ module Users end def should_fill_groupe_instructeur? - !@dossier.procedure.routee? && @dossier.groupe_instructeur_id.nil? + !@dossier.procedure.routing_enabled? && @dossier.groupe_instructeur_id.nil? end def defaut_groupe_instructeur diff --git a/app/helpers/procedure_helper.rb b/app/helpers/procedure_helper.rb index 7184f39e6..2b2fa2609 100644 --- a/app/helpers/procedure_helper.rb +++ b/app/helpers/procedure_helper.rb @@ -49,7 +49,7 @@ module ProcedureHelper end def can_manage_groupe_instructeurs?(procedure) - procedure.routee? && current_administrateur&.owns?(procedure) + procedure.routing_enabled? && current_administrateur&.owns?(procedure) end def can_send_groupe_message?(procedure) diff --git a/app/models/concerns/tags_substitution_concern.rb b/app/models/concerns/tags_substitution_concern.rb index 1d969d7b8..14827afa1 100644 --- a/app/models/concerns/tags_substitution_concern.rb +++ b/app/models/concerns/tags_substitution_concern.rb @@ -196,7 +196,7 @@ module TagsSubstitutionConcern end routage_tags = [] - if procedure.routee? + if procedure.routing_enabled? routage_tags = ROUTAGE_TAGS end diff --git a/app/models/dossier.rb b/app/models/dossier.rb index 768cc3b43..3b8df556b 100644 --- a/app/models/dossier.rb +++ b/app/models/dossier.rb @@ -615,11 +615,11 @@ class Dossier < ApplicationRecord end def show_groupe_instructeur_details? - procedure.routee? && groupe_instructeur.present? && (!procedure.feature_enabled?(:procedure_routage_api) || !defaut_groupe_instructeur?) + procedure.routing_enabled? && groupe_instructeur.present? && (!procedure.feature_enabled?(:procedure_routage_api) || !defaut_groupe_instructeur?) end def show_groupe_instructeur_selector? - procedure.routee? && !procedure.feature_enabled?(:procedure_routage_api) + procedure.routing_enabled? && !procedure.feature_enabled?(:procedure_routage_api) end def assign_to_groupe_instructeur(groupe_instructeur, author = nil) @@ -1085,7 +1085,7 @@ class Dossier < ApplicationRecord ['Instructeurs', followers_instructeurs.map(&:email).join(' ')] ] - if procedure.routee? + if procedure.routing_enabled? columns << ['Groupe instructeur', groupe_instructeur.label] end columns + self.class.champs_for_export(champs + champs_private, types_de_champ) diff --git a/app/models/procedure.rb b/app/models/procedure.rb index 678decca7..283b7571d 100644 --- a/app/models/procedure.rb +++ b/app/models/procedure.rb @@ -694,16 +694,12 @@ class Procedure < ApplicationRecord revisions.size - 2 end - def routee? - routing_enabled? - end - def instructeurs_self_management? - routee? || instructeurs_self_management_enabled? + routing_enabled? || instructeurs_self_management_enabled? end def defaut_groupe_instructeur_for_new_dossier - if !routee? || feature_enabled?(:procedure_routage_api) + if !routing_enabled? || feature_enabled?(:procedure_routage_api) defaut_groupe_instructeur end end diff --git a/app/views/administrateurs/groupe_instructeurs/_instructeurs.html.haml b/app/views/administrateurs/groupe_instructeurs/_instructeurs.html.haml index c9a42f9ef..b5ec688d7 100644 --- a/app/views/administrateurs/groupe_instructeurs/_instructeurs.html.haml +++ b/app/views/administrateurs/groupe_instructeurs/_instructeurs.html.haml @@ -3,7 +3,7 @@ .card-title Affectation des instructeurs = form_for :instructeur, url: { action: :add_instructeur, id: groupe_instructeur.id }, html: { class: 'form' } do |f| .instructeur-wrapper - - if !procedure.routee? + - if !procedure.routing_enabled? %p.notice Entrez les adresses email des instructeurs que vous souhaitez affecter à cette démarche - if disabled_as_super_admin @@ -31,7 +31,7 @@ %span.icon.person #{instructeur.email} - - confirmation_message = procedure.routee? ? "Êtes-vous sûr de vouloir retirer l’instructeur « #{instructeur.email} » du groupe « #{groupe_instructeur.label} » ?" : "Êtes-vous sûr de vouloir retirer l’instructeur « #{instructeur.email} » de la démarche ?" + - confirmation_message = procedure.routing_enabled? ? "Êtes-vous sûr de vouloir retirer l’instructeur « #{instructeur.email} » du groupe « #{groupe_instructeur.label} » ?" : "Êtes-vous sûr de vouloir retirer l’instructeur « #{instructeur.email} » de la démarche ?" %td.actions= button_to 'Retirer', { action: :remove_instructeur, id: groupe_instructeur.id }, { method: :delete, diff --git a/app/views/administrateurs/groupe_instructeurs/index.html.haml b/app/views/administrateurs/groupe_instructeurs/index.html.haml index a7e42cf52..ac4d60c66 100644 --- a/app/views/administrateurs/groupe_instructeurs/index.html.haml +++ b/app/views/administrateurs/groupe_instructeurs/index.html.haml @@ -1,4 +1,4 @@ -- if @procedure.routee? +- if @procedure.routing_enabled? = render partial: 'administrateurs/breadcrumbs', locals: { steps: [[t('.procedures'), admin_procedures_path], [@procedure.libelle.truncate_words(10), admin_procedure_path(@procedure)], @@ -19,7 +19,7 @@ instructeurs: @instructeurs, available_instructeur_emails: @available_instructeur_emails, disabled_as_super_admin: administrateur_as_manager? } - - if !@procedure.routee? + - if !@procedure.routing_enabled? = render partial: 'administrateurs/groupe_instructeurs/instructeurs_self_management', locals: { procedure: @procedure } = render partial: 'administrateurs/groupe_instructeurs/routing', locals: { procedure: @procedure } diff --git a/app/views/administrateurs/procedures/_procedures_list.html.haml b/app/views/administrateurs/procedures/_procedures_list.html.haml index 7c0d12d9b..b58e0e573 100644 --- a/app/views/administrateurs/procedures/_procedures_list.html.haml +++ b/app/views/administrateurs/procedures/_procedures_list.html.haml @@ -26,7 +26,7 @@ .admin-procedures-list-row.actions.flex.justify-between %div - - if procedure.routee? + - if procedure.routing_enabled? %span.icon.person %span.badge.baseline= procedure.groupe_instructeurs.count - else @@ -80,4 +80,3 @@ %span.icon.unarchive .dropdown-description %h4= t('administrateurs.dropdown_actions.restore') - diff --git a/app/views/instructeurs/dossiers/print.html.haml b/app/views/instructeurs/dossiers/print.html.haml index 12c2fa5d0..84cd9cf23 100644 --- a/app/views/instructeurs/dossiers/print.html.haml +++ b/app/views/instructeurs/dossiers/print.html.haml @@ -14,7 +14,7 @@ %h2 Formulaire - champs = @dossier.champs -- if champs.any? || @dossier.procedure.routee? +- if champs.any? || @dossier.procedure.routing_enabled? = render partial: "shared/dossiers/champs", locals: { champs: champs, dossier: @dossier, demande_seen_at: nil, profile: 'instructeur' } %h2 Annotations privées diff --git a/app/views/instructeurs/groupe_instructeurs/show.html.haml b/app/views/instructeurs/groupe_instructeurs/show.html.haml index ce16c307d..0b6fb642e4 100644 --- a/app/views/instructeurs/groupe_instructeurs/show.html.haml +++ b/app/views/instructeurs/groupe_instructeurs/show.html.haml @@ -1,4 +1,4 @@ -- if @procedure.routee? +- if @procedure.routing_enabled? - content_for(:title, "Instructeurs du groupe #{@groupe_instructeur.label}") = render partial: 'administrateurs/breadcrumbs', @@ -14,7 +14,7 @@ .container.groupe-instructeur %h1 - - if @procedure.routee? + - if @procedure.routing_enabled? Groupe « #{@groupe_instructeur.label} » - else Démarche « #{@procedure.libelle} » @@ -38,7 +38,7 @@ - @instructeurs.each do |instructeur| %tr %td= instructeur.email - - confirmation_message = @procedure.routee? ? "Êtes-vous sûr de vouloir retirer l’instructeur « #{instructeur.email} » du groupe « #{@groupe_instructeur.label} » ?" : "Êtes-vous sûr de vouloir retirer l’instructeur « #{instructeur.email} » de la démarche ?" + - confirmation_message = @procedure.routing_enabled? ? "Êtes-vous sûr de vouloir retirer l’instructeur « #{instructeur.email} » du groupe « #{@groupe_instructeur.label} » ?" : "Êtes-vous sûr de vouloir retirer l’instructeur « #{instructeur.email} » de la démarche ?" %td.actions= button_to 'retirer', { action: :remove_instructeur }, { method: :delete, diff --git a/app/views/instructeurs/procedures/_header.html.haml b/app/views/instructeurs/procedures/_header.html.haml index 8e54952e1..5a324dc90 100644 --- a/app/views/instructeurs/procedures/_header.html.haml +++ b/app/views/instructeurs/procedures/_header.html.haml @@ -8,7 +8,7 @@ | - if can_manage_groupe_instructeurs?(procedure) = link_to t('instructeurs.dossiers.header.banner.instructeurs'), admin_procedure_groupe_instructeurs_path(procedure), class: 'header-link' - - elsif procedure.routee? + - elsif procedure.routing_enabled? = link_to t('instructeurs.dossiers.header.banner.instructeurs'), instructeur_groupes_path(procedure), class: 'header-link' - else = link_to t('instructeurs.dossiers.header.banner.instructeurs'), instructeur_groupe_path(procedure, procedure.defaut_groupe_instructeur), class: 'header-link' @@ -16,4 +16,3 @@ - if can_send_groupe_message?(procedure) | = link_to t('instructeurs.dossiers.header.banner.contact_users'), email_usagers_instructeur_procedure_path(procedure), class: 'header-link' - diff --git a/app/views/shared/dossiers/_demande.html.haml b/app/views/shared/dossiers/_demande.html.haml index 3f7957e23..263b7fdc6 100644 --- a/app/views/shared/dossiers/_demande.html.haml +++ b/app/views/shared/dossiers/_demande.html.haml @@ -34,6 +34,6 @@ .tab-title= t('views.shared.dossiers.demande.form') - champs = dossier.champs.includes(:type_de_champ) - - if champs.any? || dossier.procedure.routee? + - if champs.any? || dossier.procedure.routing_enabled? .card = render partial: "shared/dossiers/champs", locals: { champs: champs, dossier: dossier, demande_seen_at: demande_seen_at, profile: profile } diff --git a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb index c860f010a..3ba8497e5 100644 --- a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb +++ b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb @@ -276,7 +276,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do it { expect(gi_1_2.instructeurs.pluck(:email)).to include(*new_instructeur_emails) } it { expect(flash.notice).to be_present } it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_2)) } - it { expect(procedure.routee?).to be_truthy } + it { expect(procedure.routing_enabled?).to be_truthy } it "calls GroupeInstructeurMailer with the right groupe and instructeurs" do expect(GroupeInstructeurMailer).to have_received(:add_instructeurs).with( gi_1_2, diff --git a/spec/factories/dossier.rb b/spec/factories/dossier.rb index b12591149..6377102bb 100644 --- a/spec/factories/dossier.rb +++ b/spec/factories/dossier.rb @@ -4,7 +4,7 @@ FactoryBot.define do state { Dossier.states.fetch(:brouillon) } user { association :user } - groupe_instructeur { procedure.routee? ? nil : procedure.defaut_groupe_instructeur } + groupe_instructeur { procedure.routing_enabled? ? nil : procedure.defaut_groupe_instructeur } revision { procedure.active_revision } individual { association(:individual, :empty, dossier: instance, strategy: :build) if procedure.for_individual? } diff --git a/spec/models/concern/tags_substitution_concern_spec.rb b/spec/models/concern/tags_substitution_concern_spec.rb index 2911511bc..93c55f63a 100644 --- a/spec/models/concern/tags_substitution_concern_spec.rb +++ b/spec/models/concern/tags_substitution_concern_spec.rb @@ -86,12 +86,12 @@ describe TagsSubstitutionConcern, type: :model do procedure.reload end - it { expect(procedure.routee?).to eq(true) } + it { expect(procedure.routing_enabled?).to eq(true) } it { is_expected.to eq(label) } end context 'and the dossier has no groupe instructeur' do - it { expect(procedure.routee?).to eq(false) } + it { expect(procedure.routing_enabled?).to eq(false) } it { is_expected.to eq('défaut') } end end