Skip Menu |

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

Report information
The Basics
Id: 36458
Status: resolved
Priority: 0/
Queue: Mail-Box

People
Owner: Nobody in particular
Requestors: jik [...] kamens.brookline.ma.us
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in:
  • 2.081
  • 2.082
Fixed in: (no value)



Listing imap folders doesn't work due to incorrect invocation of split. Patch is attached.
Subject: Mail-Box-2.082-imap-folders.patch
--- lib/Mail/Transport/IMAP4.pm.orig 2008-06-04 21:49:30.000000000 -0400 +++ lib/Mail/Transport/IMAP4.pm 2008-06-04 21:49:36.000000000 -0400 @@ -275,7 +275,7 @@ # There may be multiples thanks to subdirs so we uniq it my %uniq; - $uniq{(split($sep, $_))[$level]||''}++ foreach @folders; + $uniq{(split(/\Q$sep\E/, $_))[$level]||''}++ foreach @folders; delete $uniq{''}; keys %uniq;
Subject: Re: [rt.cpan.org #36458]
Date: Fri, 6 Jun 2008 11:17:39 +0200
To: Jonathan Kamens via RT <bug-Mail-Box [...] rt.cpan.org>
From: Mark Overmeer <mark [...] overmeer.net>
* Jonathan Kamens via RT (bug-Mail-Box@rt.cpan.org) [080605 01:53]: Show quoted text
> Wed Jun 04 21:53:29 2008: Request 36458 was acted upon. > Transaction: Ticket created by JIK > Queue: Mail-Box > Subject: (No subject given) > > Listing imap folders doesn't work due to incorrect invocation of split. > Patch is attached.
Patch accepted. Thanks. Since zillions of bugs got fixed in Mail::IMAPClient, it may be wise to study for more improvements in the Mail::Transport::IMAP4 implementation. I do not have the time for it. -- Regards, MarkOv ------------------------------------------------------------------------ Mark Overmeer MSc MARKOV Solutions Mark@Overmeer.net solutions@overmeer.net http://Mark.Overmeer.net http://solutions.overmeer.net
Subject: Mail::Box
2.083 will be released today, and contains your patch