Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Report information
The Basics
Id: 54071
Status: resolved
Worked: 1 hour (60 min)
Priority: 0/
Queue: Catalyst-Plugin-ErrorCatcher
People
Owner: chisel [...] chizography.net
Requestors: cjbradford [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 0.0.6.4
Fixed in: 0.0.6.5
Attachments
catalyst-plugin-errorcatcher-auth-patch

History Show all quoted text
  Thu Jan 28 13:46:06 2010 cjbradford [...] gmail.com - Ticket created
Subject: Catalyst-Plugin-ErrorCatcher fails if C:P:Authentication is not loaded.
If C:P:Authentication is not loaded, C:P:ErrorCatcher fails because it calls $c->user_exists to check if the user is logged in. This method is provided by C:P:Authentication I've attached a one-line patch that solves this issue. At the moment I can't come up with a sane way of testing it (it works in my app!), without creating a second TestApp with Authentication not loaded. For reference, perl 5.8.8, Catalyst 5.80018, on Linux dev 2.6.22.19-0.2-xen #1 SMP 2008-12-18 10:17:03 +0100 x86_64 x86_64 x86_64 GNU/Linux. Happy to try and create a proper test for this if that would help.
Subject: catalyst-plugin-errorcatcher-auth-patch
Download catalyst-plugin-errorcatcher-auth-patch
application/octet-stream 663b

Message body not shown because it is not plain text.

  Sat Jan 30 10:58:36 2010 chisel [...] chizography.net - Correspondence added
Subject: Re: [rt.cpan.org #54071] Catalyst-Plugin-ErrorCatcher fails if C:P:Authentication is not loaded.
Date: Sat, 30 Jan 2010 11:57:12 +0000
To: "bug-Catalyst-Plugin-ErrorCatcher [...] rt.cpan.org" <bug-Catalyst-Plugin-ErrorCatcher [...] rt.cpan.org>
From: Chisel <chisel [...] herlpacker.co.uk>
Thanks for the report and patch. I'll investigate and release a fix over the next few days. -- I used my iPhone to send this. It took ages to write everything on the odd little keyboard. On 28 Jan 2010, at 18:46, "Colin Bradford via RT" <bug-Catalyst-Plugin-ErrorCatcher@rt.cpan.org Show quoted text
> wrote:
Show quoted text
> Thu Jan 28 13:46:06 2010: Request 54071 was acted upon. > Transaction: Ticket created by cjbradford > Queue: Catalyst-Plugin-ErrorCatcher > Subject: Catalyst-Plugin-ErrorCatcher fails if > C:P:Authentication is not > loaded. > Broken in: 0.0.6.4 > Severity: Normal > Owner: Nobody > Requestors: cjbradford@gmail.com > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=54071 > > > > If C:P:Authentication is not loaded, C:P:ErrorCatcher fails because it > calls $c->user_exists to check if the user is logged in. This > method is > provided by C:P:Authentication > > I've attached a one-line patch that solves this issue. At the > moment I > can't come up with a sane way of testing it (it works in my app!), > without creating a second TestApp with Authentication not loaded. > > For reference, perl 5.8.8, Catalyst 5.80018, on Linux dev > 2.6.22.19-0.2-xen #1 SMP 2008-12-18 10:17:03 +0100 x86_64 x86_64 > x86_64 > GNU/Linux. > > Happy to try and create a proper test for this if that would help. > <catalyst-plugin-errorcatcher-auth-patch>
  Sat Jan 30 10:58:37 2010 The RT System itself - Status changed from 'new' to 'open'
  Sun Jan 31 11:35:41 2010 chisel [...] chizography.net - Taken
  Sun Jan 31 12:51:49 2010 chisel [...] chizography.net - Correspondence added 60 min
Applied patch, added tests, released to CPAN as 0.0.6.5 (pending indexing, etc)
  Sun Jan 31 12:51:50 2010 chisel [...] chizography.net - Status changed from 'open' to 'resolved'
  Thu Jan 20 17:45:24 2011 chisel [...] chizography.net - Fixed in 0.0.6.5 added