Skip Menu |
 

This queue is for tickets about the CGI-Application-Plugin-DevPopup CPAN distribution.

Report information
The Basics
Id: 53100
Status: resolved
Priority: 0/
Queue: CGI-Application-Plugin-DevPopup

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

Bug Information
Severity: Normal
Broken in: 1.03
Fixed in: (no value)



Subject: using CGI::Application::Plugin::DevPopup::Log produces a warning
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Type: text/plain
Charset: utf8
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 835
Download (untitled) / with headers
text/plain 835b
Hello Rhesa, I've encountered a bug when using CGI::Application::Plugin::DevPopup::Log. On every call, a warning message is produced. Exact message depends of version of Log::Dispatch used. There can be one of following: Use of uninitialized value in concatenation (.) or string at C:/bin/dev/perl/site/lib/Params/ValidatePP.pm line 344 Use of uninitialized value in subroutine entry at C:/bin/dev/perl/site/lib/Log/Dispatch/Handle.pm line 20. Maybe some others too. I take some research and found that it appears when Log::Dispatch::Handle->new is called with IO::Scalar->new with the reference to undefined value. So, to get rid of this warning, $data in line 58 of DevPopup/Log.pm should be defined. Actually, fix is easy, line 58 should be changed this way: < my $data; Show quoted text
> my $data = '';
-- Serguei Trouchelle
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Charset: utf8
Content-Type: text/plain
Message-ID: <rt-3.6.HEAD-22627-1261770193-1635.53100-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 59
Thanks Serguei, a new version is on its way to CPAN. Rhesa


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.