breadcrumb not working on Export Listing page broken

GoogleCodeExporter commented 9 years ago

Version Info
TongueTied: 1.3.5
OS: Fedora release 12 (Constantine)
Java: 1.6
Browser: 3.6.2

What steps will reproduce the problem?
1. make an export
2. from /export choose the directory with your export
3. from inside export directory - try to go back to /export directory by
either clicking breadcrumb ('/export') or by clicking on the folder icon.

What is the expected output? 
To go back to the export listing directory

What do you see instead?
You have /null appended to breadcrumb. See screenshots.

Original issue reported on code.google.com by kyle.br...@gmail.com on 25 Mar 2010 at 3:15

Attachments:

[Picture 6.png](https://storage.googleapis.com/google-code-attachments/tongue-tied/issue-103/comment-0/Picture 6.png)
[Picture 7.png](https://storage.googleapis.com/google-code-attachments/tongue-tied/issue-103/comment-0/Picture 7.png)
[Picture 8.png](https://storage.googleapis.com/google-code-attachments/tongue-tied/issue-103/comment-0/Picture 8.png)
[Picture 9.png](https://storage.googleapis.com/google-code-attachments/tongue-tied/issue-103/comment-0/Picture 9.png)

GoogleCodeExporter commented 9 years ago

This problem only seems to affect linux users. Using windows, the problem does 
not occur.

Original comment by bjs...@gmail.com on 28 Mar 2010 at 6:37

Changed state: Accepted
Added labels: Component-UI, OpSys-Linux, Priority-Medium, Type-Defect, Usability

GoogleCodeExporter commented 9 years ago

Problem is actually with the tomcat on linux. I am guessing because of JSTL lib 
in
tomcat, though I have not confirmed the exact reason. It was easier just to 
fix. Root
cause was an empty variable in a JSTL tag is interpreted as "null" not an empty
string and hence the links created in the fileListing.jsp get out appended with 
the
value "null". I have added a line to initialise the variable to an empty 
string. This
should fix the problem.

Original comment by bjs...@gmail.com on 28 Mar 2010 at 11:40

GoogleCodeExporter commented 9 years ago

Hi Kyle,

Let me know if you want the fix before an offical release. As a workaround for 
now
the browser back button will work too.

Original comment by bjs...@gmail.com on 28 Mar 2010 at 11:44

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Thanks for making the fix and offering to make it available. I can wait for an
official release though. Cheers

Original comment by kyle.br...@gmail.com on 31 Mar 2010 at 8:03

bjsion / tongue-tied

breadcrumb not working on Export Listing page broken #103