[BUG FIX] Form inputs with no name attribute blew up in Safari.

wiecklabs · Feb 13, 2009 · aa371a3 · aa371a3
1 parent 86d1c3a
commit aa371a3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/Rakefile b/Rakefile
@@ -32,7 +32,7 @@ require "rake/gempackagetask"
 
 NAME = "wheels"
 SUMMARY = "Wheels Framework"
-GEM_VERSION = "0.2.5.1"
+GEM_VERSION = "0.2.5.2"
 
 spec = Gem::Specification.new do |s|
   s.name = NAME

diff --git a/lib/wheels/rack/utils.rb b/lib/wheels/rack/utils.rb
@@ -2,12 +2,18 @@ module Rack
   module Utils
     def parse_query(qs, d = '&;')
       params = {}
+
       (qs || '').split(/[#{d}] */n).each do |param|
         next if param == ''
 
         keys, value = unescape(param).split("=", 2)
         keys = keys.scan /[^\[\]]+|(?=\[\])/
         key = keys.pop
+
+        # If an input has no key, Firefox will not send it, but
+        # Safari will, so we ignore key-less form values.
+        next unless key
+
         if key.empty?
           key = keys.pop
           hash = keys.inject(params) { |h, k| h[k] ||= {} }
@@ -21,7 +27,7 @@ def parse_query(qs, d = '&;')
       params
     end
     module_function :parse_query
-    
+
     module Multipart
       $VERBOSE, __verbose = nil, $VERBOSE
       EOL = "\r\n"