Re: Req #5480 [Opn->Fbk]: Related to debug and logging errors
| From: | Richard U | Date: | Mon, 26 Sep 2005 07:11:39 +0000 |
| Subject: | Re: Req #5480 [Opn->Fbk]: Related to debug and logging errors | ||
| References: | 1 2 | Groups: | php.pear.qa |
| Request: | Send a blank email to pear-qa+get-3073@lists.php.net to get a copy of this message | ||
i will send an example of how i think this should work tomorrow just
flat out today
On 26 Sep 2005 07:06:00 -0000, PEAR Bug Database <pear-qa@lists.php.net> wrote:
> ATTENTION! Do NOT reply to this email!
> To reply, use the web interface found at
> http://pear.php.net/bugs/bug.php?id=5480&edit=2
>
>
> ID: 5480
> Updated by: lsmith@php.net
> Reported By: richard dot udo at gmail dot com
> -Status: Open
> +Status: Feedback
> Id: 5480
> Type: Feature/Change Request
> Package: LiveUser_Admin
> Operating System: Any
> PHP Version: 4.4.1
>
>
> Previous Comments:
> ------------------------------------------------------------------------
>
> [2005-09-23 06:21:11] lsmith at php dot net
>
> I agree as well ..
>
>
>
> So how should this exactly work?
>
> Should it be done in the constructor or in the init method?
>
> Should we even keep $GLOBALS['_LIVEUSER_DEBUG']?
>
>
>
> Maybe we should just have a parameter in the constructor that can
> either be a boolean or a logger instance and we do not even bother with
> adding the windows logger outselves?
>
>
>
> I am also not sure if we should not just make the log property public
> and drop the addErrorLog() method.
>
> ------------------------------------------------------------------------
>
> [2005-09-21 21:08:57] dufuz at php dot net
>
> I think you are absolutly right on that one.
>
> ------------------------------------------------------------------------
>
> [2005-09-21 20:18:12] richard dot udo at gmail dot com
>
> Description:
> ------------
> Hey guys... looking at the 2 functions here....
>
> http://cvs.php.net/co.php/pear/LiveUser_Admin/Admin.php?r=1.46#176ò§ûN
> øyzË@L»Þhf and
>
> here...
>
> http://cvs.php.net/co.php/pear/LiveUser_Admin/Admin.php?r=1.46#176
>
> you can see that when your debugging you automatically get the window
>
> popping up with the default logger thats been used << $winlog =
>
> &Log::factory('win', 'LiveUser_Admin'); >> i think that you
> should be
>
> able to pass the default logger in the config array that would
>
> overwrite this one so that you could controll the way your debug
>
> messages are outputted.... what do you guys think?
>
> ------------------------------------------------------------------------
>
>
>