diff options
| author | Evan Winslow <evan.b.winslow@gmail.com> | 2011-11-07 00:40:23 +0000 |
|---|---|---|
| committer | Evan Winslow <evan.b.winslow@gmail.com> | 2011-11-07 00:40:23 +0000 |
| commit | bdbf47c030c6ed8903f6511c5436cd29a234f29f (patch) | |
| tree | 49a93e98c4dc90baff7779f9d3a83acb330a29f6 /documentation/coding_standards | |
| parent | 8699afa034a94cc93a18b20a21aec70e036443fc (diff) | |
| download | elgg-bdbf47c030c6ed8903f6511c5436cd29a234f29f.tar.gz elgg-bdbf47c030c6ed8903f6511c5436cd29a234f29f.tar.bz2 | |
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. |