Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.0.2
Bug #18300 typo in Figlet.php generating phrase
Submitted: 2011-02-23 23:52 UTC
From: green Assigned: wenz
Status: Closed Package: Text_CAPTCHA (version 0.4.2)
PHP Version: Irrelevant OS: linux
Roadmaps: (Not assigned)    
Subscription  


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 3 + 22 = ?

 
 [2011-02-23 23:52 UTC] green (Alexander Zubkov)
Description: ------------ It seems that there is a typo in Figlet.php init code on line 113 (ver 0.4.2): $phraseoptions = ... $this->_createPhrase($options); Here array with options is prepared, but wrong array is passed to function. It results in warning messages in log like: PHP Notice: Undefined offset: 0 in /usr/share/php/Text/CAPTCHA/Driver/Figlet.php on line 159 PHP Notice: Undefined offset: 1 in /usr/share/php/Text/CAPTCHA/Driver/Figlet.php on line 159

Comments

 [2011-02-23 23:55 UTC] green (Alexander Zubkov)
Added patch fix.patch.
 [2011-02-24 19:23 UTC] wenz (Christian Wenz)
-Status: Open +Status: Duplicate
thanks for the catch! The bug you reported is a duplicate of bug #18198, but your proposed patch fixes the root cause. Again, thanks!
 [2011-02-24 19:25 UTC] wenz (Christian Wenz)
-Status: Duplicate +Status: Assigned -Assigned To: +Assigned To: wenz
oh, sorry, #18198 has been deleted by the reporter. Anyway, an updated release will appear on the site shortly.
 [2011-02-24 19:48 UTC] wenz (Christian Wenz)
-Status: Assigned +Status: Closed
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/ fixed in release 0.4.3. Thanks for report & patch!