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

Breadcrumb navigation broken #64

Closed
kulmann opened this issue Jun 10, 2021 · 2 comments · Fixed by #69 or #76
Closed

Breadcrumb navigation broken #64

kulmann opened this issue Jun 10, 2021 · 2 comments · Fixed by #69 or #76
Assignees
Labels
bug Something isn't working

Comments

@kulmann
Copy link
Member

kulmann commented Jun 10, 2021

The breadcrumb navigation seems to be broken.

Steps to reproduce

  1. Create a folder hierarchy A/B/C/D
  2. Navigate into folder D
  3. Try to navigate into one of the folders up the hierarchy, let's pick C

Expected behaviour

Navigation into C is successful and I can see folder contents of C

Actual behaviour

  • Breadcrumbs stay the same (Home / A / B / C / D) although I clicked on C
  • No folder contents visible. Instead I see a loading spinner.

browser logs

The JS console shows that the propfind contains an encoded path, thus the propfind fails:
PROPFIND http://host.docker.internal/px1/remote.php/webdav/A%252FB%252FC 404 (Not Found)

There is another, probably unrelated, error in the JS console:

owncloud.js?4716:1 Uncaught (in promise) TypeError: Cannot read property 'data' of undefined
    at eval (owncloud.js?4716:1)

Screenshot

Screenshot 2021-06-10 at 14 28 07

Bonus points for adding tests to avoid this in the future. ;-)

@kulmann kulmann added the bug Something isn't working label Jun 10, 2021
@VP-
Copy link

VP- commented Jun 18, 2021

In 0.6.0 I get the spinner when clicking on the current folder in the breadcrumb.
The file picker attempts a request to webdav/null, which results in a 404.

But at least clicking on another folder gets it working again.

@kulmann kulmann reopened this Jun 19, 2021
@kulmann
Copy link
Member Author

kulmann commented Jun 19, 2021

cc @LukasHirt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants