Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copy/Move to same folder #3708

Closed
jesmrec opened this issue Jul 5, 2022 · 2 comments · Fixed by #3748
Closed

Copy/Move to same folder #3708

jesmrec opened this issue Jul 5, 2022 · 2 comments · Fixed by #3748

Comments

@jesmrec
Copy link
Collaborator

jesmrec commented Jul 5, 2022

In the following scenario, Android app behaves in a very different way comparing with other clients:

  1. Select any item of the list
  2. Select Copy or Move actions
  3. Select as target folder the current folder

Current behaviour across the platform:

Android:

  • Copy to same folder -> create a duplicate with (2)
  • Move to same folder -> rename to (2)

iOS:

  • Copy to same folder -> not allowed
  • Move to same folder -> not allowed

Web (classic):

  • Copy to same folder -> not allowed (by drag and drop)
  • Move to same folder -> NA

Web (oCIS):

  • Copy to same folder -> not allowed (error message)
  • Move to same folder -> not allowed (error message)

Desktop client:

  • Copy to same folder -> duplicate with copy (like any other file in the OS)
  • Move to same folder -> NA

I'd go for the "not allowed" and prevent copy/move to the current folder

Question to solve/discuss: expected behaviour for copying/moving in document provider

@jesmrec jesmrec changed the title Copy/Move behaviour Copy/Move to same folder Jul 5, 2022
@jesmrec
Copy link
Collaborator Author

jesmrec commented Jul 7, 2022

Checked also this related scenario:

  1. Select item
  2. Copy / Move
  3. Select target folder that contains other item with same file/folder name

In iOS and web, it is prevented as well.

@jesmrec
Copy link
Collaborator Author

jesmrec commented Aug 26, 2022

Will follow the oCIS behaviour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants