From 9fe93ded7f4c0b1309337f91bd9cc068a8b8529e Mon Sep 17 00:00:00 2001 From: Paul Chavard Date: Thu, 1 Dec 2022 09:13:24 +0100 Subject: [PATCH] refactor(groupe_instructeur): add instructeur.find_or_invite --- app/models/instructeur.rb | 40 ++++++++++++++++++++++++++++++++++++++- 1 file changed, 39 insertions(+), 1 deletion(-) diff --git a/app/models/instructeur.rb b/app/models/instructeur.rb index 170df6da7..362870a3a 100644 --- a/app/models/instructeur.rb +++ b/app/models/instructeur.rb @@ -49,7 +49,45 @@ class Instructeur < ApplicationRecord default_scope { eager_load(:user) } def self.by_email(email) - Instructeur.eager_load(:user).find_by(users: { email: email }) + find_by(users: { email: email }) + end + + def self.find_or_invite(emails: [], ids: [], groupe_instructeur:) + valid_emails, invalid_emails = emails + .map(&:strip) + .map(&:downcase) + .partition { URI::MailTo::EMAIL_REGEXP.match?(_1) } + valid_ids = ids.map { _1.is_a?(String) ? id_from_typed_id(_1) : _1 } + + instructeurs_by_ids = where(id: valid_ids) + instructeurs_by_emails = where(users: { email: valid_emails }) + + instructeurs = if valid_ids.present? && valid_emails.present? + instructeurs_by_ids.or(instructeurs_by_emails).distinct(:id) + elsif valid_emails.present? + instructeurs_by_emails + else + instructeurs_by_ids + end + + not_found_emails = valid_emails - instructeurs.map(&:email) + + # Send invitations to users without account + if not_found_emails.present? + administrateurs = groupe_instructeur.procedure.administrateurs + instructeurs += not_found_emails.map { invite(_1, administrateurs) } + end + + # We dont't want to assign a user to a groupe_instructeur if they are already assigned to it + instructeurs -= groupe_instructeur.instructeurs + + [instructeurs, invalid_emails] + end + + def self.invite(email, administrateurs) + user = User.create_or_promote_to_instructeur(email, SecureRandom.hex, administrateurs:) + user.invite! + user.instructeur end def email