This project is archived and is in readonly mode.
auto_link should ignore multiple trailing punctuation marks
Reported by cainlevy | April 16th, 2009 @ 12:52 AM | in 2.x
auto_link currently only ignores one trailing punctuation mark. This patch adds a test and support for an arbitrary number of trailing punctuations.
Previously
auto_link("Lorum ipsum sit dolor amet (http://example.com).")
=> "Lorum ipsum sit dolor amet (<a href=\"http://example.com)\">http://example.com)</a>."
With Patch
auto_link("Lorum ipsum sit dolor amet (http://example.com).")
=> "Lorum ipsum sit dolor amet (<a href=\"http://example.com\">http://example.com</a>)."
Note: I wrote this patch against the 2-3-stable branch.
Comments and changes to this ticket
-
cainlevy April 17th, 2009 @ 09:07 PM
- Tag set to 2-3-stable, auto_link, patch
-
Repository May 17th, 2009 @ 05:56 PM
- State changed from new to resolved
(from [11bac700784efe232083f94e3d28d171957e667e]) Ensure auto_link does not ignore multiple trailing punctuations [#2504 state:resolved]
Signed-off-by: Pratik Naik pratiknaik@gmail.com
http://github.com/rails/rails/commit/11bac700784efe232083f94e3d28d1... -
Repository May 29th, 2010 @ 03:06 AM
(from [9e08e196fa5c7183aec1f6dbeebe6bb7539e3c66]) Ensure auto_link does not ignore multiple trailing punctuations
[#2504 state:resolved] http://github.com/rails/rails/commit/9e08e196fa5c7183aec1f6dbeebe6b...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Attachments
Tags
Referenced by
- 2504 auto_link should ignore multiple trailing punctuation marks [#2504 state:resolved] http://github.com/rails/rails/com...
- 2504 auto_link should ignore multiple trailing punctuation marks (from [11bac700784efe232083f94e3d28d171957e667e]) Ensure ...