Summary | Moving folders doesn't work |
Queue | Synchronization |
Queue Version | Git master |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | mrubinsk (at) horde (dot) org |
Requester | horde (at) albasoft (dot) com |
Created | 02/03/2015 (3860 days ago) |
Due | |
Updated | 02/05/2015 (3858 days ago) |
Assigned | 02/04/2015 (3859 days ago) |
Resolved | 02/05/2015 (3858 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | Yes |
commit c1efc1260fc3c945c80447acce2b9e39dca0d9c7
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date: Thu Feb 5 12:19:09 2015 -0500
[mjr] Fix moving a mailbox to a new parent (
Bug #13839).framework/ActiveSync/package.xml | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
http://github.com/horde/horde/commit/c1efc1260fc3c945c80447acce2b9e39dca0d9c7
commit b151116bf68ccb3160cfdd4d6a89570cc7c66960
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date: Thu Feb 5 12:16:09 2015 -0500
Bug: 13839Fix moving a mailbox via ActiveSync.Can't exit early like this since if it's only the parent that has
changed, the mailbox name will still be the same.
.../lib/Horde/ActiveSync/Imap/Adapter.php | 4 ----
1 files changed, 0 insertions(+), 4 deletions(-)
http://github.com/horde/horde/commit/b151116bf68ccb3160cfdd4d6a89570cc7c66960
Assigned to Michael Rubinsky
Priority ⇒ 1. Low
New Attachment: hordewm5-activesync_foldermoved.patch
Patch ⇒ Yes
Milestone ⇒
Summary ⇒ Moving folders doesn't work
Type ⇒ Bug
State ⇒ Unconfirmed
Queue ⇒ Synchronization
with 5.3.0 and MS Outlook 2013.
I think it's because an early return in renameMailbox function in
Horde/ActiveSync/Imap/Adapter.php
When moving a folder named "folder1" into "folders/folder2", this
function is called as
renameMailbox($old, $new, $parent = null)
renameMailbox("folder1", "folder1", "folders/folder2")
and the early return prevents from moving that folder, and nothing is changed:
if ($old == $new) { return; }
Testing after appending "parent" solves that bug (attached patch).