Ambethia ReCAPTCHA

Comments and changes to this ticket

• 

  Jason L Perry November 20th, 2008 @ 03:24 AM

  Hmm, yes, great idea. I've got a ton of patches to sift through, I'm gonna try to set aside some time this week to take care of them. I'll definitely work on this one too.

• You flagged this item as spam.
  

  Roman Shterenzon January 18th, 2009 @ 01:27 PM

  • Tag set to “i18n, patch”

  Please find attached a patch which adds I18n support.

  By the way, the latest changes introduced to the recaptcha rendered the rdoc incorrect. It's impossible to use verify_recaptcha(@model) now, you have rather use verify_recaptcha(:model => @model). I wasted some time in order to understand, why I'm not getting any errors in my model. The simple fix can be adding

  
  @@ -51,6 +51,7 @@ module Ambethia
         # variable +RECAPTCHA_PUBLIC_KEY+.
         def verify_recaptcha(options = {})
           return true if SKIP_VERIFY_ENV.include? ENV['RAILS_ENV']
  +        model = options if options.is_a? ActiveRecord::Base
           private_key = options[:private_key] ||= ENV['RECAPTCHA_PRIVATE_KEY']
           raise ReCaptchaError, "No private key specified." unless private_key
           begin
  

• You flagged this item as spam.
  

  Roman Shterenzon January 18th, 2009 @ 01:29 PM

  Hmm.. no, sorry, the line "if model = options[:model]" should be changed as well.

• You flagged this item as spam.
  

  Roman Shterenzon January 18th, 2009 @ 01:37 PM

  I came up with the following, seems to work for both the old and the new way of specifying the model.

  • recaptcha_new.diff 1.4 KB