Changeset 3010 for trunk/system/controllers/captcha.php

Timestamp:

07/08/2008 04:09:39 PM (5 months ago)

Author:

Geert

Message:

Cleaning up Captcha_Controller. Using in built in text::random() function. No more special characters, like $ or %, in captcha's anymore. Just a-z0-9.

Files:

• trunk/system/controllers/captcha.php (modified) (5 diffs)

trunk/system/controllers/captcha.php

@@ -16 +16 @@
  */
 class Captcha_Controller extends Controller {
+
+        public $session;
         public $captcha;
-        public $session;
 
         protected $captcha_code;
@@ -24 +25 @@
         {
                 $this->session = Session::instance();
-
                 $this->captcha = new Captcha;
 
-                // Create a random text string for captcha code.
+                // Create and store a random captcha string
                 $this->captcha_code = $this->create_code();
-                $this->captcha->set_code($this->captcha_code) ;
+                $this->captcha->set_code($this->captcha_code);
+                $this->session->set('captcha_code', $this->captcha_code);
 
-                // Set the session to store the security code
-                $this->session->set('captcha_code', $this->captcha_code);
-                // Call the library to output the image
+                // Output the image
                 $this->captcha->render();
         }
@@ -39 +38 @@
         private function create_code()
         {
-                $num_chars = Config::item('captcha.num_chars');
-
-                if (Config::item('captcha.style') == 'math')
+                if (Config::item('captcha.style') === 'math')
                 {
                         $code = (string) mt_rand(101, 991);
@@ -47 +44 @@
                 else
                 {
-                        // Character set to use, similar characters removed.
-                        $charset = '@2345#6BCDF$GH789KMNPQRT%VWXYZ';
-                        $code = '';
-                        for ($i = 0; $i < $num_chars; $i++)
-                        {
-                        $code .= substr($charset, mt_rand(0, strlen($charset)-1), 1);
-                        }
+                        $code = text::random('distinct', Config::item('captcha.num_chars'));
                 }
 
@@ -59 +50 @@
         }
 
-}
+} // End Captcha_Controller