From 9a3590a3f1e3cdde310e8bec4f40f10b48971b59 Mon Sep 17 00:00:00 2001
From: Bryce Allen <ballen@ci.uchicago.edu>
Date: Sat, 27 Jun 2015 11:35:31 -0500
Subject: [PATCH] weaponplanner: fix missing components

---
 web/weaponplanner.html | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/web/weaponplanner.html b/web/weaponplanner.html
index 2f36116..81565db 100644
--- a/web/weaponplanner.html
+++ b/web/weaponplanner.html
@@ -165,9 +165,19 @@
  
         $.getJSON(DATA_PATH + "weapon/" + weapon_id + "_tree.json",
                   function(data) {
-                      all_components = Object.keys(
-                                        data[data.length-1]["components"]);
+                      // first pass: collect all components and sort them
+                      var all_dict = {};
+                      for (i=0; i<data.length; i++) {
+                          var components = Object.keys(data[i]["components"]);
+                          for (j=0; j<components.length; j++) {
+                              all_dict[components[j]] = 0;
+                          }
+                      }
+                      var all_components = Object.keys(all_dict);
                       all_components.sort();
+                      // second pass: generate the fieldset for each weapon
+                      //  path. Note that the template uses all components
+                      //  to order the components and make them line up
                       for (i=0; i<data.length; i++) {
                           delta = {};
                           path = data[i];