Skip Menu |

This queue is for tickets about the Catalyst-Runtime CPAN distribution.

Report information
The Basics
Id: 87998
Status: resolved
Priority: 0/
Queue: Catalyst-Runtime

People
Owner: Nobody in particular
Requestors: NIERLEIN [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 5.90042
Fixed in: (no value)



Subject: Catalyst::Plugin::Compress broken since Catalyst::Plugin::Unicode::Encoding is included (Error code: ERR_CONTENT_DECODING_FAILED)
Since the Catalyst::Plugin::Unicode::Encoding is included in Catalyst itself, its no longer possible to define the order of plugins. And since unicode has to be loaded before the compress plugin, the output is garbled. According to: http://search.cpan.org/perldoc?Catalyst%3A%3APlugin%3A%3ACompress It says: "NOTE: If you want to use this module with Catalyst::Plugin::Unicode, You MUST load this plugin AFTER Catalyst::Plugin::Unicode." The attached patch puts default plugins in first.
Subject: patch-catalyst-pm-plugin-order.patch
--- a/Catalyst.pm 2013-08-21 13:30:00.883376453 +0200 +++ b/Catalyst.pm 2013-08-21 13:29:46.523373373 +0200 @@ -3001,7 +3001,7 @@ () } : $_ } @$plugins ]; - push @$plugins, $class->_default_plugins; + unshift @$plugins, $class->_default_plugins; $plugins = Data::OptList::mkopt($plugins || []); my @plugins = map {
This should be fixed up with the most recent 5.90080+ redo of the uncode work. We updated a number of modules on CPAN to work properly. https://metacpan.org/pod/Catalyst::Plugin::Compress should now work. please let me know if this is not the case. On Wed Aug 21 07:37:38 2013, NIERLEIN wrote: Show quoted text
> Since the Catalyst::Plugin::Unicode::Encoding is included in Catalyst > itself, its no longer possible to define the order of plugins. And > since unicode has to be loaded before the compress plugin, the output > is garbled. > > According to: > http://search.cpan.org/perldoc?Catalyst%3A%3APlugin%3A%3ACompress > > It says: > "NOTE: If you want to use this module with Catalyst::Plugin::Unicode, > You MUST load this plugin AFTER Catalyst::Plugin::Unicode." > > The attached patch puts default plugins in first.