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

remove previous routing sytem code

Browse source
This commit is contained in:
Eric Leroy-Terquem 2023-06-12 22:00:41 +02:00
parent 30f6aaed34
commit 7598b0a513
14 changed files with 0 additions and 99 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -282,15 +282,6 @@ module Administrateurs
end
end
def update_routing_criteria_name
if procedure.update(routing_criteria_name: routing_criteria_name)
flash[:notice] = "Le libellé est maintenant « #{procedure.routing_criteria_name} »."
else
flash[:alert] = "Le libellé du routage doit être rempli."
end
redirect_to admin_procedure_groupe_instructeurs_path(procedure)
end
def update_instructeurs_self_management_enabled
procedure.update!(instructeurs_self_management_enabled_params)
@ -415,10 +406,6 @@ module Administrateurs
.order(:email)
end
def routing_criteria_name
params[:procedure][:routing_criteria_name]
end
def available_instructeur_emails
all = current_administrateur.instructeurs.map(&:email)
assigned = groupe_instructeur.instructeurs.map(&:email)

34
app/controllers/users/dossiers_controller.rb
View file

@ -487,32 +487,6 @@ module Users
redirect_to dossier_path(dossier)
end
def should_change_groupe_instructeur?
if params[:dossier].key?(:groupe_instructeur_id)
groupe_instructeur_id = params[:dossier][:groupe_instructeur_id]
if groupe_instructeur_id.nil?
@dossier.groupe_instructeur_id.present?
else
@dossier.groupe_instructeur_id != groupe_instructeur_id.to_i
end
end
end
def groupe_instructeur_from_params
groupe_instructeur_id = params[:dossier][:groupe_instructeur_id]
if groupe_instructeur_id.present?
@dossier.procedure.groupe_instructeurs.find(groupe_instructeur_id)
end
end
def should_fill_groupe_instructeur?
!@dossier.procedure.routing_enabled? && @dossier.groupe_instructeur_id.nil?
end
def defaut_groupe_instructeur
@dossier.procedure.defaut_groupe_instructeur
end
def update_dossier_and_compute_errors
errors = []
@dossier.assign_attributes(champs_public_params)
@ -523,10 +497,6 @@ module Users
errors += format_errors(errors: @dossier.errors)
end
if should_change_groupe_instructeur?
@dossier.assign_to_groupe_instructeur(groupe_instructeur_from_params)
end
errors
end
@ -537,10 +507,6 @@ module Users
errors += format_errors(errors: @dossier.errors)
errors += format_errors(errors: @dossier.check_mandatory_and_visible_champs)
if should_fill_groupe_instructeur?
@dossier.assign_to_groupe_instructeur(defaut_groupe_instructeur)
end
RoutingEngine.compute(@dossier)
errors

8
app/models/dossier.rb
View file

@ -664,14 +664,6 @@ class Dossier < ApplicationRecord
procedure.discarded? || (brouillon? && !procedure.dossier_can_transition_to_en_construction?)
end
def show_groupe_instructeur_details?
procedure.routing_enabled? && groupe_instructeur.present? && (!procedure.feature_enabled?(:procedure_routage_api) || !defaut_groupe_instructeur?) && !procedure.feature_enabled?(:routing_rules)
end
def show_groupe_instructeur_selector?
procedure.routing_enabled? && !procedure.feature_enabled?(:procedure_routage_api) && !procedure.feature_enabled?(:routing_rules)
end
def assign_to_groupe_instructeur(groupe_instructeur, author = nil)
return if groupe_instructeur.present? && groupe_instructeur.procedure != procedure
return if self.groupe_instructeur == groupe_instructeur

1
app/models/procedure.rb
View file

@ -356,7 +356,6 @@ class Procedure < ApplicationRecord
validates :api_entreprise_token, jwt_token: true, allow_blank: true
validates :api_particulier_token, format: { with: /\A[A-Za-z0-9\-_=.]{15,}\z/ }, allow_blank: true
validate :validate_auto_archive_on_in_the_future, if: :will_save_change_to_auto_archive_on?
validates :routing_criteria_name, presence: true, allow_blank: false
before_save :update_juridique_required
after_initialize :ensure_path_exists

1
app/models/routing_engine.rb
View file

@ -1,6 +1,5 @@
module RoutingEngine
def self.compute(dossier)
return if !dossier.procedure.feature_enabled?(:routing_rules)
return if dossier.forced_groupe_instructeur
matching_groupe = dossier.procedure.groupe_instructeurs.active.reject(&:routing_to_configure?).find do |gi|

14
app/views/shared/dossiers/_champs.html.haml
View file

@ -1,15 +1 @@
- if dossier.show_groupe_instructeur_details?
.fr-my-4v.fr-px-4v
.flex
%p.champ-label.flex-grow= dossier.procedure.routing_criteria_name
%p.fr-mb-0.fr-text--xs
- if demande_seen_at && dossier.groupe_instructeur_updated_at && demande_seen_at < dossier.groupe_instructeur_updated_at
%span.fr-badge.fr-badge--sm{ class: badge_class_if_unseen(demande_seen_at, dossier.groupe_instructeur_updated_at) }
= t(:updated_at, scope: [:views, :shared, :dossiers, :form], datetime: try_format_datetime(dossier.groupe_instructeur_updated_at))
.champ-content{ class: highlight_if_unseen_class(demande_seen_at, dossier.groupe_instructeur_updated_at) }
%p= dossier.groupe_instructeur.label
= render ViewableChamp::SectionComponent.new(champs: champs, demande_seen_at: demande_seen_at, profile: profile)

9
app/views/shared/dossiers/_edit.html.haml
View file

@ -19,15 +19,6 @@
- if dossier.procedure.notice.attached?
= render Dsfr::DownloadComponent.new(attachment: dossier.procedure.notice , url: notice_url(dossier.procedure), name: t("views.shared.dossiers.edit.notice"), ephemeral_link: administrateur_signed_in?)
- if dossier.show_groupe_instructeur_selector?
%span{ data: { controller: 'autosave' } }
= f.label :groupe_instructeur_id do
= dossier.procedure.routing_criteria_name
%span.mandatory *
= f.select :groupe_instructeur_id,
dossier.procedure.groupe_instructeurs.active.map { |gi| [gi.label, gi.id] },
{ include_blank: dossier.brouillon? }
= render EditableChamp::SectionComponent.new(champs: dossier_for_editing.champs_public)
- if dossier.pending_correction?

1
config/initializers/flipper.rb
View file

@ -17,7 +17,6 @@ features = [
:dossier_pdf_vide,
:hide_instructeur_email,
:procedure_routage_api,
:routing_rules,
:groupe_instructeur_api_hack,
:rerouting
]

1
config/locales/models/procedure/en.yml
View file

@ -31,7 +31,6 @@ en:
declarative_with_state/en_instruction: Instruction
declarative_with_state/accepte: Accepted
api_particulier_token: Token API Particulier
routing_criteria_name: Routing criteria name
initiated_mail: File sorted for processing notification email
received_mail: File submitted notification email
closed_mail: File sorted as accepted notification email

1
config/locales/models/procedure/fr.yml
View file

@ -37,7 +37,6 @@ fr:
declarative_with_state/en_instruction: En instruction
declarative_with_state/accepte: Accepté
api_particulier_token: Jeton API Particulier
routing_criteria_name: Nomination du routage
initiated_mail: L’email de notification de passage de dossier en instruction
received_mail: L’email de notification de dépôt de dossier
closed_mail: L’email de notification d’acceptation de dossier

1
config/routes.rb
View file

@ -542,7 +542,6 @@ Rails.application.routes.draw do
get 'simple_routing'
post 'create_simple_routing'
delete 'destroy_all_groups_but_defaut'
patch 'update_routing_criteria_name'
patch 'update_instructeurs_self_management_enabled'
post 'import'
get 'export_groupe_instructeurs'

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

@ -692,18 +692,6 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
end
end
describe '#update_routing_criteria_name' do
before do
patch :update_routing_criteria_name,
params: {
procedure_id: procedure.id,
procedure: { routing_criteria_name: 'new name !' }
}
end
it { expect(procedure.reload.routing_criteria_name).to eq('new name !') }
end
describe '#create_simple_routing' do
let!(:procedure3) do
create(:procedure,

2
spec/models/routing_engine_spec.rb
View file

@ -1,8 +1,6 @@
describe RoutingEngine, type: :model do
include Logic
before { Flipper.enable(:routing_rules, procedure) }
describe '.compute' do
let(:procedure) do
create(:procedure,

1
spec/system/routing/rules_full_scenario_spec.rb
View file

@ -20,7 +20,6 @@ describe 'The routing with rules', js: true do
let(:artistique_user) { create(:user, password: password) }
before do
Flipper.enable(:routing_rules, procedure)
procedure.defaut_groupe_instructeur.instructeurs << administrateur.instructeur
end