X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Runtime.git;a=blobdiff_plain;f=lib%2FCatalyst%2FEngine.pm;h=34e48c3e3d46d77d22835b32078ea49fbe8df76e;hp=225a09a14ac05ccbee14ca31247142928551d36e;hb=da57fe993af32a6fe5d1813854f69fe8dedbfc29;hpb=a24232f92b5ad336245f57a3901df13b975f67af diff --git a/lib/Catalyst/Engine.pm b/lib/Catalyst/Engine.pm index 225a09a..34e48c3 100644 --- a/lib/Catalyst/Engine.pm +++ b/lib/Catalyst/Engine.pm @@ -7,17 +7,14 @@ use CGI::Simple::Cookie; use Data::Dump qw/dump/; use Errno 'EWOULDBLOCK'; use HTML::Entities; -use HTTP::Body; use HTTP::Headers; -use URI::QueryParam; use Plack::Loader; use Catalyst::EngineLoader; -use Encode (); +use Encode 2.21 'decode_utf8'; use Plack::Request::Upload; use Hash::MultiValue; -use utf8; - use namespace::clean -except => 'meta'; +use utf8; # Amount of data to read from input on each pass our $CHUNKSIZE = 64 * 1024; @@ -593,7 +590,9 @@ sub prepare_query_parameters { # Check for keywords (no = signs) # (yes, index() is faster than a regex :)) if ( index( $query_string, '=' ) < 0 ) { - $c->request->query_keywords($self->unescape_uri($query_string)); + my $keywords = $self->unescape_uri($query_string); + $keywords = decode_utf8 $keywords; + $c->request->query_keywords($keywords); return; } @@ -607,10 +606,13 @@ sub prepare_query_parameters { for my $item ( @params ) { my ($param, $value) - = map { $self->unescape_uri($_) } + = map { decode_utf8($self->unescape_uri($_)) } split( /=/, $item, 2 ); - $param = $self->unescape_uri($item) unless defined $param; + unless(defined $param) { + $param = $self->unescape_uri($item); + $param = decode_utf8 $param; + } if ( exists $query{$param} ) { if ( ref $query{$param} ) { @@ -668,20 +670,26 @@ sub prepare_uploads { my $request = $c->request; return unless $request->_body; + my $enc = $c->encoding; my $uploads = $request->_body->upload; my $parameters = $request->parameters; foreach my $name (keys %$uploads) { + $name = $c->_handle_unicode_decoding($name) if $enc; my $files = $uploads->{$name}; my @uploads; for my $upload (ref $files eq 'ARRAY' ? @$files : ($files)) { my $headers = HTTP::Headers->new( %{ $upload->{headers} } ); + my $filename = $upload->{filename}; + $filename = $c->_handle_unicode_decoding($filename) if $enc; + my $u = Catalyst::Request::Upload->new ( size => $upload->{size}, type => scalar $headers->content_type, + charset => scalar $headers->content_type_charset, headers => $headers, tempname => $upload->{tempname}, - filename => $upload->{filename}, + filename => $filename, ); push @uploads, $u; }