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.

Sign up for GitHub

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

Jump to bottom

recursiveDelete doesn't re-fetch container listings #11

Open
michielbdejong opened this issue Dec 16, 2020 · 1 comment · May be fixed by #12
Open

recursiveDelete doesn't re-fetch container listings #11

michielbdejong opened this issue Dec 16, 2020 · 1 comment · May be fixed by #12
Assignees
@bourgeoa
Labels
bug Something isn't working

Comments

@michielbdejong
Copy link
Collaborator

When you use recursiveDelete twice, it incorrectly uses the old container listings the second time.
For instance:

  • create foo/bar
  • recursiveDelete foo/ (correctly deletes foo/bar and then foo/)
  • create foo/baz
  • recursiveDelete foo/ (incorrectly tries to delete foo/bar and then foo/, instead of foo/baz and then foo/)
michielbdejong referenced this issue in solid-contrib/web-access-control-tests Dec 16, 2020
@ylebre
Multiple changes:
67970d1 
prefix directories as workaround for failing deleteRecursive
skipping tests which check for failure but are allowed based on the default grants
changed grants for several requests
@bourgeoa
Copy link
Contributor

resolved with PR #12

@timea-solid timea-solid linked a pull request Mar 8, 2022 that will close this issue
add options to load() to resolve recursiveDelete() issue #12
Open
@timea-solid timea-solid added the bug Something isn't working label Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bourgeoa bourgeoa

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add options to load() to resolve recursiveDelete() issue bourgeoa/solid-logic
3 participants
@michielbdejong @bourgeoa @timea-solid