diff --git a/app/controllers/administrateurs/experts_procedures_controller.rb b/app/controllers/administrateurs/experts_procedures_controller.rb index d86f00ca2..54895580b 100644 --- a/app/controllers/administrateurs/experts_procedures_controller.rb +++ b/app/controllers/administrateurs/experts_procedures_controller.rb @@ -9,7 +9,7 @@ module Administrateurs end def create - emails = params['emails'].presence || [].to_json + emails = params['emails'].presence || [] emails = check_if_typo(emails) errors = Array.wrap(generate_emails_suggestions_message(@maybe_typos)) diff --git a/app/controllers/administrateurs/groupe_instructeurs_controller.rb b/app/controllers/administrateurs/groupe_instructeurs_controller.rb index 16378ca5c..aa422b281 100644 --- a/app/controllers/administrateurs/groupe_instructeurs_controller.rb +++ b/app/controllers/administrateurs/groupe_instructeurs_controller.rb @@ -219,8 +219,7 @@ module Administrateurs end def add_instructeur - emails = params['emails'].presence || [].to_json - + emails = params['emails'].presence || [] emails = check_if_typo(emails) errors = Array.wrap(generate_emails_suggestions_message(@maybe_typos)) diff --git a/app/controllers/instructeurs/groupe_instructeurs_controller.rb b/app/controllers/instructeurs/groupe_instructeurs_controller.rb index d75254b5c..932564f83 100644 --- a/app/controllers/instructeurs/groupe_instructeurs_controller.rb +++ b/app/controllers/instructeurs/groupe_instructeurs_controller.rb @@ -20,7 +20,7 @@ module Instructeurs end def add_instructeur - email = instructeur_email.present? ? [instructeur_email].to_json : [].to_json + email = instructeur_email.present? ? [instructeur_email] : [] email = check_if_typo(email)&.first errors = Array.wrap(generate_emails_suggestions_message(@maybe_typos)) diff --git a/app/models/concerns/email_sanitizable_concern.rb b/app/models/concerns/email_sanitizable_concern.rb index b8818b1d2..80b598eec 100644 --- a/app/models/concerns/email_sanitizable_concern.rb +++ b/app/models/concerns/email_sanitizable_concern.rb @@ -18,7 +18,7 @@ module EmailSanitizableConcern end def check_if_typo(emails) - emails = JSON.parse(emails).map { EmailSanitizer.sanitize(_1) } + emails = emails.map { EmailSanitizer.sanitize(_1) } @maybe_typos, no_suggestions = emails .map { |email| [email, EmailChecker.check(email:)[:suggestions]&.first] } .partition { _1[1].present? } diff --git a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb index 86fb07db8..e916435de 100644 --- a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb +++ b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb @@ -359,7 +359,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do context 'when the admin wants to assign an instructor who is already assigned on this procedure' do let(:instructeur) { create(:instructeur) } before { procedure_non_routee.groupe_instructeurs.first.add_instructeurs(emails: [instructeur.user.email]) } - let(:emails) { [instructeur.email].to_json } + let(:emails) { [instructeur.email] } it { expect(subject).to render_template(:index) } end