NOTIFICATION.php

News and discussion about beta versions and CVS changes
User avatar
matt_t_hat
Posts: 1123
Joined: Sun Aug 21, 2005 4:45 pm
Location: UK
Contact:

NOTIFICATION.php

Postby matt_t_hat » Mon Jul 18, 2011 3:36 pm

The NOTIFICATION may want updating.

Specifically I am thinking of the coding standards which specify no closing ?> tag (which it has) but also are we still supporting php4? If not then function NOTIFICATION(...) should perhaps be function __construct(...)

Not biggies in and of themselves but worth thinking about maybe?
ftruscot
Nucleus Guru
Nucleus Guru
Posts: 7430
Joined: Wed Feb 22, 2006 6:19 pm
Location: Massachusetts
Contact:

Postby ftruscot » Mon Jul 18, 2011 3:59 pm

We haven't really started full-out implementing the coding guidelines that gRegor just published. We plan to slowly retrofit them to most the existing code as we go through the process of updating other things.
Is your question not solved yet?
Search our FAQ,
read the Documentation, or
browse the list of available plugins.

Check out my plugins
User avatar
matt_t_hat
Posts: 1123
Joined: Sun Aug 21, 2005 4:45 pm
Location: UK
Contact:

Postby matt_t_hat » Mon Jul 18, 2011 4:13 pm

ftruscot wrote:We haven't really started full-out implementing the coding guidelines that gRegor just published. We plan to slowly retrofit them to most the existing code as we go through the process of updating other things.
Yeah, I started to realise that when I was seeing end tags followed by white space.

It seems like you guys are all over this already so I'll go worry about other things and leave you to get on with being very cool.

Return to “Core Development”