From d97e33fc882695bd06a6352693acdb0c78b0e52b Mon Sep 17 00:00:00 2001
From: Nick Craig-Wood <nick@craig-wood.com>
Date: Mon, 27 Jul 2026 12:27:02 +0100
Subject: [PATCH] Revert "drime: disable server side copy as it always fails"

This reverts commit 961266888fe797390c535386f3b3aa46f4853602.

This has been fixed on the server side.
---
 backend/drime/drime.go        | 111 ++++++++++++++++++++++++++++++++++
 docs/data/backends/drime.yaml |   1 +
 2 files changed, 112 insertions(+)

diff --git a/backend/drime/drime.go b/backend/drime/drime.go
index 3b3978b13..2499627f6 100644
--- a/backend/drime/drime.go
+++ b/backend/drime/drime.go
@@ -420,6 +420,14 @@ func NewFs(ctx context.Context, name, root string, m configmap.Mapper) (fs.Fs, e
 	return f, nil
 }
 
+// rootSlash returns root with a slash on if it is empty, otherwise empty string
+func (f *Fs) rootSlash() string {
+	if f.root == "" {
+		return f.root
+	}
+	return f.root + "/"
+}
+
 // Return an Object from a path
 //
 // If it can't be found it returns the error fs.ErrorObjectNotFound.
@@ -1004,6 +1012,108 @@ func (f *Fs) DirMove(ctx context.Context, src fs.Fs, srcRemote, dstRemote string
 	return nil
 }
 
+// copy a file or a folder to a new directory
+func (f *Fs) copy(ctx context.Context, id, newDirID string) (item *api.Item, err error) {
+	var resp *http.Response
+	var request = api.CopyRequest{
+		EntryIDs:      []string{id},
+		DestinationID: newDirID,
+	}
+	var result api.CopyResponse
+	opts := rest.Opts{
+		Method:     "POST",
+		Path:       "/file-entries/duplicate",
+		Parameters: url.Values{},
+	}
+	if f.opt.WorkspaceID != "" {
+		opts.Parameters.Set("workspaceId", f.opt.WorkspaceID)
+	}
+	err = f.pacer.Call(func() (bool, error) {
+		resp, err = f.srv.CallJSON(ctx, &opts, &request, &result)
+		return shouldRetry(ctx, resp, err)
+	})
+	if err != nil {
+		return nil, fmt.Errorf("failed to copy item: %w", err)
+	}
+	itemResult := result.Entries[0]
+	return &itemResult, nil
+}
+
+// copy and rename a file or folder to directoryID with leaf
+func (f *Fs) copyTo(ctx context.Context, srcID, srcLeaf, dstLeaf, dstDirectoryID string) (info *api.Item, err error) {
+	// Can have duplicates so don't have to be careful here
+
+	// Copy to dstDirectoryID first
+	info, err = f.copy(ctx, srcID, dstDirectoryID)
+	if err != nil {
+		return nil, err
+	}
+
+	// Rename if required
+	if srcLeaf != dstLeaf {
+		info, err = f.rename(ctx, info.ID.String(), dstLeaf)
+		if err != nil {
+			return nil, err
+		}
+	}
+	return info, nil
+}
+
+// Copy src to this remote using server-side copy operations.
+//
+// This is stored with the remote path given.
+//
+// It returns the destination Object and a possible error.
+//
+// Will only be called if src.Fs().Name() == f.Name()
+//
+// If it isn't possible then return fs.ErrorCantCopy
+func (f *Fs) Copy(ctx context.Context, src fs.Object, remote string) (dst fs.Object, err error) {
+	srcObj, ok := src.(*Object)
+	if !ok {
+		fs.Debugf(src, "Can't copy - not same remote type")
+		return nil, fs.ErrorCantCopy
+	}
+	srcLeaf := path.Base(srcObj.remote)
+
+	srcPath := srcObj.fs.rootSlash() + srcObj.remote
+	dstPath := f.rootSlash() + remote
+	if srcPath == dstPath {
+		return nil, fmt.Errorf("can't copy %q -> %q as are same name", srcPath, dstPath)
+	}
+
+	// Find existing object
+	existingObj, err := f.NewObject(ctx, remote)
+	if err == nil {
+		defer func() {
+			// Don't remove existing object if returning an error
+			if err != nil {
+				return
+			}
+			fs.Debugf(existingObj, "Server side copy: removing existing object after successful copy")
+			err = existingObj.Remove(ctx)
+		}()
+	}
+
+	// Create temporary object
+	dstObj, dstLeaf, dstDirectoryID, err := f.createObject(ctx, remote, srcObj.modTime, srcObj.size)
+	if err != nil {
+		return nil, err
+	}
+
+	// Copy the object
+	info, err := f.copyTo(ctx, srcObj.id, srcLeaf, dstLeaf, dstDirectoryID)
+	if err != nil {
+		return nil, err
+	}
+	err = dstObj.setMetaData(info)
+	if err != nil {
+		return nil, err
+	}
+
+	return dstObj, nil
+}
+
 // DirCacheFlush resets the directory cache - used in testing as an
 // optional interface
 func (f *Fs) DirCacheFlush() {
@@ -1562,6 +1672,7 @@ var (
 	_ fs.Fs              = (*Fs)(nil)
 	_ fs.Purger          = (*Fs)(nil)
 	_ fs.PutStreamer     = (*Fs)(nil)
+	_ fs.Copier          = (*Fs)(nil)
 	_ fs.Mover           = (*Fs)(nil)
 	_ fs.DirMover        = (*Fs)(nil)
 	_ fs.DirCacheFlusher = (*Fs)(nil)
diff --git a/docs/data/backends/drime.yaml b/docs/data/backends/drime.yaml
index 546e54a50..fa5c40997 100644
--- a/docs/data/backends/drime.yaml
+++ b/docs/data/backends/drime.yaml
@@ -14,6 +14,7 @@ remote: 'TestDrime:'
 features:
 - About
 - CanHaveEmptyDirectories
+- Copy
 - DirCacheFlush
 - DirMove
 - Move