If a theorytab title has a bracket, editing the theorytab will put me to a 404 page

If you edit this theorytab what will happen is it will send you to https://www.hooktheory.com/theorytab/view/smash-mouth/all-star-(negative-harmony-version) so I get rickrolled instead of getting sent to the theorytab

Thanks for the report. We’ll take a look and catch disallow these nonstandard chars so the crash doesn’t happen

@dave so it’s still not fixed

This loads for me:
https://www.hooktheory.com/theorytab/view/smash-mouth/all-star-%28negative-harmony-version%29

@dave I edited a different theorytab that also had a bracket and the same thing happened there

@dave you haven’t fixed it

@dave that link is actually https://www.hooktheory.com/theorytab/view/smash-mouth/all-star-%2528negative-harmony-version%2529 but if you edit the theorytab it sends you to https://www.hooktheory.com/theorytab/view/smash-mouth/all-star-%28negative-harmony-version%29

@dave why haven’t you fixed it yet

if you edit https://www.hooktheory.com/theorytab/view/smash-mouth/all-star-%2528negative-harmony-version%2529 it sends you to https://www.hooktheory.com/theorytab/view/smash-mouth/all-star-%28negative-harmony-version%29

@Vaz123,

Sorry this thread (and others related to the song db) have taken a back seat. I tried some of the above links but none work. Is it possible that the old version with the bad character in the URL was deleted?

The only ‘negative harmony’ version of All Star I can find is at this URL, and the edit link works.

https://www.hooktheory.com/theorytab/view/smash-mouth/all-star---negative-harmony

https://hookpad.hooktheory.com/?idOfWiki=dJNgq-YEgrz

@chris Yes I changed the name of that one but I still have some more:

https://www.hooktheory.com/theorytab/view/nintendo/hyness-%2528unhooded%2529-battle---kirby-star-allies

Edit the theorytab and save it and it will send you to:

https://www.hooktheory.com/theorytab/view/nintendo/hyness-%28unhooded%29-battle---kirby-star-allies

@chris so can you fix it

@Vaz, thank you for the links and for bringing this issue to our attention.

This problem has been fixed. The fix, however, was to fix the original URLs of songs with parenthesis. They were being double URL encoded in a weird way which led to the %2528 in the original URL.

The new URL for this project does not contain the extra %25 wrappers around the parenthesis encoding. Here it is:

https://www.hooktheory.com/theorytab/view/nintendo/hyness-%28unhooded%29-battle---kirby-star-allies

Editing it will return you to the same URL, as expected. This URL change will affect all songs that have parens in the artist or song name.

@chris now there’s a new bug, if a theorytab has a bracket, then clicking on a section name in the sections list at the top will also send you to a 404 page

e.g. on this theorytab if you click on ‘Verse’ in the sections list then instead of sending you to

https://www.hooktheory.com/theorytab/view/the-lorax/biggering-%28original-demo%29#verse

it sends you to

https://www.hooktheory.com/theorytab/view/the-lorax/biggering-%2528original-demo%2529#verse

and when submitting a theorytab with parantheses it will also add the %25

@chris @Ryan @dave This still isn’t fixed

and I haven’t tested this recently but I’m 99% sure this is also not fixed