From dd6a47c24a5546dc40c083700af16efcfe6154cf Mon Sep 17 00:00:00 2001
From: alecpl <alec@alec.pl>
Date: Fri, 19 Aug 2011 08:07:05 +0000
Subject: - Fix parent folder ACL checking in some cases

---
 program/steps/settings/save_folder.inc | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

(limited to 'program/steps')

diff --git a/program/steps/settings/save_folder.inc b/program/steps/settings/save_folder.inc
index 2f515627d..48c957239 100644
--- a/program/steps/settings/save_folder.inc
+++ b/program/steps/settings/save_folder.inc
@@ -55,16 +55,6 @@ else {
     }
 }
 
-// Check access rights to the parent folder
-if (!$error && strlen($path)) {
-    $parent_opts = $RCMAIL->imap->mailbox_info($path);
-    if ($parent_opts['namespace'] != 'personal'
-        && (empty($parent_opts['rights']) || !preg_match('/[ck]/', implode($parent_opts)))
-    ) {
-        $error = rcube_label('parentnotwritable');
-    }
-}
-
 if ($error) {
     $OUTPUT->command('display_message', $error, 'error');
 }
@@ -78,7 +68,19 @@ else {
     else {
         $name_imap = $RCMAIL->imap->mod_mailbox($name_imap, 'in');
     }
+}
+
+// Check access rights to the parent folder
+if (!$error && strlen($path) && (!strlen($old_imap) || $old_imap != $name_imap)) {
+    $parent_opts = $RCMAIL->imap->mailbox_info($path);
+    if ($parent_opts['namespace'] != 'personal'
+        && (empty($parent_opts['rights']) || !preg_match('/[ck]/', implode($parent_opts)))
+    ) {
+        $error = rcube_label('parentnotwritable');
+    }
+}
 
+if (!$error) {
     $folder['name']     = $name_imap;
     $folder['oldname']  = $old_imap;
     $folder['class']    = '';
-- 
cgit v1.2.3