From 9b442b69d363d47162a4d499c5184a3c394b75dc Mon Sep 17 00:00:00 2001
From: erdgeist <erdgeist@erdgeist.org>
Date: Sat, 1 Aug 2026 19:26:25 +0200
Subject: Report validation failures without the exception preamble

---
 app/controllers/nodes_controller.rb | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'app/controllers/nodes_controller.rb')

diff --git a/app/controllers/nodes_controller.rb b/app/controllers/nodes_controller.rb
index 383fb72c..ca30976f 100644
--- a/app/controllers/nodes_controller.rb
+++ b/app/controllers/nodes_controller.rb
@@ -163,7 +163,7 @@ class NodesController < ApplicationController
     flash[:error] = t("flash.common.locked_by_other")
     redirect_to node_path(@node)
   rescue ActiveRecord::RecordInvalid => e
-    flash[:error] = e.message
+    flash[:error] = e.record.errors.full_messages.to_sentence
     redirect_to node_path(@node)
   end
 
@@ -176,7 +176,7 @@ class NodesController < ApplicationController
     flash[:error] = t("flash.nodes.restore_target_missing")
     redirect_to node_path(@node)
   rescue ActiveRecord::RecordInvalid => e
-    flash[:error] = e.message
+    flash[:error] = e.record.errors.full_messages.to_sentence
     redirect_to node_path(@node)
   end
 
@@ -186,7 +186,8 @@ class NodesController < ApplicationController
     redirect_to trashed_nodes_path
 
   rescue ActiveRecord::RecordInvalid, ActiveRecord::RecordNotDestroyed => e
-    flash[:error] = e.message
+    messages = e.record.errors.full_messages
+    flash[:error] = messages.any? ? messages.to_sentence : e.message
     redirect_to node_path(@node)
   end
 
@@ -195,7 +196,7 @@ class NodesController < ApplicationController
     flash[:notice] = t("flash.nodes.published")
     redirect_to node_path(@node)
   rescue ActiveRecord::RecordInvalid => e
-    flash[:error] = e.message
+    flash[:error] = e.record.errors.full_messages.to_sentence
     redirect_to node_path(@node)
   end
 
-- 
cgit v1.3