Forums

Non-static methods being called as static

Permalink
There is one thing i cannot understand about concrete5 - why on earth are some methods (such as Job::installByPackage) NOT declared as static when they are supposed to be and are used as static ones.

This is annoying because the autocomplete and contex help in IDE's (such as NetBeans) doesn't work..

Does this dates back to PHP 4 times or is it a part of some conception i don't clearly get?
 
jordanlev replied on at Permalink Reply
jordanlev
If I had to guess I'd say it's either because that code was written long ago when PHP4 was more widely used, or that it's just sloppy coding (and then nobody ever went back to refactor it). I don't think there's any master plan behind it. If you have the time and inclination, you could contribute to the project via github to clean those up.

In case anyone is interested, here are some stackoverflow questions about the optional-ness of the static keyword in function declarations:
http://stackoverflow.com/questions/3754786/calling-non-static-metho...
http://stackoverflow.com/questions/3387633/is-the-static-keyword-ne...