Bug #66287 for Mail-IMAPClient: flags / Maxcommandlength

Tue Mar 01 09:13:46 2011 eco [...] ecocode.net - Ticket created

Subject:

flags / Maxcommandlength

I suspect that when Maxcommandlength splits a query, only the last chunk of requested flags is saved in the returned hash. -- Erik

Tue Mar 01 10:43:59 2011 PLOBBES [...] cpan.org - Correspondence added

Can you provide any details to support your hunch? What method are you calling and with what arguments? Bugs certainly may exist, but it will be hard to fix them without some supporting details to help isolate the problem you believe you are seeing.

Tue Mar 01 10:44:00 2011 The RT System itself - Status changed from 'new' to 'open'

Tue Mar 01 10:44:00 2011 PLOBBES [...] cpan.org - Status changed from 'open' to 'stalled'

Tue Mar 01 10:44:00 2011 PLOBBES [...] cpan.org - Given to PLOBBES

Tue Mar 01 10:46:21 2011 PLOBBES [...] cpan.org - Correspondence added

Any chance this is when calling flags() with a large list of messages? If so, then I think this seems plausible.

Tue Mar 01 10:46:22 2011 The RT System itself - Status changed from 'stalled' to 'open'

Tue Mar 01 11:20:28 2011 PLOBBES [...] cpan.org - Correspondence added

Assuming this is just against flags() then can you try the following patch? diff --git a/lib/Mail/IMAPClient.pm b/lib/Mail/IMAPClient.pm index d3a034a..8897728 100644 --- a/lib/Mail/IMAPClient.pm +++ b/lib/Mail/IMAPClient.pm @@ -2207,13 +2207,13 @@ sub flags { $msg->cat(@_) if @_; # Send command - $self->fetch( $msg, "FLAGS" ) or return undef; + my $ref = $self->fetch( $msg, "FLAGS" ) or return undef; my $u_f = $self->Uid; my $flagset = {}; # Parse results, setting entry in result hash for each line - foreach my $line ( $self->Results ) { + foreach my $line ( @$ref ) { $self->_debug("flags: line = '$line'"); if ( $line =~ /\* \s+ (\d+) \s+ FETCH \s+ # * nnn FETCH

Tue Mar 01 11:20:28 2011 PLOBBES [...] cpan.org - Status changed from 'open' to 'patched'

Sat Mar 05 00:14:09 2011 PLOBBES [...] cpan.org - Fixed in 3.28 added

Sat Mar 05 00:14:09 2011 PLOBBES [...] cpan.org - Broken in 3.18 added

Sat Mar 05 00:14:09 2011 PLOBBES [...] cpan.org - Broken in 3.27 deleted

Sat Mar 05 00:14:09 2011 PLOBBES [...] cpan.org - Correspondence added

Mail::IMAPClient 3.28 was released today. If you still see any problems after upgrading, please let me know!

Sat Mar 05 00:14:10 2011 The RT System itself - Status changed from 'patched' to 'open'

Sat Mar 05 00:14:10 2011 PLOBBES [...] cpan.org - Status changed from 'open' to 'resolved'