SDWebServer Example - Link in /edit/index.htm not working

[PhilippRott]

Description

If you try to edit files on the sdcard by espXX.local/edit, this will work only, if file is in root path like: "/test.txt"

If you create a folder on the sd-card "/folder1/test.txt" the link on tree-view is:
<li id="/folder1/folder1/test.txt"> ...

So, the path will be double and the link is not working - files can not be edited or displayed.

My solution to fix this problem:

edit line 378 in Arduino/libraries/ESP8266WebServer/examples/SDWebServer/SdRoot/edit/index.htm
change this:
leaf.id = (((path == "/")?"":path)+"/"+name).toLowerCase();
to following:
leaf.id = (((path == "/")?"":"")+"/"+name).toLowerCase();

Hardware

Hardware: ESP32, but expecting the same for esp8266, because its a html problem!

[earlephilhower]

Would you like to submit a PR for your fix? Seems like you've already got a failing and passing test case with it, and it'll get your name in the "git blame" for all time. :)

[PhilippRott]

I never tried to submit a PR before, need to read how to do this in the correct way ;-)

[earlephilhower]

If it's a problem I can do it with your info here, but it's not too complicated...there are official readmes, but below is my general flow:

Fork the repo from github website

git clone https://github.com/PhilippRott/Arduino
git checkout -b fixsdwebserver
<do changes, test>
git commit -a
git push (it'll give you the proper command line to push to a branch on your repo @ github)

Go back to your forked repo @ GH and there'll be a new button saying "Branch X pushed Y mins ago. Click here to cubmit a pull request")

[PhilippRott]

thanks for your help, my PR is now pending...

[PhilippRott]

@earlephilhower seems to finished with a PR error - any idea what's the problem?

[earlephilhower]

I just restarted it. There was a build timeout (you can check the details link at the bottom), sometimes happens when there's heavy load on the CI system.

[earlephilhower]

PR #4451 fixes, link

[earlephilhower]

Related to #5525

[devyte]

@earlephilhower what's the status here? I see the PR is ready, I assume it just needs testing?

[earlephilhower]

The PR's not in an acceptable state (x += (cond)? "" : "" i.e. a no-op ternary is the solution proposed).

I need to verify the actual error, too. I don't remember seeing it fail.

[devyte]

Pushing back after internal discussion