From 83230afa51fd1ec6a1e224baeac59bd4be4eb0d4 Mon Sep 17 00:00:00 2001
From: David Thompson <dthompson2@worcester.edu>
Date: Thu, 29 Jan 2015 19:46:38 -0500
Subject: js: Clean up packageCount function.

* js/utils.js (guix.add): New function.
* js/controllers/packages.js (guix.packages.controller.packageCount):
  Reimplement.
---
 js/controller/packages.js | 7 ++++---
 js/utils.js               | 7 +++++++
 2 files changed, 11 insertions(+), 3 deletions(-)

(limited to 'js')

diff --git a/js/controller/packages.js b/js/controller/packages.js
index 89f01e9..d9e0839 100644
--- a/js/controller/packages.js
+++ b/js/controller/packages.js
@@ -61,9 +61,10 @@
     };
 
     controller.prototype.packageCount = function() {
-      return this.pager().pages.reduce(function(memo, page) {
-        return memo + page.length;
-      }, 0);
+      return _.chain(this.pager().pages)
+        .pluck('length')
+        .reduce(guix.add, 0)
+        .value();
     };
 
     controller.prototype.doSearch = function() {
diff --git a/js/utils.js b/js/utils.js
index c44ee23..cfaa7fd 100644
--- a/js/utils.js
+++ b/js/utils.js
@@ -17,6 +17,13 @@
 
 var guix = {};
 
+// Here is a perfect example of why Scheme is better than JavaScript:
+// + is an operator, not a function.  So if I want to, say, compute
+// the sum of an array of numbers, I need to write a wrapper function.
+guix.add = function(x, y) {
+  return x + y;
+};
+
 guix.clamp = function(n, min, max) {
   return Math.max(Math.min(n, max), min);
 };
-- 
cgit v1.2.3