[Autotest] [PATCH] Handle missing 'content' parameter in TKO Spreadsheet View

[James Ren]

Handle missing 'content' parameter in TKO Spreadsheet View

Signed-off-by: James Ren <james...@google.com>

--- autotest/frontend/client/src/autotest/tko/ContentSelect.java        
2010-05-05 13:51:22.000000000 -0700
+++ autotest/frontend/client/src/autotest/tko/ContentSelect.java        
2010-05-06 11:02:01.000000000 -0700
@@ -106,21 +106,25 @@
     }
 
     public void addHistoryArguments(HistoryToken arguments, String name) {
-      List<String> fields = new ArrayList<String>();
-      for (int i = 0; i < contentSelect.getItemCount(); i++) {
-          if (contentSelect.isItemSelected(i)) {
-              fields.add(contentSelect.getValue(i));
-          }
-      }
-      String fieldList = Utils.joinStrings(",", fields);
-      arguments.put(name, fieldList);
+        List<String> fields = new ArrayList<String>();
+        for (int i = 0; i < contentSelect.getItemCount(); i++) {
+            if (contentSelect.isItemSelected(i)) {
+                fields.add(contentSelect.getValue(i));
+            }
+        }
+        String fieldList = Utils.joinStrings(",", fields);
+        arguments.put(name, fieldList);
 
-      if (contentSelect.isVisible()) {
-          arguments.put(name + HISTORY_OPENED, "true");
-      }
-  }
+        if (contentSelect.isVisible()) {
+            arguments.put(name + HISTORY_OPENED, "true");
+        }
+    }
 
     public void handleHistoryArguments(Map<String, String> arguments, String 
name) {
+        if (!arguments.containsKey(name)) {
+            return;
+        }
+
         Set<String> fields = new 
HashSet<String>(Arrays.asList(arguments.get(name).split(",")));
         for (int i = 0; i < contentSelect.getItemCount(); i++) {
             if (fields.contains(contentSelect.getValue(i))) {
_______________________________________________
Autotest mailing list
Autotest@test.kernel.org
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest