User needs to choose a reason

1 files changed, 12 insertions, 4 deletions

diff --git a/app/controllers/reports_controller.rb b/app/controllers/reports_controller.rb
index 924eb0108..a1dd53125 100644
--- a/app/controllers/reports_controller.rb
+++ b/app/controllers/reports_controller.rb
@@ -1,15 +1,23 @@
 class ReportsController < ApplicationController
     def create
-        info_request = InfoRequest.find_by_url_title!(params[:request_id])
+        @info_request = InfoRequest.find_by_url_title!(params[:request_id])
+        @reason = params[:reason]
+        @message = params[:message]
+        if @reason.empty?
+            flash[:error] = _("Please choose a reason")
+            render "new"
+            return
+        end
+
         if !authenticated_user
             flash[:notice] = _("You need to be logged in to report a request for administrator attention")
-        elsif info_request.attention_requested
+        elsif @info_request.attention_requested
             flash[:notice] = _("This request has already been reported for administrator attention")
         else
-            info_request.report!(params[:reason], params[:message], @user)
+            @info_request.report!(@reason, @message, @user)
             flash[:notice] = _("This request has been reported for administrator attention")
         end
-        redirect_to request_url(info_request)
+        redirect_to request_url(@info_request)
     end
 
     def new