[dev] Notification_Event first run

Jon Parise jon at horde.org
Fri Jan 10 19:31:46 PST 2003


On Fri, Jan 10, 2003 at 07:24:17PM -0500, Hans Lellelid wrote:

> > There are a few style-related problems, and I'd make $message
> > explicitly private, but looks good otherwise.
> 
> I attached a cleaned-up version -- hopefully this fixes the style
> issues.  Let me know if you see anything else.

Private variables should be prefixed with a leading underscope (e.g.
$this->_message).  Also, some more whitespace in the comments would be
nice.

-- 
Jon Parise (jon at horde.org) :: The Horde Project (http://horde.org/)


More information about the dev mailing list