Tickets :: [#10941] Broken drafts folder creation

unknown

5/18/25

Summary	Broken drafts folder creation
Queue	IMP
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	jan (at) horde (dot) org
Created	01/25/2012 (4862 days ago)
Due
Updated	08/29/2012 (4645 days ago)
Assigned	01/26/2012 (4861 days ago)
Resolved	02/02/2012 (4854 days ago)
Milestone
Patch	No

08/29/2012 12:29:36 PM	Git Commit	Comment #11	Reply to this comment
Changes have been made in Git (master): commit c7228338e68c930820993e7b4b67d0262a3e0468 Author: Jan Schneider <jan@horde.org> Date: Thu Feb 2 15:31:18 2012 +0100 The conversion is actually not required at all (~~Bug #10941~~). imp/lib/Mailbox.php \| 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) http://git.horde.org/horde-git/-/commit/c7228338e68c930820993e7b4b67d0262a3e0468

08/29/2012 12:29:33 PM	Git Commit	Comment #10	Reply to this comment
Changes have been made in Git (master): commit 2d849cc052359f9f27affbb5ff0ad21dbeb697a8 Author: Jan Schneider <jan@horde.org> Date: Thu Feb 2 12:12:25 2012 +0100 This line got lost along the way (~~Bug #10941~~). imp/lib/Mailbox.php \| 1 + 1 files changed, 1 insertions(+), 0 deletions(-) http://git.horde.org/horde-git/-/commit/2d849cc052359f9f27affbb5ff0ad21dbeb697a8

08/29/2012 12:28:29 PM	Git Commit	Comment #9	Reply to this comment
Changes have been made in Git (master): commit 7fc58d48d5049ee34c5bc1066307f4770f4966dc Author: Michael M Slusarz <slusarz@horde.org> Date: Thu Jan 26 11:49:25 2012 -0700 ~~Bug #10941~~: Fix conversion of special mailbox prefs by the upgrade task. imp/lib/LoginTasks/SystemTask/Upgrade.php \| 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/7fc58d48d5049ee34c5bc1066307f4770f4966dc

02/02/2012 02:32:09 PM	Git Commit	Comment #8	Reply to this comment
Changes have been made in Git (develop): commit c7228338e68c930820993e7b4b67d0262a3e0468 Author: Jan Schneider <jan@horde.org> Date: Thu Feb 2 15:31:18 2012 +0100 The conversion is actually not required at all (~~Bug #10941~~). imp/lib/Mailbox.php \| 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) http://git.horde.org/horde-git/-/commit/c7228338e68c930820993e7b4b67d0262a3e0468

02/02/2012 11:16:44 AM	Jan Schneider	State ⇒ Resolved Taken from Michael Slusarz Assigned to Jan Schneider

02/02/2012 11:13:21 AM	Git Commit	Comment #7	Reply to this comment
Changes have been made in Git (develop): commit 2d849cc052359f9f27affbb5ff0ad21dbeb697a8 Author: Jan Schneider <jan@horde.org> Date: Thu Feb 2 12:12:25 2012 +0100 This line got lost along the way (~~Bug #10941~~). imp/lib/Mailbox.php \| 1 + 1 files changed, 1 insertions(+), 0 deletions(-) http://git.horde.org/horde-git/-/commit/2d849cc052359f9f27affbb5ff0ad21dbeb697a8

02/02/2012 11:06:49 AM	Jan Schneider	Comment #6	Reply to this comment
In IMP_Mailbox::_getDisplay() line 1647, the mailbox label is converted to UTF7-IMAP.

01/27/2012 09:34:51 AM	Jan Schneider	Comment #5	Reply to this comment
[Show Quoted Text - 10 lines][Hide Quoted Text] - the (translated) folder label sent back is UTF7-IMAP encoded. This also happens if the drafts folder already exists. Doesn't happen in master. - the new folder is not displayed in the folder list until the folder list is reloaded manually. I could not reproduce, so likely it was resolved by fixing the first issue. I still see this. The automatically created draft folder's label is Entw&APw-rfe And it still doesn't show up in the folder list.

01/27/2012 09:33:43 AM	Jan Schneider	Comment #4	Reply to this comment
- the (translated) folder label sent back is UTF7-IMAP encoded. This also happens if the drafts folder already exists. Doesn't happen in master. - the new folder is not displayed in the folder list until the folder list is reloaded manually. I could not reproduce, so likely it was resolved by fixing the first issue. I still see this. The automatically created draft folder's label is Entw&APw-rfe

01/26/2012 06:54:27 PM	Michael Slusarz	Comment #3	Reply to this comment
commit 7fc58d48d5049ee34c5bc1066307f4770f4966dc Author: Michael M Slusarz <slusarz@horde.org> Date: Thu Jan 26 11:49:25 2012 -0700 ~~Bug #10941~~: Fix conversion of special mailbox prefs by the upgrade task. imp/lib/LoginTasks/SystemTask/Upgrade.php \| 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/7fc58d48d5049ee34c5bc1066307f4770f4966dc

01/26/2012 06:51:30 PM	Michael Slusarz	Comment #2 State ⇒ Feedback	Reply to this comment
In the develop branch (didn't test master), if the draft folder is created while saving the draft in DIMP, several things go wrong: - the folder is not the preference value, but the preference name, i.e. "drafts_folder". This was due to a broken 5.0 -> 5.1 upgrade task. This is verified to be fixed. - the (translated) folder label sent back is UTF7-IMAP encoded. This also happens if the drafts folder already exists. Doesn't happen in master. - the new folder is not displayed in the folder list until the folder list is reloaded manually. I could not reproduce, so likely it was resolved by fixing the first issue.

01/25/2012 03:08:58 PM	Jan Schneider	Comment #1 State ⇒ Assigned Patch ⇒ No Milestone ⇒ Assigned to Michael Slusarz Queue ⇒ IMP Summary ⇒ Broken drafts folder creation Type ⇒ Bug Priority ⇒ 1. Low	Reply to this comment
In the develop branch (didn't test master), if the draft folder is created while saving the draft in DIMP, several things go wrong: - the folder is not the preference value, but the preference name, i.e. "drafts_folder". - the (translated) folder label sent back is UTF7-IMAP encoded. This also happens if the drafts folder already exists. Doesn't happen in master. - the new folder is not displayed in the folder list until the folder list is reloaded manually.