Skip Menu |

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

Report information
The Basics
Id: 79476
Status: resolved
Priority: 0/
Queue: Mail-IMAPClient
People
Owner: PLOBBES [...] cpan.org
Requestors: oleg [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 3.00
Fixed in: (no value)

History Show all quoted text
  Thu Sep 06 06:23:52 2012 oleg [...] cpan.org - Ticket created
Subject: move() with "1:50" sequence produces numeric warning
$imap = Mail::IMAPClient->new( ... Uid => 0 ); ... $imap->move($to, "1:$cnt"); Code above produces following warning message: Argument "1:22" isn't numeric in sort at /usr/local/share/perl/5.14.2/Mail/IMAPClient.pm line 3221 But messages moves successfully. Is it correct to use such sequence with move() method?
  Thu Sep 06 09:21:46 2012 PLOBBES [...] cpan.org - Correspondence added
On Thu Sep 06 06:23:52 2012, OLEG wrote: Show quoted text
> ... Is it correct to use such sequence with move() method?
That usage is perfectly valid. There is some legacy code that's probably trying to do and I can't see any reason for it. Method move() calls copy() which tries to sort the message(s) provided as arguments. However, the order provided by the caller is important as it relates to the order of the returned UID values. Look for a fix in the next release, probably by removing the sort completely.
  Thu Sep 06 09:21:48 2012 The RT System itself - Status changed from 'new' to 'open'
  Thu Sep 06 09:21:55 2012 PLOBBES [...] cpan.org - Taken
  Thu Sep 06 10:01:23 2012 PLOBBES [...] cpan.org - Broken in 3.00 added
  Thu Sep 06 10:01:23 2012 PLOBBES [...] cpan.org - Broken in 3.32 deleted
  Thu Sep 06 10:01:23 2012 PLOBBES [...] cpan.org - Correspondence added
I have patched the source with fix/workaround for this bug. This will be included in the next version release (v3.33). You are welcome to pull the latest code from git if you're interested in seeing what I have checked in and want to test: http://sourceforge.net/p/mail-imapclient/git/ Thanks for taking the time to report this bug!
  Thu Sep 06 10:01:25 2012 PLOBBES [...] cpan.org - Status changed from 'open' to 'patched'
  Fri Sep 07 00:33:06 2012 oleg [...] cpan.org - Correspondence added
Чтв Сен 06 10:01:23 2012, PLOBBES писал: Show quoted text
> I have patched the source with fix/workaround for this bug. This > will be included in the next version release (v3.33). > > You are welcome to pull the latest code from git if you're > interested in seeing what I have checked in and want to test: > http://sourceforge.net/p/mail-imapclient/git/ > > Thanks for taking the time to report this bug!
No warnings now. Thanks!
  Fri Sep 07 00:33:07 2012 The RT System itself - Status changed from 'patched' to 'open'
  Fri Sep 07 00:33:07 2012 oleg [...] cpan.org - Status changed from 'open' to 'resolved'