Merge branch 'pr/135' into release-candidates/rc-5.90116
Jay Hannah [Tue, 16 Jan 2018 22:36:20 +0000 (16:36 -0600)]
1  2 
lib/Catalyst/Engine.pm

diff --combined lib/Catalyst/Engine.pm
@@@ -574,42 -574,44 +574,48 @@@ sub prepare_query_parameters 
      my ($self, $c) = @_;
      my $env = $c->request->env;
      my $do_not_decode_query = $c->config->{do_not_decode_query};
 -    my $default_query_encoding = $c->config->{default_query_encoding} || 
 -      ($c->config->{decode_query_using_global_encoding} ?
 -        $c->encoding : 'UTF-8');
  
 +    my $old_encoding;
 +    if(my $new = $c->config->{default_query_encoding}) {
 +      $old_encoding = $c->encoding;
 +      $c->encoding($new);
 +    }
 +
 +    my $check = $c->config->{do_not_check_query_encoding} ? undef :$c->_encode_check;
      my $decoder = sub {
        my $str = shift;
        return $str if $do_not_decode_query;
 -      return $str unless $default_query_encoding;
 -      return decode( $default_query_encoding, $str);
 +      return $c->_handle_param_unicode_decoding($str, $check);
      };
  
      my $query_string = exists $env->{QUERY_STRING}
          ? $env->{QUERY_STRING}
          : '';
  
-     # Check for keywords (no = signs)
-     # (yes, index() is faster than a regex :))
-     if ( index( $query_string, '=' ) < 0 ) {
-         my $keywords = $self->unescape_uri($query_string);
-         $keywords = $decoder->($keywords);
-         $c->request->query_keywords($keywords);
-         return;
-     }
      $query_string =~ s/\A[&;]+//;
  
-     my $p = Hash::MultiValue->new(
-         map { defined $_ ? $decoder->($self->unescape_uri($_)) : $_ }
-         map { ( split /=/, $_, 2 )[0,1] } # slice forces two elements
-         split /[&;]+/, $query_string
-     );
+     my @unsplit_pairs = split /[&;]+/, $query_string;
+     my $p = Hash::MultiValue->new();
+     my $is_first_pair = 1;
+     for my $pair (@unsplit_pairs) {
+         my ($name, $value)
+           = map { defined $_ ? $decoder->($self->unescape_uri($_)) : $_ }
+             ( split /=/, $pair, 2 )[0,1]; # slice forces two elements
+         if ($is_first_pair) {
+             # If the first pair has no equal sign, then it means the isindex
+             # flag is set.
+             $c->request->query_keywords($name) unless defined $value;
+             $is_first_pair = 0;
+         }
+         $p->add( $name => $value );
+     }
  
 +    $c->encoding($old_encoding) if $old_encoding;
      $c->request->query_parameters( $c->request->_use_hash_multivalue ? $p : $p->mixed );
  }