| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 0.4.6 |
| Request #11653 | Text_Password::create() call | ||
|---|---|---|---|
| Submitted: | 2007-07-20 12:56 UTC | Modified: | 2009-07-27 21:10 UTC |
| From: | jausions | Assigned: | wenz |
| Status: | Closed | Package: | Text_CAPTCHA (version 0.2.1) |
| PHP Version: | Irrelevant | OS: | Irrelevant |
| Roadmaps: | (Not assigned) | ||
[2007-07-20 12:56 UTC] jausions
(Philippe Jausions)
Description:
------------
For a better CAPTCHA, I think calling Text_Password::create($len, 'unpronounceable', 'alphanumeric'); would be a better choice (might want to pass list of non-ambiguous numbers/letters, i.e. no-zeroes, O's or lowercase L's.)
And better yet, why not accept a 'phraseOptions' => array() in the options passed to the Text_CAPTCHA_driver::init() function.
-Philippe
Comments
[2007-07-20 13:01 UTC] jausions
(Philippe Jausions)
Nope, my bad... I thought I listed it under Text_Password package... hahah
[2007-08-01 05:07 UTC] wenz
(Christian Wenz)
I agree that passing the options is a good idea. I disagree however with defaulting to unpronounceable. Several websites tried this in the last few months, and most of them went back to the old system since user complained. Enabling developers to set the captcha options is a good idea, though.
[2009-07-27 21:10 UTC] wenz
(Christian Wenz)
-Status: Analyzed
+Status: Closed
-Assigned To:
+Assigned To: wenz
Thank you for your bug report. This issue has been fixed
in the latest released version of the package, which you can download at
http://pear.php.net/get/
implemented in release 0.4.0