| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.2.1 |
| Bug #7254 | E_STRICT: Only variable references should be returned by reference | ||
|---|---|---|---|
| Submitted: | 2006-03-30 04:53 UTC | ||
| From: | jausions | Assigned: | doconnor |
| Status: | Closed | Package: | Crypt_RSA (version 1.2.0b) |
| PHP Version: | 5.1.1 | OS: | Irrelevant |
| Roadmaps: | (Not assigned) | ||
[2006-03-30 04:53 UTC] jausions
(Philippe Jausions)
Description:
------------
When the MathLoader fails to load an extension, the returned error triggers this output (with E_ALL):
Notice: Only variable references should be returned by reference in c:\OpenSA\Apache\php\PEAR\Crypt\RSA\MathLoader.php on line 127
Fix: first assign the error to a variable, then return it.
<?php
$e = $err_handler->getLastError();
return $e;
By the way: please use single quotes ' instead of double-quotes " for all the text string.
Comments
[2008-11-01 09:12 UTC] doconnor
(Daniel O'Connor)
Attached patch drops the use of references; which should be fine for php 5.X
Its been so long since I set foot in PHP 4 land that I can't effectively judge the impact of these changes in that context.
[2009-01-05 08:02 UTC] doconnor
(Daniel O'Connor)
This bug has been fixed in CVS.
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.