Re: Working Version of Net_Monitor - Please Review :)

Looks ok - a few minor things: return TRUE/FALSE should be return true/false PEAR_ERROR is PEAR_Error require() doesnt need brackets - getAlerter()/getClient() etc. File names should not have '_' in them | ||require_once 'Net/Monitor/Service/Generic.php'; rather than ||require_once 'Net/Monitor/Service_Generic.php';| I'd use Net_Monitor_Service, rather than Net_Monitor_Service_Generic as the base class. It would be nice to make the subject for the error email configurable (eg. "[System Failure] %s" ) so It's easy to filter and delete them. Regards Alan Robert Peake wrote:

Dear Pear Group and Pear Developers Mailing List, A working version of the Net_Monitor package is available from: http://pear.php.net/pepr/pepr-proposal-show.php?id=171 Please take a moment to review the proposal writeup, package documentation, and sample source file (browseable phps). If you are a voting member and see anything that would cause you *not* to vote for this package, please, please let me know. Other comments are welcome of course, but I am especially interested in comments from voting members at this point as I hope to move this up to proposal status soon. Thanks very much for your time. Best, Robert