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

[QA] Trashbin cannot delete a deep tree #1077

Closed
jnweiger opened this issue Dec 11, 2020 · 7 comments
Closed

[QA] Trashbin cannot delete a deep tree #1077

jnweiger opened this issue Dec 11, 2020 · 7 comments
Labels
Category:Defect Existing functionality is not working as expected Type:Bug

Comments

@jnweiger
Copy link
Contributor

Deployed ocis master 2020-12-11 13:30h via hetzner-deploy/make_ocis_test.sh -> ocis/deployments/examples/ocis_traefik/

  • log in as einstein via web ui,
  • upload a deep tree with 1000 files ca 900MB (from randfiles.py)
  • delete the top folder
  • visit the trashcan
  • try delete the deep tree. It fails. BAD
    image

Expected behaviour: delete works. such a tree is very hard to delete piecewise.

Restore works fine, though.
1211c.log.zip

@individual-it
Copy link
Member

individual-it commented Dec 16, 2020

@individual-it individual-it self-assigned this Dec 16, 2020
@individual-it
Copy link
Member

@jnweiger is that about the deep tree, or just about having a file in the folder, seems like I cannot delete a folder that contains a file from the trashbin

@individual-it
Copy link
Member

here the issue about deleting nested folders from trashbin owncloud/product#188

@individual-it
Copy link
Member

maybe related #1116

@jnweiger
Copy link
Contributor Author

Reproduced on ocis-1-0-0-rc8 with a small tree
2 folders deep, 4 files.

@refs
Copy link
Member

refs commented Jan 12, 2021

confirming that happens with the latest master

@refs refs added the bug label Jan 12, 2021
@settings settings bot removed the bug label Jan 12, 2021
@refs refs changed the title [QA] trashcan cannot delete a deep tree [QA] Trashbin cannot delete a deep tree Jan 13, 2021
@refs refs added the Category:Defect Existing functionality is not working as expected label Jan 13, 2021
@settings settings bot removed the p3-medium label Apr 7, 2021
@C0rby
Copy link
Contributor

C0rby commented Apr 9, 2021

It works now in the current master and released version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category:Defect Existing functionality is not working as expected Type:Bug
Projects
None yet
Development

No branches or pull requests

4 participants