PHP error notices running demos about firephp

[GoogleCodeExporter]

Original comment by [email protected] on 16 Sep 2008 at 5:15

• Changed state: Accepted

[GoogleCodeExporter]

Original comment by [email protected] on 22 Sep 2008 at 9:45

• Added labels: release-0.2

[GoogleCodeExporter]

I did not notice there is already a patch available, so I fixed this, too.
I used empty() instead of isset() and the patch contains the filename to patch 
for
easy merging.

Original comment by [email protected] on 23 Sep 2008 at 9:14

Attachments:

• notices.patch

[GoogleCodeExporter]

Fixed in 0.2 BETA

Original comment by [email protected] on 1 Oct 2008 at 6:33

• Changed state: Started

[GoogleCodeExporter]

I've just installed FirePHP 0.2.b.2 extension, and PEAR package 0.2.0beta4

And there are still some NOTICE errors. 
Detected under my PEAR::Log driver proposal
(http://pear.php.net/pepr/pepr-proposal-show.php?id=578), with the backtrace 
example.

Here are the two lines 517-518 fixed (to avoid notice errors)

'File'=>$this->_escapeTraceFile(isset($trace[$i]['file'])?$trace[$i]['file']:'')
,
                          'Line'=>isset($trace[$i]['line'])?$trace[$i]['line']:'',

Original comment by [email protected] on 14 Oct 2008 at 9:29

[GoogleCodeExporter]

Will be fixed in BETA 5. Thanks!

Original comment by [email protected] on 14 Oct 2008 at 10:31

[GoogleCodeExporter]

Original comment by [email protected] on 22 Oct 2008 at 5:11

• Changed state: Fixed