DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

Merge pull request #9472 from demarches-simplifiees/can-route-from-regions-champ

Browse source 
ETQ admin je peux router depuis un type de champ régions
This commit is contained in:
Eric Leroy-Terquem 2023-09-13 08:34:02 +00:00 committed by GitHub
commit 49d02274b5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
10 changed files with 83 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/components/procedure/instructeurs_options_component/instructeurs_options_component.fr.yml
View file

@ -4,7 +4,7 @@ fr:
Le routage permet d’acheminer les dossiers vers différents groupes d’instructeurs.
routing_configuration_notice_2_html: |
<p>Pour le configurer, votre formulaire doit comporter
au moins un champ de type « choix simple » ou « départements ».</p>
au moins un champ de type « choix simple », « départements » ou « régions ».</p>
<p>Ajoutez ce champ dans la page <a href="%{path}">« Configuration des champs »</a>.</p>
delete_title: Aucun dossier ne sera supprimé. Les groupes d'instructeurs vont être supprimés. Seuls les instructeurs du groupe « %{defaut_label} » resteront affectés à la démarche.
delete_confirmation: |

6
app/components/procedure/one_groupe_management_component.rb
View file

@ -81,6 +81,8 @@ class Procedure::OneGroupeManagementComponent < ApplicationComponent
case @revision.types_de_champ_public.find_by(stable_id: targeted_champ.stable_id).type_champ
when TypeDeChamp.type_champs.fetch(:departements)
departements_for_select
when TypeDeChamp.type_champs.fetch(:regions)
regions_for_select
when TypeDeChamp.type_champs.fetch(:drop_down_list)
targeted_champ
.options(@revision.types_de_champ_public)
@ -91,4 +93,8 @@ class Procedure::OneGroupeManagementComponent < ApplicationComponent
def departements_for_select
APIGeoService.departements.map { ["#{_1[:code]} – #{_1[:name]}", constant(_1[:code]).to_json] }
end
def regions_for_select
APIGeoService.regions.map { ["#{_1[:code]} – #{_1[:name]}", constant(_1[:code]).to_json] }
end
end

40
app/controllers/administrateurs/groupe_instructeurs_controller.rb
View file

@ -43,23 +43,13 @@ module Administrateurs
case tdc.type_champ
when TypeDeChamp.type_champs.fetch(:departements)
tdc_options = APIGeoService.departements.map { ["#{_1[:code]} – #{_1[:name]}", _1[:code]] }
tdc_options.each do |code_and_name, code|
routing_rule = ds_eq(champ_value(stable_id), constant(code))
@procedure
.groupe_instructeurs
.find_or_create_by(label: code_and_name)
.update(instructeurs: [current_administrateur.instructeur], routing_rule:)
end
create_groups_from_territorial_tdc(tdc_options, stable_id)
when TypeDeChamp.type_champs.fetch(:regions)
tdc_options = APIGeoService.regions.map { ["#{_1[:code]} – #{_1[:name]}", _1[:code]] }
create_groups_from_territorial_tdc(tdc_options, stable_id)
when TypeDeChamp.type_champs.fetch(:drop_down_list)
tdc_options = tdc.drop_down_options.reject(&:empty?)
tdc_options.each do |option_label|
routing_rule = ds_eq(champ_value(stable_id), constant(option_label))
@procedure
.groupe_instructeurs
.find_or_create_by(label: option_label)
.update(instructeurs: [current_administrateur.instructeur], routing_rule:)
end
create_groups_from_drop_down_list_tdc(tdc_options, stable_id)
end
if tdc.drop_down_other?
@ -460,5 +450,25 @@ module Administrateurs
def flash_message_for_invalid_csv
flash[:alert] = "Importation impossible, veuillez importer un csv suivant #{view_context.link_to('ce modèle', "/csv/import-instructeurs-test.csv")} pour une procédure sans routage ou #{view_context.link_to('celui-ci', "/csv/#{I18n.locale}/import-groupe-test.csv")} pour une procédure routée"
end
def create_groups_from_territorial_tdc(tdc_options, stable_id)
tdc_options.each do |label, code|
routing_rule = ds_eq(champ_value(stable_id), constant(code))
@procedure
.groupe_instructeurs
.find_or_create_by(label: label)
.update(instructeurs: [current_administrateur.instructeur], routing_rule:)
end
end
def create_groups_from_drop_down_list_tdc(tdc_options, stable_id)
tdc_options.each do |label|
routing_rule = ds_eq(champ_value(stable_id), constant(label))
@procedure
.groupe_instructeurs
.find_or_create_by(label: label)
.update(instructeurs: [current_administrateur.instructeur], routing_rule:)
end
end
end
end

2
app/models/groupe_instructeur.rb
View file

@ -107,6 +107,8 @@ class GroupeInstructeur < ApplicationRecord
options = case routing_tdc.type_champ
when TypeDeChamp.type_champs.fetch(:departements)
APIGeoService.departements.map { _1[:code] }
when TypeDeChamp.type_champs.fetch(:regions)
APIGeoService.regions.map { _1[:code] }
when TypeDeChamp.type_champs.fetch(:drop_down_list)
routing_tdc.options_with_drop_down_other
end

2
app/models/logic/champ_value.rb
View file

@ -44,7 +44,7 @@ class Logic::ChampValue < Logic::Term
targeted_champ.selected
when "Champs::MultipleDropDownListChamp"
targeted_champ.selected_options
when "Champs::DepartementChamp"
when "Champs::DepartementChamp", "Champs::RegionChamp"
targeted_champ.code
end
end

2
app/models/procedure_revision.rb
View file

@ -224,7 +224,7 @@ class ProcedureRevision < ApplicationRecord
end
def routable_types_de_champ
types_de_champ_public.filter { |tdc| [:drop_down_list, :departements].include?(tdc.type_champ.to_sym) }
types_de_champ_public.filter { |tdc| [:drop_down_list, :departements, :regions].include?(tdc.type_champ.to_sym) }
end
private

20
spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
View file

@ -761,6 +761,26 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
expect(procedure3.routing_enabled).to be_truthy
end
end
context 'with a regions type de champ' do
let!(:procedure3) do
create(:procedure,
types_de_champ_public: [{ type: :regions }],
administrateurs: [admin])
end
let!(:regions_tdc) { procedure3.draft_revision.types_de_champ.first }
before { post :create_simple_routing, params: { procedure_id: procedure3.id, create_simple_routing: { stable_id: regions_tdc.stable_id } } }
it do
expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure3))
expect(flash.notice).to eq 'Les groupes instructeurs ont été ajoutés'
expect(procedure3.groupe_instructeurs.pluck(:label)).to include("01 – Guadeloupe")
expect(procedure3.reload.defaut_groupe_instructeur.routing_rule).to eq(ds_eq(champ_value(regions_tdc.stable_id), constant('01')))
expect(procedure3.routing_enabled).to be_truthy
end
end
end
describe '#wizard' do

6
spec/models/logic/champ_value_spec.rb
View file

@ -76,6 +76,12 @@ describe Logic::ChampValue do
it { is_expected.to eq(true) }
end
context 'region tdc' do
let(:champ) { create(:champ_regions, value: 'La Réunion') }
it { is_expected.to eq('04') }
end
describe 'errors' do
let(:champ) { create(:champ) }

3
spec/models/procedure_revision_spec.rb
View file

@ -948,9 +948,10 @@ describe ProcedureRevision do
p.draft_revision.add_type_de_champ(type_champ: :text, libelle: 'l1')
p.draft_revision.add_type_de_champ(type_champ: :drop_down_list, libelle: 'l2')
p.draft_revision.add_type_de_champ(type_champ: :departements, libelle: 'l3')
p.draft_revision.add_type_de_champ(type_champ: :regions, libelle: 'l4')
end
end
it { expect(draft.routable_types_de_champ.pluck(:libelle)).to eq(['l2', 'l3']) }
it { expect(draft.routable_types_de_champ.pluck(:libelle)).to eq(['l2', 'l3', 'l4']) }
end
end

19
spec/models/routing_engine_spec.rb
View file

@ -92,6 +92,25 @@ describe RoutingEngine, type: :model do
end
end
context 'with a regions type de champ' do
let(:procedure) do
create(:procedure, types_de_champ_public: [{ type: :regions }]).tap do |p|
p.groupe_instructeurs.create(label: 'a third group')
end
end
let(:regions_tdc) { procedure.draft_revision.types_de_champ.first }
context 'with a matching rule' do
before do
gi_2.update(routing_rule: ds_eq(champ_value(regions_tdc.stable_id), constant('04')))
dossier.champs.first.update(value: 'La Réunion')
end
it { is_expected.to eq(gi_2) }
end
end
context 'routing rules priorities' do
let(:procedure) do
create(:procedure,