[myexperiment-hackers] [2831] trunk/app/controllers/feedback_controller.rb: Minor fix to prevent stack error when CAPTCHA form data is mysteriously

[noreply]

Revision

2831

Author

dtm

Date

2011-11-21 05:07:32 -0500 (Mon, 21 Nov 2011)

Log Message

Minor fix to prevent stack error when CAPTCHA form data is mysteriously
missing.

Modified Paths

• trunk/app/controllers/feedback_controller.rb

Diff

Modified: trunk/app/controllers/feedback_controller.rb (2830 => 2831)

--- trunk/app/controllers/feedback_controller.rb	2011-11-20 16:31:43 UTC (rev 2830)
+++ trunk/app/controllers/feedback_controller.rb	2011-11-21 10:07:32 UTC (rev 2831)
@@ -21,7 +21,7 @@
         format.html { redirect_to "/feedback" }
       end
     else
-      if captcha_valid?(params[:feedback][:captcha_id], params[:feedback][:captcha_validation])
+      if params[:feedback] && captcha_valid?(params[:feedback][:captcha_id], params[:feedback][:captcha_validation])
     
         from_user = ( params[:from].blank? ? 'no from': params[:from] ) + ' (' + (!params[:email].blank? ? params[:email] : 'no email') + ')';
         Mailer.deliver_feedback(from_user, params[:subject], params[:content])