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

Move defaut_groupe_instructeur as has_one and eager_load

Browse source
This commit is contained in:
simon lehericey 2019-09-16 15:37:12 +02:00
parent afa0237349
commit acb99609ba
2 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/controllers/instructeurs/procedures_controller.rb
View file

@ -8,7 +8,7 @@ module Instructeurs
def index def index
@procedures = current_instructeur @procedures = current_instructeur
.visible_procedures .visible_procedures
.includes(:logo_attachment, :logo_active_storage_attachment) .includes(:logo_attachment, :logo_active_storage_attachment, :defaut_groupe_instructeur)
.order(archived_at: :desc, published_at: :desc, created_at: :desc) .order(archived_at: :desc, published_at: :desc, created_at: :desc)
groupe_instructeurs = current_instructeur.groupe_instructeurs.where(procedure: @procedures) groupe_instructeurs = current_instructeur.groupe_instructeurs.where(procedure: @procedures)

6
app/models/procedure.rb
View file

@ -27,6 +27,8 @@ class Procedure < ApplicationRecord
has_one :refused_mail, class_name: "Mails::RefusedMail", dependent: :destroy has_one :refused_mail, class_name: "Mails::RefusedMail", dependent: :destroy
has_one :without_continuation_mail, class_name: "Mails::WithoutContinuationMail", dependent: :destroy has_one :without_continuation_mail, class_name: "Mails::WithoutContinuationMail", dependent: :destroy
has_one :defaut_groupe_instructeur, -> { where(label: GroupeInstructeur::DEFAULT_LABEL) }, class_name: 'GroupeInstructeur', inverse_of: :procedure
has_one_attached :logo has_one_attached :logo
has_one_attached :logo_active_storage has_one_attached :logo_active_storage
has_one_attached :notice has_one_attached :notice
@ -477,10 +479,6 @@ class Procedure < ApplicationRecord
end end
end end
def defaut_groupe_instructeur
groupe_instructeurs.find_by(label: GroupeInstructeur::DEFAULT_LABEL)
end
def missing_instructeurs? def missing_instructeurs?
!AssignTo.exists?(groupe_instructeur: groupe_instructeurs) !AssignTo.exists?(groupe_instructeur: groupe_instructeurs)
end end