| 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 | 2/3/15 (4073 days ago) |
| Due | |
| Updated | 2/5/15 (4071 days ago) |
| Assigned | 2/4/15 (4072 days ago) |
| Resolved | 2/5/15 (4071 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).