Edit report at http://pear.php.net/bugs/bug.php?id=16730&edit=1

ID: 16730
Updated by: till@php.net
Reported By: till at php dot net
Summary: Spin off Log related code in System_Daemon
Status: Feedback
Type: Feature/Change Request
Package: System_Daemon
Operating System: n/a
Package Version: 0.9.2
PHP Version: 5.2.5
Roadmap Versions:
New Comment:

Heh, I wasn't calling for so drastic measures. ;-)

I mean, since you are pre-1.0, you can technically break the API as

Previous Comments:

[2009-10-23 14:12:38] kvz

-Status: Open
+Status: Feedback


[2009-10-23 14:12:24] kvz


I've actually been thinking of doing a complete rewrite of
System_Daemon, I've learned a trick or two since I made this code.
Aim is to keep the API the same, but revamp all other code. My guess is
it can be done in 30% of the lines.

Now to find me some time... ; )


[2009-10-23 13:53:11] till

IMHO, the System_Daemon::log() function and all related methods and
constants, should be pushed into a System_Daemon_Log class.

Of course there's nothing particulary "wrong" with this code, but I
feel like the System_Daemon class itself is too monolithic. It does too
many things at the same time, in order to work with that, I'd suggest
you spin off the log code into System_Daemon_Log. :-)


Search Discussions

Discussion Posts


Follow ups

Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 3 of 4 | next ›
Discussion Overview
grouppear-bugs @
postedOct 23, '09 at 12:12p
activeOct 23, '09 at 12:42p

2 users in discussion

Kevin: 3 posts Till: 1 post



site design / logo © 2022 Grokbase