Merge pull request #8413 from tchak/fix-groupe-instructeur-count

fix(groupe_instructeur): only show to administrators the number of visible files

app/models/dossier.rb

@@ -652,11 +652,13 @@ class Dossier < ApplicationRecord
 
   def assign_to_groupe_instructeur(groupe_instructeur, author = nil)
     if (groupe_instructeur.nil? || groupe_instructeur.procedure == procedure) && self.groupe_instructeur != groupe_instructeur
-      if update(groupe_instructeur: groupe_instructeur, groupe_instructeur_updated_at: Time.zone.now)
+      if update(groupe_instructeur:, groupe_instructeur_updated_at: Time.zone.now)
-        unfollow_stale_instructeurs
+        if !brouillon?
+          unfollow_stale_instructeurs
 
-        if author.present?
+          if author.present?
-          log_dossier_operation(author, :changer_groupe_instructeur, self)
+            log_dossier_operation(author, :changer_groupe_instructeur, self)
+          end
         end
 
         true

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

@@ -59,9 +59,9 @@
                   = t('.group_management.delete')
             - else
               %td.actions
-                = link_to reaffecter_dossiers_admin_procedure_groupe_instructeur_path(procedure, group), class: 'button', title: t('.group_management.move_folders_confirmation') do
+                = link_to reaffecter_dossiers_admin_procedure_groupe_instructeur_path(procedure, group), class: 'button', title: t('.group_management.move_files_confirmation') do
                   %span.icon.follow
-                  = t('.group_management.move_folders', count: group.dossiers.count)
+                  = t('.group_management.move_files', count: group.dossiers.visible_by_administration.size)
 
 
   = paginate groupes_instructeurs

config/locales/views/administrateurs/groupe_instructeurs/en.yml

@@ -38,10 +38,11 @@ en:
           title: Group management
           delete: delete the group
           delete_confirmation: Are you sure you want to delete the group "%{group_name}"
-          move_folders:
+          move_files:
-            one: move 1 file
+            zero: move draft files
+            one: move one file
             other: move the %{count} files
-          move_folders_confirmation: Reassign folders to another group so you can delete it
+          move_files_confirmation: Reassign folders to another group so you can delete it
         add_a_group:
           title: Add a group
           notice: This group will be a choice from the list "%{routing_criteria_name}"

config/locales/views/administrateurs/groupe_instructeurs/fr.yml

@@ -44,10 +44,11 @@ fr:
           title: Gestion des Groupes
           delete: supprimer le groupe
           delete_confirmation: Êtes-vous sûr de vouloir supprimer le groupe "%{group_name}"
-          move_folders:
+          move_files:
-            one: déplacer 1 dossier
+            zero: déplacer les dossiers en brouillon
+            one: déplacer un dossier
             other: déplacer les %{count} dossiers
-          move_folders_confirmation: Réaffecter les dossiers à un autre groupe afin de pouvoir le supprimer
+          move_files_confirmation: Réaffecter les dossiers à un autre groupe afin de pouvoir le supprimer
         add_a_group:
           title: Ajouter un nom de groupe
           notice: Ce groupe sera un choix de la liste "%{routing_criteria_name}"