diff options
| author | Evan Winslow <evan@elgg.org> | 2011-11-06 22:17:01 -0800 |
|---|---|---|
| committer | Evan Winslow <evan@elgg.org> | 2011-11-06 22:17:01 -0800 |
| commit | 97a1e51939e3c8085f7a5d144a1a2174f6191278 (patch) | |
| tree | 49a93e98c4dc90baff7779f9d3a83acb330a29f6 /documentation/coding_standards | |
| parent | 8699afa034a94cc93a18b20a21aec70e036443fc (diff) | |
| parent | bdbf47c030c6ed8903f6511c5436cd29a234f29f (diff) | |
| download | elgg-97a1e51939e3c8085f7a5d144a1a2174f6191278.tar.gz elgg-97a1e51939e3c8085f7a5d144a1a2174f6191278.tar.bz2 | |
Merge pull request #91 from ewinslow/3397-positive-variable-names
Fixes #3397: Adds 'positive' variable name standard to best practices file.
Diffstat (limited to 'documentation/coding_standards')
| -rw-r--r-- | documentation/coding_standards/best_practices.txt | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/documentation/coding_standards/best_practices.txt b/documentation/coding_standards/best_practices.txt index 069ac42aa..df04aa845 100644 --- a/documentation/coding_standards/best_practices.txt +++ b/documentation/coding_standards/best_practices.txt @@ -15,6 +15,8 @@ work for developers, which means happier, more productive developers. * Use self-documenting variable names. $group_guids is better than $array. +* Use "positive" variable names. Prefer `$enable = true` to `$disable = false`. + * Functions returning an array should return an empty array instead of FALSE on no results. |