Merge pull request #205 from airbnb/chris/sunburst-fix

fix sunburst error. add `less` to package.json
apache · Mar 29, 2016 · 5561a49 · 5561a49
2 parents 74c72b3 + 1aee5b7
commit 5561a49
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/dashed/assets/package.json b/dashed/assets/package.json
@@ -59,6 +59,7 @@
     "imports-loader": "^0.6.5",
     "jquery": "^2.2.1",
     "jquery-ui": "^1.10.5",
+    "less": "^2.6.1",
     "less-loader": "^2.2.2",
     "nvd3": "1.8.2",
     "react": "^0.14.7",

diff --git a/dashed/assets/visualizations/sunburst.js b/dashed/assets/visualizations/sunburst.js
@@ -293,7 +293,7 @@ function sunburstVis(slice) {
         }
         var currentNode = root;
         for (var j = 0; j < levels.length; j++) {
-          var children = currentNode.children;
+          var children = currentNode.children || [];
           var nodeName = levels[j];
           // If the next node has the name "0", it will
           var isLeafNode = (j >= levels.length - 1) || levels[j+1] === 0;