Avoid multiple returns

This commit is contained in:
Tom Hughes 2018-08-29 08:47:02 +01:00
parent 05514ced0c
commit 720da8b78c

View file

@ -26,13 +26,14 @@ class MessagesController < ApplicationController
if current_user.sent_messages.where("sent_on >= ?", Time.now.getutc - 1.hour).count >= MAX_MESSAGES_PER_HOUR
flash[:error] = t ".limit_exceeded"
render :action => "new"
elsif @message.save
flash[:notice] = t ".message_sent"
Notifier.message_notification(@message).deliver_now
redirect_to :action => :inbox
return
else
render :action => "new"
end
render :action => "new"
end
# Allow the user to reply to another message.