TrueCloudLab/rclone
16
0
Fork 2
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

Revert "drive: server side copy docs use default description if empty"

Browse source 
This reverts commit 9e4b68a364.

This does not work as intended - it only changes docs files and to
make it change drive files would take an extra roundtrip.

I think the sematics of server side copy are now correct - additional
features should be added with a new flag.

See #4230
This commit is contained in:
Nick Craig-Wood 2020-05-19 16:48:02 +01:00
parent 4d431e94b9
commit cf5d0f5c1f
1 changed files with 1 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
backend/drive/drive.go
View file

@ -2244,10 +2244,7 @@ func (f *Fs) Copy(ctx context.Context, src fs.Object, remote string) (fs.Object,
if err != nil { if err != nil {
return nil, errors.Wrap(err, "failed to read description for Google Doc") return nil, errors.Wrap(err, "failed to read description for Google Doc")
} }
// set the description if there is one, or use the default if not createInfo.Description = info.Description
if info.Description != "" {
createInfo.Description = info.Description
}
} else { } else {
// don't overwrite the description on copy for files // don't overwrite the description on copy for files
// this should work for docs but it doesn't - it is probably a bug in Google Drive // this should work for docs but it doesn't - it is probably a bug in Google Drive