» Details |
---|
|
» Comment |
Pleas fix your indenting (I saw at least one place where two spaces were used instead of four) All functions that return values need @return in their docblocks. All var and function docblocks should have @access in them. Any private vars or functions should have '_' prepended to their names. It looks like most of the member vars should be private. I don't believe it's in CS, but I would prefer the docblocks be directly above their code (no extra line between the docblock and var or function). |