[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