From: John Napiorkowski Date: Tue, 28 Jul 2015 18:44:39 +0000 (-0500) Subject: prep release X-Git-Tag: 5.90097^0 X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Runtime.git;a=commitdiff_plain;h=a1dba4cf60acfe8c592a80dad86a076cd3eb1dec prep release --- diff --git a/Changes b/Changes index 92c49f3..f5ff718 100644 --- a/Changes +++ b/Changes @@ -1,9 +1,13 @@ # This file documents the revision history for Perl extension Catalyst. -TBA +5.90097 - 2015-07-28 - $c->uri_for now defines a final argument for setting the URL fragment /URL anchor. This is now the canonical approach to setting a fragment via uri_for. + - Reverted how we treat $c->uri_for($path) where $path is a string. When + we introduced the UTF-8 work we started encoding stringy paths, which + breaks code that did not expect that. We now consider stringy $path to + be 'expert' mode and you are expected to perform all nessary encoding. 5.90096 - 2015-07-27 - Fixed regression introduced in previous release that prevented a URI diff --git a/lib/Catalyst.pm b/lib/Catalyst.pm index 6523d21..5a45b92 100644 --- a/lib/Catalyst.pm +++ b/lib/Catalyst.pm @@ -180,7 +180,7 @@ sub composed_stats_class { __PACKAGE__->_encode_check(Encode::FB_CROAK | Encode::LEAVE_SRC); # Remember to update this in Catalyst::Runtime as well! -our $VERSION = '5.90096'; +our $VERSION = '5.90097'; $VERSION = eval $VERSION if $VERSION =~ /_/; # numify for warning-free dev releases sub import { diff --git a/lib/Catalyst/Runtime.pm b/lib/Catalyst/Runtime.pm index cafc869..cad8cb2 100644 --- a/lib/Catalyst/Runtime.pm +++ b/lib/Catalyst/Runtime.pm @@ -7,7 +7,7 @@ BEGIN { require 5.008003; } # Remember to update this in Catalyst as well! -our $VERSION = '5.90096'; +our $VERSION = '5.90097'; $VERSION = eval $VERSION if $VERSION =~ /_/; # numify for warning-free dev releases =head1 NAME diff --git a/lib/Catalyst/Upgrading.pod b/lib/Catalyst/Upgrading.pod index ac230c1..487787a 100644 --- a/lib/Catalyst/Upgrading.pod +++ b/lib/Catalyst/Upgrading.pod @@ -8,10 +8,6 @@ In older versions of Catalyst one could construct a L with a fragment (such https://localhost/foo/bar#fragment) by using a '#' in the path or final argument, for example: - $c->uri_for('/mypath#fragment'); - -or: - $c->uri_for($action, 'foo#fragment'); This behavior was never documented and would break if using the Unicode plugin, or when @@ -39,7 +35,7 @@ delimiter, which is going to be a breaking change for you. You need to alter yo to match the new specification or modify uri_for for your local case. Patches to solve this are very welcomed, as long as they don't break existing test cases. -B If you are using the alternative: +B If you are using the string form of the first argument: $c->uri_for('/foo/bar#baz')