Merge pull request #6819 from betagouv/feat/6818

ETQ Administrateur, lorsque je teste ma démarche, je souhaite que l'import CSV n'envoie pas de mails aux instructeurs

app/assets/stylesheets/utils.scss

@@ -54,6 +54,10 @@
   font-size: 14px;
 }
 
+.text-lg {
+  font-size: 18px;
+}
+
 .mt-1 {
   margin-top: $default-spacer;
 }

app/controllers/administrateurs/groupe_instructeurs_controller.rb

@@ -208,34 +208,33 @@ module Administrateurs
     end
 
     def import
-      if !CSV_ACCEPTED_CONTENT_TYPES.include?(group_csv_file.content_type) && !CSV_ACCEPTED_CONTENT_TYPES.include?(marcel_content_type)
-        flash[:alert] = "Importation impossible : veuillez importer un fichier CSV"
-        redirect_to admin_procedure_groupe_instructeurs_path(procedure)
+      if procedure.publiee?
+        if !CSV_ACCEPTED_CONTENT_TYPES.include?(group_csv_file.content_type) && !CSV_ACCEPTED_CONTENT_TYPES.include?(marcel_content_type)
+          flash[:alert] = "Importation impossible : veuillez importer un fichier CSV"
 
-      elsif group_csv_file.size > CSV_MAX_SIZE
-        flash[:alert] = "Importation impossible : le poids du fichier est supérieur à #{number_to_human_size(CSV_MAX_SIZE)}"
-        redirect_to admin_procedure_groupe_instructeurs_path(procedure)
+        elsif group_csv_file.size > CSV_MAX_SIZE
+          flash[:alert] = "Importation impossible : le poids du fichier est supérieur à #{number_to_human_size(CSV_MAX_SIZE)}"
 
-      else
-        file = group_csv_file.read
-        base_encoding = CharlockHolmes::EncodingDetector.detect(file)
-        groupes_emails = ACSV::CSV.new_for_ruby3(file.encode("UTF-8", base_encoding[:encoding], invalid: :replace, replace: ""), headers: true, header_converters: :downcase)
-          .map { |r| r.to_h.slice('groupe', 'email') }
-
-        groupes_emails_has_keys = groupes_emails.first.has_key?("groupe") && groupes_emails.first.has_key?("email")
-
-        if groupes_emails_has_keys.blank?
-          flash[:alert] = "Importation impossible, veuillez importer un csv #{view_context.link_to('suivant ce modèle', "/csv/#{I18n.locale}/import-groupe-test.csv")}"
         else
-          add_instructeurs_and_get_errors = InstructeursImportService.import(procedure, groupes_emails)
+          file = group_csv_file.read
+          base_encoding = CharlockHolmes::EncodingDetector.detect(file)
+          groupes_emails = ACSV::CSV.new_for_ruby3(file.encode("UTF-8", base_encoding[:encoding], invalid: :replace, replace: ""), headers: true, header_converters: :downcase)
+            .map { |r| r.to_h.slice('groupe', 'email') }
 
-          if add_instructeurs_and_get_errors.empty?
-            flash[:notice] = "La liste des instructeurs a été importée avec succès"
+          groupes_emails_has_keys = groupes_emails.first.has_key?("groupe") && groupes_emails.first.has_key?("email")
+
+          if groupes_emails_has_keys.blank?
+            flash[:alert] = "Importation impossible, veuillez importer un csv #{view_context.link_to('suivant ce modèle', "/csv/#{I18n.locale}/import-groupe-test.csv")}"
           else
-            flash[:alert] = "Import terminé. Cependant les emails suivants ne sont pas pris en compte: #{add_instructeurs_and_get_errors.join(', ')}"
+            add_instructeurs_and_get_errors = InstructeursImportService.import(procedure, groupes_emails)
+
+            if add_instructeurs_and_get_errors.empty?
+              flash[:notice] = "La liste des instructeurs a été importée avec succès"
+            else
+              flash[:alert] = "Import terminé. Cependant les emails suivants ne sont pas pris en compte: #{add_instructeurs_and_get_errors.join(', ')}"
+            end
           end
         end
-
         redirect_to admin_procedure_groupe_instructeurs_path(procedure)
       end
     end

app/views/administrateurs/groupe_instructeurs/_edit.html.haml

@@ -21,13 +21,17 @@
     = f.submit 'Ajouter le groupe', class: "button primary send"
 
   - csv_max_size = Administrateurs::GroupeInstructeursController::CSV_MAX_SIZE
-  = form_tag import_admin_procedure_groupe_instructeurs_path(procedure), method: :post, multipart: true, class: "mt-4 form" do
-    = label_tag "Importer par fichier CSV"
-    %p.notice Le fichier csv doit comporter 2 colonnes (Groupe, Email) et être séparé par des virgules. L'import n'écrase pas les groupes et les instructeurs existants.
-    %p.notice Le poids du fichier doit être inférieur à #{number_to_human_size(csv_max_size)}
-    %p.mt-2.mb-2= link_to "Télécharger l'exemple de fichier CSV", "/csv/#{I18n.locale}/import-groupe-test.csv"
-    = file_field_tag :group_csv_file, required: true, accept: 'text/csv', size: "1"
-    = submit_tag "Importer le fichier", class: 'button primary send', data: { disable_with: "Envoi..." }
+  - if procedure.publiee?
+    = form_tag import_admin_procedure_groupe_instructeurs_path(procedure), method: :post, multipart: true, class: "mt-4 form" do
+      = label_tag "Importer par fichier CSV"
+      %p.notice Le fichier csv doit comporter 2 colonnes (Groupe, Email) et être séparé par des virgules. L'import n'écrase pas les groupes et les instructeurs existants.
+      %p.notice Le poids du fichier doit être inférieur à #{number_to_human_size(csv_max_size)}
+      %p.mt-2.mb-2= link_to "Télécharger l'exemple de fichier CSV", "/csv/#{I18n.locale}/import-groupe-test.csv"
+      = file_field_tag :group_csv_file, required: true, accept: 'text/csv', size: "1"
+      = submit_tag "Importer le fichier", class: 'button primary send', data: { disable_with: "Envoi..." }
+  - else
+    %p.mt-4.form.bold.mb-2.text-lg Importer par fichier CSV
+    %p.notice L’import d’instructeurs par fichier CSV est disponible une fois la démarche publiée
 
   %table.table.mt-2
     %thead