Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.5.2

Bug #19753 noisy version_compare warning
Submitted: 2012-12-17 01:52 UTC
From: caronc Assigned: heino
Status: Closed Package: Net_NNTP (version 1.5.0RC2)
PHP Version: 5.3.19 OS: Centos 5.8
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 : 49 - 27 = ?

 
 [2012-12-17 01:52 UTC] caronc (Chris Caron)
Description: ------------ What was meant to be a warning message for people with a specific php version has become a warning message for absolutely everyone except the target audience. if (version_compare(PHP_VERSION, '5.2.11') === 1) { should read: if (version_compare(PHP_VERSION, '5.2.11') === 0) { residing in NNTP/Protocol/Client.php Test script: --------------- use this package with any PHP version 'other' than 5.2.11 and observe the warning: PHP bug 16657 breaks feof() on socket streams! Connection consistency might be compromised! For those who like leaving warnings on to help debug code... this repetitive warning that isn't applicable can get annoying :) Expected result: ---------------- <nothing> Actual result: -------------- PHP bug 16657 breaks feof() on socket streams! Connection consistency might be compromised!

Comments

 [2012-12-17 01:52 UTC] caronc (Chris Caron)
 [2012-12-17 02:19 UTC] heino (Heino H. Gehlsen)
-Status: Open +Status: Verified -Assigned To: +Assigned To: heino
 [2012-12-17 18:01 UTC] heino (Heino H. Gehlsen)
Relates to bug bug #16657 and #19227...
 [2013-05-31 20:51 UTC] janpascal (Jan-Pascal van Best)
-Status: Verified +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 1.5.0RC3, thank you for reporting this bug and providing the patch to fix it