refactor: show different flash message when SeversController#destroy …

…fail (nanego#288)
pantographe · Dec 5, 2024 · cf14bf3 · cf14bf3
1 parent fbbf3f0
commit cf14bf3
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/app/controllers/servers_controller.rb b/app/controllers/servers_controller.rb
@@ -97,10 +97,14 @@ def import
   end
 
   def destroy
-    @server.destroy
     respond_to do |format|
-      format.html { redirect_to servers_path(search_params), notice: 'Le matériel a bien été supprimé.' }
-      format.json { head :no_content }
+      if @server.destroy
+        format.html { redirect_to servers_path(search_params), notice: t(".flashes.destroyed") }
+        format.json { head :no_content }
+      else
+        format.html { redirect_to servers_path(search_params), alert: t(".flashes.not_destroyed") }
+        format.json { head :bad_request }
+      end
     end
   end
 

diff --git a/config/locales/fr.yml b/config/locales/fr.yml
@@ -425,9 +425,13 @@ fr:
       delete_confirmation: Toutes les données concernant ce matériel seront supprimées, ainsi que toutes les connexions associées. Souhaitez-vous confirmer ?
     duplicate:
       title: Dupliquer le matériel %{server_name}
+    destroy:
+      flashes:
+        destroyed: Le matériel a bien été supprimé.
+        not_destroyed: Une erreur est survenue lors de la suppression du matériel.
     flashes:
-        connections_destroyed: Les connexions ont été supprimées.
-        connections_not_destroyed: Une erreur a empêché la suppression des connexions.
+      connections_destroyed: Les connexions ont été supprimées.
+      connections_not_destroyed: Une erreur a empêché la suppression des connexions.
   glpi:
     computer:
       name: Nom