From 40b0e66efec91b08b3ea09fdaff8943ae0bdbb5f Mon Sep 17 00:00:00 2001
From: KirCute_ECT <951206789@qq.com>
Date: Wed, 25 Dec 2024 21:12:30 +0800
Subject: [PATCH] feat(ftp-server): treat moving across file systems as copying
 (#7704 close #7701)

* feat(ftp-server): treat moving across file systems as copying

* fix: ensure compatibility across different fs on the same driver
---
 server/ftp/fsmanage.go | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/server/ftp/fsmanage.go b/server/ftp/fsmanage.go
index 5199a473..fb03c1b9 100644
--- a/server/ftp/fsmanage.go
+++ b/server/ftp/fsmanage.go
@@ -2,6 +2,7 @@ package ftp
 
 import (
 	"context"
+	"fmt"
 	"github.com/alist-org/alist/v3/internal/errs"
 	"github.com/alist-org/alist/v3/internal/fs"
 	"github.com/alist-org/alist/v3/internal/model"
@@ -64,8 +65,14 @@ func Rename(ctx context.Context, oldPath, newPath string) error {
 		if !user.CanFTPManage() || !user.CanMove() || (srcBase != dstBase && !user.CanRename()) {
 			return errs.PermissionDenied
 		}
-		if err := fs.Move(ctx, srcPath, dstDir); err != nil {
-			return err
+		if err = fs.Move(ctx, srcPath, dstDir); err != nil {
+			if srcBase != dstBase {
+				return err
+			}
+			if _, err1 := fs.Copy(ctx, srcPath, dstDir); err1 != nil {
+				return fmt.Errorf("failed move for %+v, and failed try copying for %+v", err, err1)
+			}
+			return nil
 		}
 		if srcBase != dstBase {
 			return fs.Rename(ctx, stdpath.Join(dstDir, srcBase), dstBase)