Merge branch 'staging' of github.com:xwiki-labs/cryptpad into staging

www/common/userObject.js

@@ -460,12 +460,6 @@ define([
             var element = find(elementPath);
             var newParent = find(newParentPath);
 
-            // Never move a folder in one of its children
-            if (isSubpath(newParentPath, elementPath)) {
-                log(Messages.fo_moveFolderToChildError);
-                return;
-            }
-
             // Move to Trash
             if (isPathIn(newParentPath, [TRASH])) {
                 if (!elementPath || elementPath.length < 2 || elementPath[0] === TRASH) {
@@ -518,9 +512,15 @@ define([
             paths.forEach(function (p) {
                 var parentPath = p.slice();
                 parentPath.pop();
-                if (comparePath(parentPath, newPath)) { return; }
+                if (comparePath(parentPath, newPath)) { return; }
-                copyElement(p, newPath);
+                if (isSubpath(newPath, p)) {
-                toRemove.push(p);
+                    log(Messages.fo_moveFolderToChildError);
+                    return;
+                }
+                // Try to copy, and if success, remove the element from the old location
+                if (copyElement(p, newPath)) {
+                    toRemove.push(p);
+                }
             });
             exp.delete(toRemove, cb);
         };