URL of link not recognized when parentheses badly nested
Reported by Matt Jankowski | September 17th, 2008 @ 09:22 AM
That pasted in poorly, attaching a paste instead
Comments and changes to this ticket
-
Matt Jankowski September 17th, 2008 @ 09:23 AM
- no changes were found...
-
Matt Jankowski September 17th, 2008 @ 09:26 AM
test string is...
Test -(read this), ok
I think that this person most likely intended to put that opening paren to the left of the quote that opens the link, but should this scenario really raise an error? It'd be preferable to just get incorrect results than an error.
-
Jason Garber September 19th, 2008 @ 05:08 PM
- Title changed from “html_esc issue” to “URL of link not recognized under certain conditions”
- State changed from “new” to “open”
- Tag set to “links, parser”
A rare thing, but one I want to fix anyway. The problem goes away if you do any of these things in the test case: remove the stray parenthesis, remove the text after the link... and a bunch of other stuff. It's very touchy, but I will fix it.
-
Matt Jankowski September 19th, 2008 @ 05:19 PM
Yeah - I was able to correct this in production by just editing the content, by assume that even if things are poorly formed textile and can't parse, that they should just fail to render results properly, rather than raising an error.
-
Jason Garber September 22nd, 2008 @ 03:09 PM
(from [b02f629a9af454930a08c78e482db0cbc3968448]) Fixed an error that occurred when a link was badly nested in parentheses.
The error is fixed, but the mis-recognition of the badly written link is still a problem. [#55 state:open] http://github.com/jgarber/redclo...
-
Jason Garber September 22nd, 2008 @ 03:10 PM
- Title changed from “URL of link not recognized under certain conditions” to “URL of link not recognized when parentheses badly nested”
-
Jason Garber October 1st, 2008 @ 04:42 PM
- Milestone cleared.
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
RedCloth is a Ruby library for converting Textile into HTML
People watching this ticket
Jason Garber
Matt Jankowski
Attachments
Referenced by
-
55
URL of link not recognized under certain conditions
The error is fixed, but the mis-recognition of the badly ...