Skip Menu |

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

Report information
The Basics
Id: 41073
Status: resolved
Priority: 0/
Queue: Catalyst-Controller-WrapCGI

People
Owner: Nobody in particular
Requestors: bobtfish [...] bobtfish.net
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in:
  • 0.002
  • 0.0021
  • 0.0022
  • 0.0023
Fixed in: (no value)



Subject: Does not work with Catalyst 5.80
The tests for your module fail as you import a 'meta' function into a Catalyst::Controller package. The patch attached fixes the issue in the test suite, but I think you also need to change the documentation to point out the issue, and discourage importing into a Catalyst controller.
Subject: Catalyst-Controller-WrapCGI.diff
diff -ur Catalyst-Controller-WrapCGI-0.0023/t/lib/TestApp/Controller/Root.pm Catalyst-Controller-WrapCGI-0.0023-t0m/t/lib/TestApp/Controller/Root.pm --- Catalyst-Controller-WrapCGI-0.0023/t/lib/TestApp/Controller/Root.pm 2008-06-28 20:29:30.000000000 +0100 +++ Catalyst-Controller-WrapCGI-0.0023-t0m/t/lib/TestApp/Controller/Root.pm 2008-11-19 21:57:38.000000000 +0000 @@ -5,7 +5,7 @@ __PACKAGE__->config->{namespace} = ''; my $cgi = sub { - use CGI ':standard'; + use CGI 'param'; print header; print 'foo:',param('foo'),' bar:',param('bar')
On Wed Nov 19 17:05:00 2008, BOBTFISH wrote: Show quoted text
> The tests for your module fail as you import a 'meta' function into a > Catalyst::Controller package. > > The patch attached fixes the issue in the test suite, but I think you > also need to change the documentation to point out the issue, and > discourage importing into a Catalyst controller.
So that's what it was! Thank you, fix is on CPAN and in svn. -- Rafael