6.0.0-beta1
8/29/25

[#13839] Moving folders doesn't work
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

History
02/05/2015 05:19:40 PM Michael Rubinsky State ⇒ Resolved
 
02/05/2015 05:19:29 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (master):

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
02/05/2015 05:19:22 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit b151116bf68ccb3160cfdd4d6a89570cc7c66960
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Thu Feb 5 12:16:09 2015 -0500

     Bug: 13839  Fix 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
02/04/2015 04:40:35 PM Michael Rubinsky State ⇒ Assigned
Assigned to Michael Rubinsky
 
02/03/2015 08:34:20 AM horde (at) albasoft (dot) com Comment #1
Priority ⇒ 1. Low
New Attachment: hordewm5-activesync_foldermoved.patch Download
Patch ⇒ Yes
Milestone ⇒
Summary ⇒ Moving folders doesn't work
Type ⇒ Bug
State ⇒ Unconfirmed
Queue ⇒ Synchronization
Reply to this comment
When moving a folder into another, the folder is not moved. Tested 
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).

Saved Queries