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

Bug #20222 1.8.8 not compatible with PHP4
Submitted: 2014-03-18 15:28 UTC Modified: 2014-05-14 09:43 UTC
From: happydog Assigned: alec
Status: Closed Package: Mail_Mime (version 1.8.8)
PHP Version: Irrelevant OS: N/A
Roadmaps: 1.8.9    
Subscription  


 [2014-03-18 15:28 UTC] happydog (Mark Clements)
Description: ------------ In 1.8.8, a number of function signatures were changed so that some pass-by-reference arguments were given default values. In PHP4 this is invalid, and throws a parse error. However, the documentation for the release says the package is compatible with "PHP 4.3.0 or newer". If that is correct, then this is a bug that needs fixing (by removing the default values for these fields, e.g. in mime.php on line 500). Alternatively, if PHP4 support has been deliberately dropped, then this is a documentation bug, and the changelog should be updated to say PHP4 support was dropped in this release, and that the minimum version is now PHP 5.0. Expected result: ---------------- The package works on the versions of PHP that it claims to support. Actual result: -------------- It does not work on PHP4, despite having a minimum requirement of PHP 4.3.0.

Comments

 [2014-03-22 08:53 UTC] doconnor (Daniel O'Connor)
 [2014-03-22 08:53 UTC] doconnor (Daniel O'Connor)
-Status: Open +Status: Analyzed
 [2014-03-22 09:05 UTC] alec (Aleksander Machniak)
I'd be happy to forget about PHP4. However, http://pear.php.net/pepr/pepr-proposal-show.php?id=606 wasn't accepted. That's why we have https://github.com/pear/Mail_Mime2.
 [2014-05-14 09:24 UTC] alec (Aleksander Machniak)
-Status: Analyzed +Status: Closed -Assigned To: +Assigned To: alec
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. This was broken in https://github.com/pear/Mail_Mime/commit/b46f25b29a387463261ebee2cd0dba6f465adfca. Fixed.
 [2014-05-14 09:43 UTC] alec (Aleksander Machniak)
-Roadmap Versions: +Roadmap Versions: 1.8.9