agreed,
tho compared to code from projects like xt:commerce, cmsms is like heaven!
i know its always very difficult to agree on a common set of rules for formating code,
when ur working on it with more than say 3 ppl at the same time. but with every additional line of code
it gets more important.
Code cleanup...
-
- Support Guru
- Posts: 8169
- Joined: Tue Oct 19, 2004 6:44 pm
- Location: Fernie British Columbia, Canada
Re: Code cleanup...
It's very difficult, and hard to enforce on an open source project such as this. If you want cooperation, we feel you can't be too strict with coding standards, (well at least code formatting standards). Especially since what is easy to read for one developer is hard for another. It's kinda like a religious war, therer's no real point to it.
I'd rather be more concerned that people weren't adding XSS vulnerabilities, huge memory requirements, or were doing things in a database portable, translatable, and at least readable manner.
We can give guidelines for example, on how to use dates in tables, how to avoid XSS vulnerabilities, how to write your queries in a database independant way for adodb.... efficient use of smarty, how to make your module translatable, how to minimize the amount of weird bugs by using DEFINES where ever possible, etc, etc. and these are much more laudable goals than making sure everybody has their browsers set to tabs of 4, or where they put their opening brackets.
I'd rather be more concerned that people weren't adding XSS vulnerabilities, huge memory requirements, or were doing things in a database portable, translatable, and at least readable manner.
We can give guidelines for example, on how to use dates in tables, how to avoid XSS vulnerabilities, how to write your queries in a database independant way for adodb.... efficient use of smarty, how to make your module translatable, how to minimize the amount of weird bugs by using DEFINES where ever possible, etc, etc. and these are much more laudable goals than making sure everybody has their browsers set to tabs of 4, or where they put their opening brackets.
Follow me on twitter
Please post system information from "Extensions >> System Information" (there is a bbcode option) on all posts asking for assistance.
--------------------
If you can't bother explaining your problem well, you shouldn't expect much in the way of assistance.
Please post system information from "Extensions >> System Information" (there is a bbcode option) on all posts asking for assistance.
--------------------
If you can't bother explaining your problem well, you shouldn't expect much in the way of assistance.
Re: Code cleanup...
There exists a thread in the past, but I can't see any real results (docs or other)
...
http://forum.cmsmadesimple.org/index.ph ... 866.0.html

http://forum.cmsmadesimple.org/index.ph ... 866.0.html