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

Bug #10149 Problems when E_STRICT is turned on in php.ini
Submitted: 2007-02-22 00:05 UTC
From: zoltee at yahoo dot com Assigned: davidc
Status: Closed Package: HTML_AJAX (version 0.5.0)
PHP Version: 5.2.1 OS: WinXP SP2
Roadmaps: 0.5.1    
Subscription  
Comments Add Comment Add patch


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 : 30 - 3 = ?

 
 [2007-02-22 00:05 UTC] zoltee at yahoo dot com (zoltan)
Description: ------------ I have error_reporting = E_ALL | E_STRICT in php.ini and this is what I get: <br /> 2<b>Strict Standards</b>: Assigning the return value of new by reference is deprecated in <b>C:\work\web\pear\HTML\AJAX\Server.php</b> on line <b>158</b><br /> 3<br /> 4<b>Strict Standards</b>: Assigning the return value of new by reference is deprecated in <b>C:\work\web\pear\HTML\AJAX.php</b> on line <b>612</b><br /> 5<br /> 6<b>Strict Standards</b>: Assigning the return value of new by reference is deprecated in <b>C:\work\web\pear\HTML\AJAX\Serializer\JSON.php</b> on line <b>46</b><br /> 7<br />

Comments

 [2007-03-04 21:06 UTC] davidc (David Coallier)
HTML_AJAX is not only for php5 and is not E_STRICT compliant. Unless we decide to make a php5 only version, this is going to stay something you have to fix on your development server side (E_ALL ^ E_STRICT)
 [2007-03-05 16:13 UTC] jeichorn (Joshua Eichorn)
Were not going to fix ever E_STRICT error, but I don't see a problem with fixing the returning a new value by reference stuff. Its just a matter of $return = new Class(); return $return; instead of return new Class();
 [2007-03-05 21:01 UTC] jeichorn (Joshua Eichorn)
In this case all warnings are from lines like: $this->debug =& new HTML_AJAX_Debug($errstr, $errline, $errno, $errfile); In general i prefer not to add the extra reference in these cases unless its actually needed, and in these 3 cases I didn't see the need so I removed the &. Fixed in svn