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

restoring the first of multiple items with the same name from the trashbin shows the wrong item remaining in the trashbin #1723

Closed
individual-it opened this issue Aug 8, 2019 · 5 comments
Assignees
Labels
Milestone

Comments

@individual-it
Copy link
Member

similar to #1385

Steps to reproduce

  1. create a file a1.txt
  2. delete the file
  3. create a folder a1.txt
  4. delete the folder
  5. navigate to DELETED FILES
  6. restore the file a1.txt (should be first in list)

Expected behaviour

the file should not be listed in the trashbin

Actual behaviour

the folder is not listed in the trashbin, the file is listed in the trashbin
the file was restored, so that is correct

@individual-it individual-it added the Type:Bug Something isn't working label Aug 8, 2019
@DeepDiver1975 DeepDiver1975 added the Priority:p4-low Low priority label Aug 16, 2019
@DeepDiver1975 DeepDiver1975 self-assigned this Aug 16, 2019
@LukasHirt LukasHirt added this to the backlog milestone Aug 23, 2019
@LukasHirt
Copy link
Collaborator

Tested now with Phoenix version Version: 0.2.7-9fa84171 (1579092217051) and works fine. Can you pls retest? @individual-it

@individual-it individual-it added dev:acceptance-tests QA:team and removed Type:Bug Something isn't working labels Jan 20, 2020
@individual-it
Copy link
Member Author

seems to work, need to add tests to prevent future regressions,
added QA-team label and added to the sprint

@swoichha
Copy link
Contributor

After restoring the file a1.txt, when the user tries to restore the folder a1.txt, "restoration failed" error is shown.

@individual-it
Copy link
Member Author

@swoichha that is covered in #1730

@individual-it
Copy link
Member Author

tests done, closing issue

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

No branches or pull requests

4 participants