1926: DiffViewer JavaScript script does not escape special characters string in paths

Jan.Ko*******@gmai***** (Google Code) (Is this you? Claim this profile.)
April 4, 2011
What version are you running?
Review Board 1.5


What's the URL of the page containing the problem?
http://localhost:8000/r/1/diff/#index_header

What steps will reproduce the problem?
1. Public ClearCase review request under Windows e.g file:
SCM\tools\main\2\test.c

What is the expected output? What do you see instead?
Expected: In Files Changed I should see "SCM\tools\main\2\test.c" position
What is see: "SCM[TAB]ools[TAB]est.c" where [TAB] is invisible character.

What operating system are you using? What browser?
Windows 7

Please provide any additional information below.
I know this is JavaScript issue because after turning off JavaScript paths looks fine.
david
#1 david
Fixed in reviewboard 7553ec1 and rbtools 47fb2ce. Thanks!
  • +Fixed