» Details |
---|
|
» Comment |
Code looks great. You've improved it considerably since your initial proposal. I'd still like to see better property and method documentation in the docblocks. Having the property or method name as the first line in a docblock is pretty much useless as the generated documentation index will look like: - $foo -foo - bar() - bar - baz() - baz It's much better to have a short descriptive sentence like - $foo - Contains the foo for bar - bar() - Bars the foo - baz() - Inverts the foo |