Our code: right idea, wrong execution.  We just needed to do the rtrim 
on the whole string, not just the part we were adding.  So slightly 
modified your patch to do this and committed to IMP 4.2.1.

Changes have been made in CVS for this ticket:

   
http://cvs.horde.org/diff.php/imp/docs/CHANGES?r1=1.699.2.308&r2=1.699.2.309&ty=u
   
http://cvs.horde.org/diff.php/imp/lib/IMAP/Tree.php?r1=1.25.2.49&r2=1.25.2.50&ty=u

Changes have been made in CVS for this ticket:

   http://cvs.horde.org/diff.php/imp/docs/CHANGES?r1=1.1136&r2=1.1137&ty=u
   http://cvs.horde.org/diff.php/imp/lib/IMAP/Tree.php?r1=1.185&r2=1.186&ty=u

I found the source of the issue. IMP tried to create folders

below "INBOX" like this: "INBOX//new_folder".



The error occurs in imp/lib/IMAP/Tree.php::createMailboxName($parent, $new).

Here's a dump of the $ns_info structure:



Array

(

     [name] => INBOX/

     [delimiter] => /

     [type] => personal

     [hidden] =>

)



$parent is set to "INBOX", $new is "new_folder".



Here's the code that computes the folder "base":



     $mbox = $ns_info['name']; --> mbox is "INBOX/"

     $mbox .= rtrim(substr_replace($parent, '', 0, 
strlen($ns_info['name'])), $ns_info['delimiter']) . 
$ns_info['delimiter'];



substr_replace($parent, '', 0, strlen($ns_info['name'])) will lead to 
an empty string.

rtim() is a noop in this case.



Then we append the delimiter, so this results in "INBOX//".



I've developed a fix, though I don't know

if "name -> INBOX/" is right in the first place.

Hello together,



I'm currently seeing a strange issue when I try to create an IMAP 
subfolder in IMP. When I check the "INBOX" and select "Create folder", 
the popup appears. I enter "subfolder" as folder name. The creation of 
the new folder fails as it tries to create the folder "/subfolder".



When I do the same thing without checking "INBOX", so it defaults to 
INBOX, my subfolder INBOX/subfolder gets created fine.



Anyone else seeing this?



Thomas