Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Mail-Audit CPAN distribution.

Report information
The Basics
Id: 20934
Status: resolved
Priority: 0/
Queue: Mail-Audit
People
Owner: rjbs [...] cpan.org
Requestors: p.d.oliver [...] mavit.freeserve.co.uk
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Thu Aug 10 09:56:28 2006 p.d.oliver [...] mavit.freeserve.co.uk - Ticket created
Date: Thu, 10 Aug 2006 14:56:04 +0100 (BST)
To: bug-Mail-ListDetector [...] rt.cpan.org
From: Peter Oliver <p.d.oliver [...] mavit.freeserve.co.uk>
The internals of recent versions of Mail::Audit have changed such that Mail::ListDetector can no longer find the underlying Mail::Internet object. Here's a patch: --- List.pm~ 2006-07-16 21:16:31.000000000 +0100 +++ List.pm 2006-08-10 14:50:03.000000000 +0100 @@ -16,7 +16,9 @@ $dir ||= $DEFAULT_DIR; $arg ||= {}; - my $list = Mail::ListDetector->new($self->{obj}); + my $list = Mail::ListDetector->new( $Mail::Audit::VERSION <= 2.0 + ? $self->{obj} + : $self ); if (!(defined $list)) { return 0; -- Peter Oliver
  Sun Aug 13 12:26:35 2006 MSTEVENS [...] cpan.org - Correspondence added
From: MSTEVENS [...] cpan.org
List.pm is part of Mail::Audit and the patch will need to be applied there. I'll see if I can move the bug between queues, otherwise please refile. On Thu Aug 10 09:56:28 2006, p.d.oliver@mavit.freeserve.co.uk wrote: Show quoted text
> The internals of recent versions of Mail::Audit have changed such that > Mail::ListDetector can no longer find the underlying Mail::Internet > object. Here's a patch: > > --- List.pm~ 2006-07-16 21:16:31.000000000 +0100 > +++ List.pm 2006-08-10 14:50:03.000000000 +0100 > @@ -16,7 +16,9 @@ > $dir ||= $DEFAULT_DIR; > $arg ||= {}; > > - my $list = Mail::ListDetector->new($self->{obj}); > + my $list = Mail::ListDetector->new( $Mail::Audit::VERSION <= 2.0 > + ? $self->{obj} > + : $self ); > > if (!(defined $list)) { > return 0; >
  Sun Aug 13 12:26:36 2006 The RT System itself - Status changed from 'new' to 'open'
  Sun Aug 13 12:27:19 2006 MSTEVENS [...] cpan.org - Status changed from 'open' to 'new'
  Sun Aug 13 12:27:19 2006 MSTEVENS [...] cpan.org - Queue changed from Mail-ListDetector to Mail-Audit
  Tue Sep 19 07:43:45 2006 rjbs [...] cpan.org - Correspondence added
Sorry for the delay; I don't think RT notified me when this ticket was transferred in! I've updated the code; I don't need to check version, since Mail::Audit::List is bundled with Mail::Audit. If you have a newer version of one, you'll have the newer version of the other. List.pm is not tested; patches more than welcome! -- rjbs
  Tue Sep 19 07:43:52 2006 The RT System itself - Status changed from 'new' to 'open'
  Tue Sep 19 07:43:59 2006 rjbs [...] cpan.org - Status changed from 'open' to 'resolved'
  Tue Sep 19 07:44:13 2006 rjbs [...] cpan.org - Given to RJBS