Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#567 closed defect (fixed)

ContentPane::splitAndFixPaths causes "unterminated parenthetical"

Reported by: js-dojo-tester Owned by: mumme@…
Priority: high Milestone:
Component: Widgets Version: 0.3
Keywords: Cc:
Blocked By: Blocking:

Description

R3357 introduces an issue in ContentPane::splitAndFixPaths. => Spaces in a href-attribute causes one of the regex-matches/searches to stop after the first space.

OK <a href="javascript:doSomething(50,100);" > test </a> NOK: <a href="javascript:doSomething(50, 100);" > test </a> => The space between '50,' and '100' causes a parsing error and generates a 'unterminated parenthetical'-javascript error in FF.

P.S: As a workaround I can use the alternative syntax: OK: <a href="javascript:" onclick="doSomething(50, 100); return false;" >

Change History (3)

comment:1 Changed 13 years ago by anonymous

Owner: changed from anonymous to mumme@…

comment:2 Changed 13 years ago by mumme@…

Resolution: fixed
Status: newclosed

Enhanced splitAndFixPaths in ContentPane? to exclude fix on these protocols:

http: https: ftp: ftps: file: news: mailto: and javascript:

Should be fixed as of 3457, extended testcase for test_RemotePane.html

Thank you js-dojo-tester for finding this!

/ Fredrik

comment:3 Changed 12 years ago by (none)

Milestone: 0.3release

Milestone 0.3release deleted

Note: See TracTickets for help on using tickets.