From 1a085e3b973726893dc220c060225dc42d9a4edb Mon Sep 17 00:00:00 2001 From: Robin Houston Date: Mon, 28 May 2012 10:27:11 +0100 Subject: Improve explanation I struggled to understand this comment. I hope the new wording is clearer. --- app/controllers/request_controller.rb | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'app/controllers/request_controller.rb') diff --git a/app/controllers/request_controller.rb b/app/controllers/request_controller.rb index 07faf4445..2250747e1 100644 --- a/app/controllers/request_controller.rb +++ b/app/controllers/request_controller.rb @@ -304,9 +304,11 @@ class RequestController < ApplicationController # See if values were valid or not if !@existing_request.nil? || !@info_request.valid? - # We don't want the error "Outgoing messages is invalid", as the outgoing message - # will be valid for a specific reason which we are displaying anyway. + # We don't want the error "Outgoing messages is invalid", as in this + # case the list of errors will also contain a more specific error + # describing the reason it is invalid. @info_request.errors.delete("outgoing_messages") + render :action => 'new' return end -- cgit v1.2.3