Changeset 3047 for trunk/application

Timestamp:

07/11/2008 07:14:01 AM (5 months ago)

Author:

Geert

Message:

Captcha updates:

• Bug fix: made sure the captcha_response session cannot be overwritten too soon. The problem was that Captcha::valid() would break if you created a new Captcha instance before it. Another good example of the power of Kohana events.
  
• Added a static instance() method so Captcha::valid() can access non-static methods, e.g. the counters.
  
• Shortened method names a bit: from "valid_response" to "valid", same for counters.
  
• Added a reset_count() method.
  
• Updated the example.
  

Files:

• trunk/application/controllers/examples.php (modified) (3 diffs)

trunk/application/controllers/examples.php

@@ -178 +178 @@
                 new Profiler;
 
-                // Ban bots after 10 invalid responses
+                // Load Captcha library
+                $captcha = new Captcha;
+
+                // Ban bots (that accept session cookies) after 50 invalid responses
                 // Be careful not to ban real people though! Set the threshold high enough.
-                if (Captcha::invalid_response_count() > 9)
+                if ($captcha->invalid_count() > 49)
                         exit('Bye! Stupid bot.');
 
@@ -187 +190 @@
                 {
                         // User has not given three valid responses yet
-                        if (Captcha::valid_response_count() < 3)
+                        if ($captcha->valid_count() < 3)
                         {
-                                // Valid response has been submitted
-                                if (Captcha::valid_response($this->input->post('captcha_response')))
+                                // Captcha::valid() is a static method that can be used as a Validation rule also
+                                if (Captcha::valid($this->input->post('captcha_response')))
                                 {
                                         echo '<p style="color:green">Good answer!</p>';
@@ -208 +211 @@
 
                 // Don't show Captcha anymore after three or more valid responses
-                if (Captcha::valid_response_count() < 3)
+                if ($captcha->valid_count() < 3)
                 {
                         echo '<p>';
-                        echo Captcha::factory()->render(); // Shows the Captcha challenge (image/riddle/etc)
+                        echo $captcha->render(); // Shows the Captcha challenge (image/riddle/etc)
                         echo '</p>';
                         echo form::input('captcha_response');