Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Changeset 6743

Show
Ignore:
Timestamp:
05/15/07 23:56:09 (1 year ago)
Author:
bitsweat
Message:

Simplify CgiRequest#query_parameters

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/actionpack/lib/action_controller/cgi_ext/parameters.rb

    r6742 r6743  
    2424 
    2525      module ClassMethods 
    26         # DEPRECATED: Use parse_form_encoded_parameters 
    2726        def parse_query_parameters(query_string) 
     27          return {} if query_string.blank? 
     28 
    2829          pairs = query_string.split('&').collect do |chunk| 
    2930            next if chunk.empty? 
    …… 
    3435          end.compact 
    3536 
    36           FormEncodedPairParser.new(pairs).result 
    37         end 
    38  
    39         # DEPRECATED: Use parse_form_encoded_parameters 
     37          UrlEncodedPairParser.new(pairs).result 
     38        end 
     39 
    4040        def parse_request_parameters(params) 
    41           parser = FormEncodedPairParser.new 
     41          parser = UrlEncodedPairParser.new 
    4242 
    4343          params = params.dup 
    …… 
    113113      end 
    114114 
    115       class FormEncodedPairParser < StringScanner #:nodoc: 
     115      class UrlEncodedPairParser < StringScanner #:nodoc: 
    116116        attr_reader :top, :parent, :result 
    117117 

  • trunk/actionpack/lib/action_controller/cgi_process.rb

    r6742 r6743  
    7070 
    7171    def query_parameters 
    72       @query_parameters ||= 
    73         (qs = self.query_string).empty? ? {} : CGI.parse_query_parameters(qs) 
     72      @query_parameters ||= CGI.parse_query_parameters(query_string) 
    7473    end 
    7574