Merge pull request #6081 from betagouv/main

2021-04-09-01

app/controllers/instructeurs/dossiers_controller.rb

@@ -60,7 +60,9 @@ module Instructeurs
     def avis
       @avis_seen_at = current_instructeur.follows.find_by(dossier: dossier)&.avis_seen_at
       @avis = Avis.new
-      @experts_emails = dossier.procedure.experts_procedures.where(revoked_at: nil).map(&:expert).map(&:email).sort
+      if @dossier.procedure.feature_enabled?(:admin_affect_experts_to_avis)
+        @experts_emails = dossier.procedure.experts_procedures.where(revoked_at: nil).map(&:expert).map(&:email).sort
+      end
     end
 
     def personnes_impliquees

app/models/assign_to.rb

@@ -11,7 +11,6 @@
 #  updated_at                                  :datetime
 #  groupe_instructeur_id                       :bigint
 #  instructeur_id                              :integer
-#  procedure_id                                :integer
 #
 class AssignTo < ApplicationRecord
   self.ignored_columns = [:procedure_id]

app/models/champs/address_champ.rb

@@ -39,7 +39,7 @@ class Champs::AddressChamp < Champs::TextChamp
   end
 
   def to_s
-    address_label
+    address_label.presence || ''
   end
 
   def for_tag

db/migrate/20210409075105_remove_procedure_id_from_assign_tos.rb

@@ -0,0 +1,5 @@
+class RemoveProcedureIdFromAssignTos < ActiveRecord::Migration[6.1]
+  def change
+    remove_column :assign_tos, :procedure_id, :number
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2021_04_07_174523) do
+ActiveRecord::Schema.define(version: 2021_04_09_075105) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -83,7 +83,6 @@ ActiveRecord::Schema.define(version: 2021_04_07_174523) do
 
   create_table "assign_tos", id: :serial, force: :cascade do |t|
     t.integer "instructeur_id"
-    t.integer "procedure_id"
     t.datetime "created_at"
     t.datetime "updated_at"
     t.bigint "groupe_instructeur_id"
@@ -93,9 +92,7 @@ ActiveRecord::Schema.define(version: 2021_04_07_174523) do
     t.boolean "instant_email_dossier_notifications_enabled", default: false, null: false
     t.index ["groupe_instructeur_id", "instructeur_id"], name: "unique_couple_groupe_instructeur_instructeur", unique: true
     t.index ["groupe_instructeur_id"], name: "index_assign_tos_on_groupe_instructeur_id"
-    t.index ["instructeur_id", "procedure_id"], name: "index_assign_tos_on_instructeur_id_and_procedure_id", unique: true
     t.index ["instructeur_id"], name: "index_assign_tos_on_instructeur_id"
-    t.index ["procedure_id"], name: "index_assign_tos_on_procedure_id"
   end
 
   create_table "attestation_templates", id: :serial, force: :cascade do |t|