aboutsummaryrefslogtreecommitdiff
path: root/documentation/coding_standards
diff options
context:
space:
mode:
authorEvan Winslow <evan@elgg.org>2011-11-06 22:17:01 -0800
committerEvan Winslow <evan@elgg.org>2011-11-06 22:17:01 -0800
commit97a1e51939e3c8085f7a5d144a1a2174f6191278 (patch)
tree49a93e98c4dc90baff7779f9d3a83acb330a29f6 /documentation/coding_standards
parent8699afa034a94cc93a18b20a21aec70e036443fc (diff)
parentbdbf47c030c6ed8903f6511c5436cd29a234f29f (diff)
downloadelgg-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.txt2
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.