Fixed the 'moveTo' method of the 'PhysicalResource'

Author: AdrienCastex

lib/resource/physical/PhysicalResource.js

@@ -24,12 +24,34 @@ var PhysicalResource = (function (_super) {
             else
                 fsManager = new PhysicalFSManager_1.PhysicalFSManager();
         _this = _super.call(this, parent, fsManager) || this;
+        _this.deleteOnMoved = false;
         _this.realPath = path.resolve(realPath);
         _this.name = path.basename(_this.realPath);
         return _this;
     }
     PhysicalResource.prototype.moveTo = function (parent, newName, overwrite, callback) {
-        StandardResource_1.StandardResource.standardMoveTo(this, parent, newName, overwrite, callback);
+        var _this = this;
+        var pRealPath = parent.realPath;
+        if (!(parent.fsManager && this.fsManager && parent.fsManager.uid === this.fsManager.uid && pRealPath)) {
+            StandardResource_1.StandardResource.standardMoveByCopy(this, parent, newName, overwrite, this.deleteOnMoved, callback);
+            return;
+        }
+        var newRealPath = path.join(pRealPath, newName);
+        fs.rename(this.realPath, newRealPath, function (e) {
+            if (e) {
+                callback(e);
+                return;
+            }
+            _this.realPath = newRealPath;
+            _this.name = path.basename(_this.realPath);
+            _this.removeFromParent(function (e) {
+                if (e) {
+                    callback(e);
+                    return;
+                }
+                _this.addToParent(parent, callback);
+            });
+        });
     };
     PhysicalResource.prototype.rename = function (newName, callback) {
         var _this = this;

src/resource/physical/PhysicalResource.ts

@@ -21,6 +21,7 @@ export abstract class PhysicalResource extends StandardResource
 
         super(parent, fsManager);
 
+        this.deleteOnMoved = false;
         this.realPath = path.resolve(realPath);
         this.name = path.basename(this.realPath);
     }
@@ -30,7 +31,34 @@ export abstract class PhysicalResource extends StandardResource
     abstract delete(callback : SimpleCallback)
     moveTo(parent : IResource, newName : string, overwrite : boolean, callback : SimpleCallback)
     {
-        StandardResource.standardMoveTo(this, parent, newName, overwrite, callback);
+        const pRealPath = (parent as any).realPath;
+        if(!(parent.fsManager && this.fsManager && parent.fsManager.uid === this.fsManager.uid && pRealPath))
+        {
+            StandardResource.standardMoveByCopy(this, parent, newName, overwrite, this.deleteOnMoved, callback);
+            return;
+        }
+        
+        const newRealPath = path.join(pRealPath, newName);
+        fs.rename(this.realPath, newRealPath, (e) => {
+            if(e)
+            {
+                callback(e);
+                return;
+            }
+
+            this.realPath = newRealPath;
+            this.name = path.basename(this.realPath);
+
+            this.removeFromParent((e) => {
+                if(e)
+                {
+                    callback(e);
+                    return;
+                }
+
+                this.addToParent(parent, callback);
+            })
+        })
     }
     rename(newName : string, callback : Return2Callback<string, string>)
     {