remove unused instructeur logic

app/controllers/instructeurs/avis_controller.rb

@@ -2,131 +2,14 @@ module Instructeurs
   class AvisController < InstructeurController
     include CreateAvisConcern
 
-    before_action :authenticate_instructeur!, except: [:sign_up, :create_instructeur]
+    before_action :authenticate_instructeur!
-    before_action :check_if_avis_revoked, only: [:show]
-    before_action :redirect_if_no_sign_up_needed, only: [:sign_up]
-    before_action :check_avis_exists_and_email_belongs_to_avis, only: [:sign_up, :create_instructeur]
-    before_action :set_avis_and_dossier, only: [:show, :instruction, :messagerie, :create_commentaire, :update]
-
     A_DONNER_STATUS = 'a-donner'
     DONNES_STATUS   = 'donnes'
 
-    def index
-      avis = current_instructeur.avis.includes(:procedure)
-      @avis_by_procedure = avis.to_a.group_by(&:procedure)
-    end
-
-    def procedure
-      @procedure = Procedure.find(params[:procedure_id])
-      instructeur_avis = current_instructeur.avis.includes(:dossier).where(dossiers: { revision: @procedure.revisions })
-      @avis_a_donner = instructeur_avis.without_answer
-      @avis_donnes = instructeur_avis.with_answer
-
-      @statut = params[:statut].presence || A_DONNER_STATUS
-
-      @avis = case @statut
-      when A_DONNER_STATUS
-        @avis_a_donner
-      when DONNES_STATUS
-        @avis_donnes
-      end
-
-      @avis = @avis.page([params[:page].to_i, 1].max)
-    end
-
-    def show
-    end
-
-    def instruction
-      @new_avis = Avis.new
-    end
-
-    def update
-      if @avis.update(avis_params)
-        flash.notice = 'Votre réponse est enregistrée.'
-        @avis.dossier.update!(last_avis_updated_at: Time.zone.now)
-        redirect_to instruction_instructeur_avis_path(@avis.procedure, @avis)
-      else
-        flash.now.alert = @avis.errors.full_messages
-        @new_avis = Avis.new
-        render :instruction
-      end
-    end
-
-    def messagerie
-      @commentaire = Commentaire.new
-    end
-
-    def create_commentaire
-      @commentaire = CommentaireService.build(current_instructeur, avis.dossier, commentaire_params)
-
-      if @commentaire.save
-        @commentaire.dossier.update!(last_commentaire_updated_at: Time.zone.now)
-        flash.notice = "Message envoyé"
-        redirect_to messagerie_instructeur_avis_path(avis.procedure, avis)
-      else
-        flash.alert = @commentaire.errors.full_messages
-        render :messagerie
-      end
-    end
-
-    def create_avis
-      @procedure = Procedure.find(params[:procedure_id])
-      if !@procedure.feature_enabled?(:expert_not_allowed_to_invite)
-        @new_avis = create_avis_from_params(avis.dossier, avis.confidentiel)
-
-        if @new_avis.nil?
-          redirect_to instruction_instructeur_avis_path(avis.procedure, avis)
-        else
-          set_avis_and_dossier
-          render :instruction
-        end
-      else
-        flash.alert = "Cette démarche ne vous permet pas de demander un avis externe"
-        redirect_to instruction_instructeur_avis_path(avis.procedure, avis)
-      end
-    end
-
-    def bilans_bdf
-      if avis.dossier.etablissement&.entreprise_bilans_bdf.present?
-        extension = params[:format]
-        render extension.to_sym => avis.dossier.etablissement.entreprise_bilans_bdf_to_sheet(extension)
-      else
-        redirect_to instructeur_avis_path(avis)
-      end
-    end
-
-    def sign_up
-      @email = params[:email]
-      @dossier = Avis.includes(:dossier).find(params[:id]).dossier
-
-      render
-    end
-
-    def create_instructeur
-      procedure_id = params[:procedure_id]
-      avis_id = params[:id]
-      email = params[:email]
-      password = params[:user][:password]
-
-      # Not perfect because the password will not be changed if the user already exists
-      user = User.create_or_promote_to_instructeur(email, password)
-
-      if user.valid?
-        sign_in(user)
-
-        Avis.link_avis_to_instructeur(user.instructeur)
-        redirect_to url_for(instructeur_all_avis_path)
-      else
-        flash[:alert] = user.errors.full_messages
-        redirect_to url_for(sign_up_instructeur_avis_path(procedure_id, avis_id, email))
-      end
-    end
-
     def revoquer
       avis = Avis.find(params[:id])
       if avis.revoke_by!(current_instructeur)
-        flash.notice = "#{avis.email_to_display} ne peut plus donner son avis sur ce dossier."
+        flash.notice = "#{avis.expert.email} ne peut plus donner son avis sur ce dossier."
         redirect_back(fallback_location: avis_instructeur_dossier_path(avis.procedure, avis.dossier))
       end
     end
@@ -136,60 +19,13 @@ module Instructeurs
       if avis.revivable_by?(current_instructeur)
         if avis.answer.blank?
           AvisMailer.avis_invitation(avis).deliver_later
-          flash.notice = "Un mail de relance a été envoyé à #{avis.email_to_display}"
+          flash.notice = "Un mail de relance a été envoyé à #{avis.expert.email}"
           redirect_back(fallback_location: avis_instructeur_dossier_path(avis.procedure, avis.dossier))
         else
-          flash.alert = "#{avis.email} a déjà donné son avis"
+          flash.alert = "#{avis.expert.email} a déjà donné son avis"
           redirect_back(fallback_location: avis_instructeur_dossier_path(avis.procedure, avis.dossier))
         end
       end
     end
-
-    private
-
-    def set_avis_and_dossier
-      @avis = avis
-      @dossier = avis.dossier
-    end
-
-    def redirect_if_no_sign_up_needed
-      avis = Avis.find(params[:id])
-
-      if current_instructeur.present?
-        # a instructeur is authenticated ... lets see if it can view the dossier
-
-        redirect_to instructeur_avis_url(avis.procedure, avis)
-      elsif avis.instructeur&.email == params[:email]
-        # the avis instructeur has already signed up and it sould sign in
-
-        redirect_to new_user_session_url
-      end
-    end
-
-    def check_if_avis_revoked
-      avis = Avis.find(params[:id])
-      if avis.revoked?
-        flash.alert = "Vous n'avez plus accès à ce dossier."
-        redirect_to url_for(root_path)
-      end
-    end
-
-    def check_avis_exists_and_email_belongs_to_avis
-      if !Avis.avis_exists_and_email_belongs_to_avis?(params[:id], params[:email])
-        redirect_to url_for(root_path)
-      end
-    end
-
-    def avis
-      current_instructeur.avis.includes(dossier: [:avis, :commentaires]).find(params[:id])
-    end
-
-    def avis_params
-      params.require(:avis).permit(:answer, :piece_justificative_file)
-    end
-
-    def commentaire_params
-      params.require(:commentaire).permit(:body, :piece_jointe)
-    end
   end
 end

app/controllers/instructeurs/dossiers_controller.rb

@@ -66,7 +66,7 @@ module Instructeurs
       @following_instructeurs_emails = dossier.followers_instructeurs.map(&:email)
       previous_followers = dossier.previous_followers_instructeurs - dossier.followers_instructeurs
       @previous_following_instructeurs_emails = previous_followers.map(&:email)
-      @avis_emails = dossier.avis.includes(:instructeur).map(&:email_to_display)
+      @avis_emails = dossier.experts.map(&:email)
       @invites_emails = dossier.invites.map(&:email)
       @potential_recipients = dossier.groupe_instructeur.instructeurs.reject { |g| g == current_instructeur }
     end
@@ -181,7 +181,7 @@ module Instructeurs
     end
 
     def create_avis
-      @avis = create_avis_from_params(dossier)
+      @avis = create_avis_from_params(dossier, current_instructeur)
 
       if @avis.nil?
         redirect_to avis_instructeur_dossier_path(procedure, dossier)

app/models/avis.rb

@@ -44,9 +44,8 @@ class Avis < ApplicationRecord
   validates :introduction_file, size: { less_than: 20.megabytes }
 
   before_validation -> { sanitize_email(:email) }
-  before_create :try_to_assign_instructeur
 
-default_scope { joins(:dossier) }
+  default_scope { joins(:dossier) }
   scope :with_answer, -> { where.not(answer: nil) }
   scope :without_answer, -> { where(answer: nil) }
   scope :for_dossier, -> (dossier_id) { where(dossier_id: dossier_id) }
@@ -124,14 +123,4 @@ default_scope { joins(:dossier) }
       destroy!
     end
   end
-  
-  private
-
-  def try_to_assign_instructeur
-    instructeur = Instructeur.by_email(email)
-    if instructeur
-      self.instructeur = instructeur
-      self.email = nil
-    end
-  end
 end