Skip Menu |

This queue is for tickets about the perl-ldap CPAN distribution.

Report information
The Basics
Id: 70708
Status: resolved
Priority: 0/
Queue: perl-ldap
People
Owner: Nobody in particular
Requestors: john [...] nixnuts.net
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: 0.42
Fixed in: 0.43

History Show all quoted text
  Sat Sep 03 13:44:01 2011 john [...] nixnuts.net - Ticket created
Subject: warnings in Net/LDAP/Message.pm
I can't check cpan testers at the moment, but on my machines (5.8.8 and 5.14.1), the Net::LDAP unit tests all fail because of some valid warnings in Net/LDAP/Message.pm sub code { $self->{resultcode} || LDAP_SUCCESS } sub error { $self->{errorMessage} || "" } should probably be: sub code { shift->{resultcode} || LDAP_SUCCESS } sub error { shift->{errorMessage} || "" }
  Sat Sep 03 13:51:41 2011 john [...] nixnuts.net - Correspondence added
Subject: Re: [rt.cpan.org #70708] AutoReply: warnings in Net/LDAP/Message.pm
Date: Sat, 03 Sep 2011 12:54:46 -0500
To: bug-perl-ldap [...] rt.cpan.org
From: John Lightsey <john [...] nixnuts.net>
On 09/03/2011 12:44 PM, Bugs in perl-ldap via RT wrote: Show quoted text
> should probably be: > > sub code { shift->{resultcode} || LDAP_SUCCESS } > sub error { shift->{errorMessage} || "" } >
Sorry, this still generates warnings from LDAP_SUCCESS not being imported to the package.. It should be: sub code { shift->{resultcode} || Net::LDAP::Constant::LDAP_SUCCESS } sub error { shift->{errorMessage} || "" }
  Sat Sep 03 13:52:46 2011 gbarr [...] pobox.com - Correspondence added
Subject: Re: [rt.cpan.org #70708] AutoReply: warnings in Net/LDAP/Message.pm
Date: Sat, 3 Sep 2011 12:52:32 -0500
To: bug-perl-ldap [...] rt.cpan.org
From: Graham Barr <gbarr [...] pobox.com>
It should also bee resultCode not resultcode. pushing 0.43 now On Sep 3, 2011, at 12:51 , John D. Lightsey via RT wrote: Show quoted text
> Queue: perl-ldap > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=70708 > > > On 09/03/2011 12:44 PM, Bugs in perl-ldap via RT wrote:
>> should probably be: >> >> sub code { shift->{resultcode} || LDAP_SUCCESS } >> sub error { shift->{errorMessage} || "" } >>
> > Sorry, this still generates warnings from LDAP_SUCCESS not being > imported to the package.. > > It should be: > > sub code { shift->{resultcode} || Net::LDAP::Constant::LDAP_SUCCESS } > sub error { shift->{errorMessage} || "" } >
  Sat Sep 03 13:52:47 2011 The RT System itself - Status changed from 'new' to 'open'
  Sun Jan 29 14:15:47 2012 MARSCHAP [...] cpan.org - Fixed in 0.43 added
  Thu Feb 09 10:19:05 2012 MARSCHAP [...] cpan.org - Status changed from 'open' to 'resolved'