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

Expanded folder operations (Move/Copy) #957

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

DerfJagged
Copy link

This PR adds recursive Copy and Move operations for folders and adds "merge" functionality if a file/folder is renamed to an existing name.

Handles these edge cases:

  • Folder already exists upon folder copy/move/rename.
    • Asks user if overwrite is OK. Applies to all subfolders and files for this operation.
    • Recursively copies files and folders into existing structure.
    • Shows progress bar for every file.
  • File already exists upon file rename.
    • Asks user if overwrite is OK.
    • Deletes destination file and renames source file.
  • Folder copy/move to same location attempted.
    • Notify user that folder can't be copied over self and stop.
  • Folder copy/move to inside itself attempted.
    • Notify user that folder can't be copied inside itself and stop.

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

Successfully merging this pull request may close these issues.

1 participant