Skip Menu |

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

Report information
The Basics
Id: 42313
Status: resolved
Priority: 0/
Queue: Catalyst-Plugin-DebugCookie

People
Owner: jgoulah [...] cpan.org
Requestors: bobtfish [...] bobtfish.net
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in:
  • 0.999001
  • 0.999002
Fixed in: (no value)



Subject: Checking $c->{debug} is broken, and breaks Catalyst 5.80.
The attached diff fixes Catalyst::Plugin::DebugCookie for Catalyst 5.80 by allways calling the debug method, rather than looking in $self->{debug} directly. Note that I am _also_ fixing RenderView overwriting $c->config->{debug}.
Subject: Catalyst-Plugin-DebugCookie-5.80.diff
Index: lib/Catalyst/Plugin/DebugCookie.pm =================================================================== --- lib/Catalyst/Plugin/DebugCookie.pm (revision 9055) +++ lib/Catalyst/Plugin/DebugCookie.pm (working copy) @@ -6,7 +6,7 @@ use Class::C3; use Catalyst::Plugin::DebugCookie::Util qw/check_debug_cookie_value/; -our $VERSION = '0.999002'; +our $VERSION = '0.999003'; =head1 NAME @@ -99,7 +99,7 @@ sub debug { my $self = shift; if (ref $self) { - return $self->{debug} ||= $self->valid_debug_mode; + return $self->next::method(@_) || $self->valid_debug_mode; } else { $self->next::method(@_); } Index: Changes =================================================================== --- Changes (revision 9055) +++ Changes (working copy) @@ -1,3 +1,5 @@ +0.999003 + - Fix to work correctly with Catalyst 5.80 0.999002 2008-10-02 - up Catalyst dep b/c this requires Catalyst::Stats
applied and uploaded module to cpan