Only attempt to update the collections cache if there were no errors.

Related to Bug: 13338
horde · Jul 10, 2014 · 589fabd · 589fabd
1 parent 986c75c
commit 589fabd
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/framework/ActiveSync/lib/Horde/ActiveSync/Request/FolderCreate.php b/framework/ActiveSync/lib/Horde/ActiveSync/Request/FolderCreate.php
@@ -189,8 +189,10 @@ protected function _handle()
 
             $this->_encoder->endTag();
         } elseif ($update) {
-            $collections->updateFolderInHierarchy($folder, true);
-            $collections->save();
+            if ($status == self::STATUS_SUCCESS) {
+                $collections->updateFolderInHierarchy($folder, true);
+                $collections->save();
+            }
 
             $this->_encoder->startTag(self::FOLDERUPDATE);
 
@@ -204,7 +206,9 @@ protected function _handle()
 
             $this->_encoder->endTag();
         } elseif ($delete) {
-            $collections->deleteFolderFromHierarchy($server_uid);
+            if ($status == self::STATUS_SUCCESS) {
+                $collections->deleteFolderFromHierarchy($server_uid);
+            }
             $this->_encoder->startTag(self::FOLDERDELETE);
 
             $this->_encoder->startTag(Horde_ActiveSync::FOLDERHIERARCHY_STATUS);