Fixed the single-submission workflow

compare.rb

@@ -78,21 +78,24 @@ def link_to(where, exp, result = nil)
   @exp = Experiment.where(:name => experiment).first
 
   # Policy: we store one sample per user-agent and title combination.
-  @result = Canvas.where(:useragent => env["HTTP_USER_AGENT"],
-                         :experiment_id => @exp.id,
-                         :title => params["title"]).first
+  @sample = Sample.where(:useragent => env["HTTP_USER_AGENT"],
+                          :userinput => params["title"]).first
+
+  if @sample.nil? then
+    @sample = Sample.create()
+    @sample.useragent = env["HTTP_USER_AGENT"]
+    @sample.userinput = params["title"]
+    @sample.save
+  end
+
+  @result = Canvas.where(:sample_id => @sample.id,
+                         :experiment_id => @exp.id).first
 
   puts "Creating new canvas" if @result.nil?
   @result = Canvas.create() if @result.nil?
 
-  sample = Sample.new
-  sample.useragent = env["HTTP_USER_AGENT"]
-  sample.userinput = params["title"]
-  sample.save
-
   @result.experiment_id = @exp.id
-  @result.sample_id = sample.id
-  @result.title = params["title"]
+  @result.sample_id = @sample.id
   @result.pixels = params["pixels"]
   @result.png = params["png"]
   @result.save

model.rb

@@ -61,6 +61,9 @@ def graphics_card
   end
 
   def graphics_card_helper
+    return "UNKNOWN" if userinput.nil?
+    return "UNKNOWN" if userinput.length() == 0
+
     device_id = ""
     device_id = " (Device #{$1.strip})" if /Device ID\r?\n([^\n]*?)\n/m =~ userinput
     device_id = " (Device #{$1.strip})" if /Device ID(.*?)Adapter RAM/m =~ userinput
@@ -120,6 +123,7 @@ def graphics_card_helper
     # Last resort
     return "Intel(R) HD Graphics" if /Intel\(R\) HD Graphics/ =~ userinput
     return "Intel(R) HD Graphics" if /Inter\(R\) HD Graphics/ =~ userinput
+    return "UNKNOWN"
   end
 
   def title