diff options
| author | Brian Ford | 2014-01-02 14:48:03 -0800 |
|---|---|---|
| committer | Brian Ford | 2014-01-02 22:59:43 -0800 |
| commit | 01c5be4681e34cdc5f5c461b7a618fefe8038919 (patch) | |
| tree | 0d96eb38e9df874ea9553d6caacd1275e8882855 /src/Angular.js | |
| parent | fd9a03e147aac7e952c6dda1f381fd4662276ba2 (diff) | |
| download | angular.js-01c5be4681e34cdc5f5c461b7a618fefe8038919.tar.bz2 | |
fix(ngShow/ngHide, ngIf): functions with zero args should be truthy
Previously, expressions that were a function with one or more arguments evaluated to
true, but functions with zero arguments evaluated to false.
This behavior seems both unintentional and undesirable. This patch makes a function
truthy regardless of its number of arguments.
Closes #5414
Diffstat (limited to 'src/Angular.js')
| -rw-r--r-- | src/Angular.js | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/Angular.js b/src/Angular.js index 386682a3..11ce5cf5 100644 --- a/src/Angular.js +++ b/src/Angular.js @@ -959,7 +959,9 @@ function fromJson(json) { function toBoolean(value) { - if (value && value.length !== 0) { + if (typeof value === 'function') { + value = true; + } else if (value && value.length !== 0) { var v = lowercase("" + value); value = !(v == 'f' || v == '0' || v == 'false' || v == 'no' || v == 'n' || v == '[]'); } else { |