This project is archived and is in readonly mode.
auto_link helper fails to parse some URLs with brackets
Found another URL that auto_link was failing on:
http://connect.oraclecorp.com/se...[q]=green+france&search[type]=Group
Anyway, fixed. Patch attached.
-
auto_link_brackets.diff
1.98 KB
-
RichAfter testing some more, I noticed that with or without this patch, a URL like the following gets parsed improperly. Check out the pastie.
I'll put in a separate ticket.
Rich
-
MislavHere is a patch that completely refactors auto_link_urls, fixing this and #1151 auto_link anchor's error of module TextHelper , #1148 Make auto_link_urls accept commas in the querystring of URLs (the last remaining issues about this).
-
auto_link.patch
22.3 KB
-
-
Repository- State changed from new to committed
(from [4f984c9d0e66601a81cb5ae6e3b50582e6dc0c2d]) auto_link helper: add intelligent ending closing bracket handling. add new tests and reorder new ones for readability
Signed-off-by: Michael Koziarski michael@koziarski.com [#1353 auto_link helper fails to parse some URLs with brackets state:committed] http://github.com/rails/rails/co...