From 675bbdad1547a73d5d06cc2c2a00dd9206d0363d Mon Sep 17 00:00:00 2001 From: Christophe Robillard Date: Fri, 17 Jan 2020 16:24:33 +0100 Subject: [PATCH] transfere les services lors de suppression admin MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Lorsqu'un administrateur est supprimé, ses services sont transférés à un autre administrateur pour chacune de ses procédures --- .../manager/administrateurs_controller.rb | 6 +----- app/models/administrateur.rb | 13 +++++++++++++ spec/models/administrateur_spec.rb | 15 +++++++++++++++ 3 files changed, 29 insertions(+), 5 deletions(-) diff --git a/app/controllers/manager/administrateurs_controller.rb b/app/controllers/manager/administrateurs_controller.rb index f976f3cf6..10dec19d5 100644 --- a/app/controllers/manager/administrateurs_controller.rb +++ b/app/controllers/manager/administrateurs_controller.rb @@ -22,11 +22,7 @@ module Manager def delete administrateur = Administrateur.find(params[:id]) - if !administrateur.can_be_deleted? - fail "Impossible de supprimer cet administrateur car il a des dossiers ou des procédures" - end - administrateur.dossiers.each(&:delete_and_keep_track) - administrateur.destroy + administrateur.delete_and_transfer_services logger.info("L'administrateur #{administrateur.id} est supprimé par #{current_administration.id}") flash[:notice] = "L'administrateur #{administrateur.id} est supprimé" diff --git a/app/models/administrateur.rb b/app/models/administrateur.rb index 4d4b84d4a..1174d4b60 100644 --- a/app/models/administrateur.rb +++ b/app/models/administrateur.rb @@ -70,4 +70,17 @@ class Administrateur < ApplicationRecord def can_be_deleted? dossiers.state_instruction_commencee.none? && procedures.all? { |p| p.administrateurs.count > 1 } end + + def delete_and_transfer_services + if !can_be_deleted? + fail "Impossible de supprimer cet administrateur car il a des dossiers ou des procédures" + end + dossiers.each(&:delete_and_keep_track) + + procedures.each do |procedure| + next_administrateur = procedure.administrateurs.where.not(id: self.id).first + procedure.service.update(administrateur: next_administrateur) + end + destroy + end end diff --git a/spec/models/administrateur_spec.rb b/spec/models/administrateur_spec.rb index 9f0c30923..3e6864466 100644 --- a/spec/models/administrateur_spec.rb +++ b/spec/models/administrateur_spec.rb @@ -46,6 +46,21 @@ describe Administrateur, type: :model do end end + describe '#delete_and_transfer_services' do + let!(:administrateur) { create(:administrateur) } + let!(:autre_administrateur) { create(:administrateur) } + let!(:procedure) { create(:procedure, :with_service, administrateurs: [administrateur, autre_administrateur]) } + let(:service) { procedure.service } + + it "delete and transfer services to other admin" do + service.update(administrateur: administrateur) + administrateur.delete_and_transfer_services + + expect(Administrateur.find_by(id: administrateur.id)).to be_nil + expect(service.reload.administrateur).to eq(autre_administrateur) + end + end + # describe '#password_complexity' do # let(:email) { 'mail@beta.gouv.fr' } # let(:passwords) { ['pass', '12pass23', 'démarches ', 'démarches-simple', 'démarches-simplifiées-pwd'] }