diff options
| author | cweiske <cweiske@b3834d28-1941-0410-a4f8-b48e95affb8f> | 2009-11-02 09:41:32 +0000 |
|---|---|---|
| committer | cweiske <cweiske@b3834d28-1941-0410-a4f8-b48e95affb8f> | 2009-11-02 09:41:32 +0000 |
| commit | fee364b94e1bafdb751b93014b43c5576bd9e039 (patch) | |
| tree | d68d218b6b5175f7a6fc1dc41d9732f1549bb2c1 /doc | |
| parent | 064c45d77e5ca6078b756de6c03c34f6be36641a (diff) | |
| download | semanticscuttle-fee364b94e1bafdb751b93014b43c5576bd9e039.tar.gz semanticscuttle-fee364b94e1bafdb751b93014b43c5576bd9e039.tar.bz2 | |
basic developer rules
git-svn-id: https://semanticscuttle.svn.sourceforge.net/svnroot/semanticscuttle/trunk@504 b3834d28-1941-0410-a4f8-b48e95affb8f
Diffstat (limited to 'doc')
| -rw-r--r-- | doc/developers/rules | 30 |
1 files changed, 30 insertions, 0 deletions
diff --git a/doc/developers/rules b/doc/developers/rules new file mode 100644 index 0000000..701a215 --- /dev/null +++ b/doc/developers/rules @@ -0,0 +1,30 @@ +Rules for developers +==================== + +1. Coding style +--------------- +SemanticScuttle uses the PEAR Coding Standards. +While quite some parts still do not follow them, all of the +code will be coverted to them. When developing new code, +adhere to it. + +A helpful tool to check your coding style is PHP CodeSniffer, +http://pear.php.net/package/PHP_CodeSniffer + + +2. Unit tests +------------- +At least the service and model classes have unit tests. +If you fix things in there, make sure you +a) do not break the tests or +b) fix the tests if the old behavior was broken + + +3. Keep security in mind +------------------------ +As a web application, there are several attack vectors to SemanticScuttle. +When processing user input (form variables, URL parameters) +be sure to convert and validate them. If you expect a bookmark id, +there is no reason not to cast the variable to (int). + +Filter input, escape output. |