From 3bf238445157912b419e237b5ce7e29052daf81b Mon Sep 17 00:00:00 2001
From: soulgalore <peter@soulgalore.com>
Date: Fri, 25 Jul 2014 22:09:44 +0200
Subject: [PATCH] use the aggregated data and show it, the same way as in
 summary #428

---
 lib/htmlRenderer.js | 22 ++++++++++++++++++----
 templates/sites.hb  |  8 ++++----
 2 files changed, 22 insertions(+), 8 deletions(-)

diff --git a/lib/htmlRenderer.js b/lib/htmlRenderer.js
index bdab109a0..447fd3462 100644
--- a/lib/htmlRenderer.js
+++ b/lib/htmlRenderer.js
@@ -84,10 +84,23 @@ HTMLRenderer.prototype.renderRules = function (cb) {
 renderHtmlToFile('rules', renderData, cb);
 };
 
-HTMLRenderer.prototype.renderSites = function (sitesAggregates, cb) {
+HTMLRenderer.prototype.renderSites = function(sitesAggregates, cb) {
+
+  var sitesAndAggregates = [];
+
+  // Add all sites data sorted
+  Object.keys(sitesAggregates).forEach(function(site) {
+    sitesAndAggregates.push({
+      "site": site,
+      "aggregates": sitesAggregates[site].filter(function(box) {
+        return (config.sitesColumns.indexOf(box.id) > -1);
+      }).sort(function(box, box2) {
+        return config.sitesColumns.indexOf(box.id) - config.sitesColumns.indexOf(box2.id);
+      })
+    });
+  });
   var renderData = {
-    "sites": Object.keys(sitesAggregates),
-    "aggregates": sitesAggregates,
+    "sitesAndAggregates": sitesAndAggregates,
     "columns": config.sitesColumns,
     "config": config,
     "pageMeta": {
@@ -95,7 +108,8 @@ HTMLRenderer.prototype.renderSites = function (sitesAggregates, cb) {
       "description": "",
     }
   };
-renderHtmlToFile('sites', renderData, cb, 'sites.html', '..');
+
+  renderHtmlToFile('sites', renderData, cb, 'sites.html', '..');
 };
 
 
diff --git a/templates/sites.hb b/templates/sites.hb
index 4dc41668f..a21e6f272 100644
--- a/templates/sites.hb
+++ b/templates/sites.hb
@@ -19,13 +19,13 @@
                 </tr>
               </thead>
               <tbody>
-                {{#each sites}}
+                {{#each sitesAndAggregates}}
               <tr>
                 <td>
-                <a href="{{getHostname this}}/index.html">{{this}}</a>
+                <a href="{{getHostname this.site}}/index.html">{{this.site}}</a>
                 </td>
-                {{#each ../columns}}
-                  <td>{{getSiteAggregatedValue .. 'median' this ../../aggregates }}</td>
+                {{#each aggregates}}
+                  <td data-sort-value="{{stats.median}}">{{getHumanReadable this stats.median true}}</td>
                 {{/each}}
               </tr>
               {{/each}}