[tor-commits] [atlas/master] Fix bug in history parsing function

art at torproject.org art at torproject.org
Thu Aug 23 10:16:42 UTC 2012


commit 259734c30993b77244cf4d27dd06e35d0f00fe12
Author: Arturo Filastò <arturo at filasto.net>
Date:   Thu Aug 23 12:16:27 2012 +0200

    Fix bug in history parsing function
---
 js/models/graph.js |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/js/models/graph.js b/js/models/graph.js
index 232f651..ed2601c 100644
--- a/js/models/graph.js
+++ b/js/models/graph.js
@@ -6,7 +6,8 @@ define([
   'backbone'
 ], function($, _, Backbone){
 
-    function parseHistory(history, model, relay) {
+    function parseHistory(history, model, relay, name) {
+        var newar;
         _.each(_.keys(history), function(period, i) {
             var first = history[period].first.split(' ');
             var date = first[0].split('-');
@@ -29,12 +30,11 @@ define([
                 // models.
                 // XXX probably want to refactor.
                 var mperiod = "bw_" + period.split("_")[1]
-                var newar = model.get(mperiod).write;
+                var newar = model.get(mperiod)[name];
                 newar.push([y,x]);
-                var toset = {mperiod: {write: newar}};
-                model.set(toset);
             });
         });
+        return newar;
     };
 
     function parseWeightHistory(history, model, name) {
@@ -100,8 +100,8 @@ define([
             var relay = data.relays[0];
             this.fingerprint = relay.fingerprint;
             // Parse the read and write history of the relay
-            var write_history = parseHistory(relay.write_history, model, relay);
-            var read_history = parseHistory(relay.read_history, model, relay);
+            var write_history = parseHistory(relay.write_history, model, relay, 'write');
+            var read_history = parseHistory(relay.read_history, model, relay, 'read');
             var toset = {mperiod: {read: read_history, write: write_history}};
             model.set(toset);
         },



More information about the tor-commits mailing list