Pluf Framework

Issue 642: migrate.php: PlufErrorHandler pokes on E_STRICT violation in PEAR

Reported by Thomas Keller, Mar 23, 2011

I think this should sound familiar:

Error in getopt command line: Non-static method PEAR::isError() 
should not be called statically, assuming $this from incompatible 
context

Problem is that my PEAR installation resides in /opt/local/lib/php 
and that, while the PlufErrorHandler tries to ignore PEAR warnings 
and errors already, it fails to do that properly for violations 
coming from command line parsing.

The reason is that Pluf::start($what['conf']) is called many lines 
later in migrate.php and only this sets 'pear_path', so the error 
handler still assumes the default '/usr/share/php'.

I think all this boils down to the fact that Pluf's error handler is 
set as soon as Pluf.php is required (line 396 of Pluf.php) and that 
Pluf.php is require'd too early in migrate.php. I'll post a patch 
for this in a few.

Comment 1 by Thomas Keller, Mar 28, 2011

The promised patch.

Comment 2 by Thomas Keller, May 25, 2011

The patch has been committed in revision 3972264.
Status: Fixed
Owner: tommyd

Created: 2 years 1 month ago by Thomas Keller

Updated: 1 year 11 months ago

Status: Fixed

Owner: Thomas Keller

Labels:
Priority:Medium
Type:Defect