diff options
author | Michael Leibman <michael.leibman@gmail.com> | 2012-02-05 14:23:20 -0800 |
---|---|---|
committer | Michael Leibman <michael.leibman@gmail.com> | 2012-02-05 14:23:20 -0800 |
commit | bca0a0f6717be4940b670d88bdaba788ce681e46 (patch) | |
tree | 4120daabaa608ab51bf1a227fdb4a2218cbee4a4 /slick.dataview.js | |
parent | 302780f55c9f89a2f0a5beb26a6e1bc589dab63f (diff) | |
download | SlickGrid-bca0a0f6717be4940b670d88bdaba788ce681e46.zip SlickGrid-bca0a0f6717be4940b670d88bdaba788ce681e46.tar.gz SlickGrid-bca0a0f6717be4940b670d88bdaba788ce681e46.tar.bz2 |
Formatting fixup of a previous pull request merge.
Diffstat (limited to 'slick.dataview.js')
-rw-r--r-- | slick.dataview.js | 35 |
1 files changed, 18 insertions, 17 deletions
diff --git a/slick.dataview.js b/slick.dataview.js index a654aec..bcb3051 100644 --- a/slick.dataview.js +++ b/slick.dataview.js @@ -825,7 +825,7 @@ groupTotals.avg = {}; } if (this.nonNullCount_ != 0) { - groupTotals.avg[this.field_] = (this.sum_ / this.nonNullCount_)*100/100; + groupTotals.avg[this.field_] = this.sum_ / this.nonNullCount_; } }; } @@ -879,26 +879,27 @@ } function SumAggregator(field) { - this.field_ = field; + this.field_ = field; - this.init = function() { - this.sum_ = null; - }; + this.init = function () { + this.sum_ = null; + }; - this.accumulate = function(item) { - var val = item[this.field_]; - if (val != null && val != "" && val != NaN) { - this.sum_ += parseFloat(val); - } - }; + this.accumulate = function (item) { + var val = item[this.field_]; + if (val != null && val != "" && val != NaN) { + this.sum_ += parseFloat(val); + } + }; - this.storeResult = function(groupTotals) { - if (!groupTotals.sum) { - groupTotals.sum = {}; - } - groupTotals.sum[this.field_] = this.sum_; - } + this.storeResult = function (groupTotals) { + if (!groupTotals.sum) { + groupTotals.sum = {}; + } + groupTotals.sum[this.field_] = this.sum_; } + } + // TODO: add more built-in aggregators // TODO: merge common aggregators in one to prevent needles iterating |