summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJean-Baptiste Barth <jeanbaptiste.barth@gmail.com>2010-10-23 22:35:02 +0000
committerJean-Baptiste Barth <jeanbaptiste.barth@gmail.com>2010-10-23 22:35:02 +0000
commitd642964035e6be4687c37e9e01af4b5590580396 (patch)
tree6dcdab0f7b40f4f7f7a8b794d78cf527e01a02b0
parent7824eca77561dea52960b3d094389bd4e85eeccf (diff)
downloadredmine-d642964035e6be4687c37e9e01af4b5590580396.tar.gz
redmine-d642964035e6be4687c37e9e01af4b5590580396.zip
Fixed: auto links ending with right angle bracket shouldn't include the bracket in the URL. #5652
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4287 e93f8b46-1217-0410-a6f0-8f06a7374b81
-rw-r--r--lib/redmine/wiki_formatting/textile/formatter.rb2
-rw-r--r--test/unit/helpers/application_helper_test.rb2
2 files changed, 3 insertions, 1 deletions
diff --git a/lib/redmine/wiki_formatting/textile/formatter.rb b/lib/redmine/wiki_formatting/textile/formatter.rb
index a622e8039..6794f82c7 100644
--- a/lib/redmine/wiki_formatting/textile/formatter.rb
+++ b/lib/redmine/wiki_formatting/textile/formatter.rb
@@ -121,7 +121,7 @@ module Redmine
(\S+?) # url
(\/)? # slash
)
- ([^\w\=\/;\(\)]*?) # post
+ ((?:&gt;)?|[^\w\=\/;\(\)]*?) # post
(?=<|\s|$)
}x unless const_defined?(:AUTO_LINK_RE)
diff --git a/test/unit/helpers/application_helper_test.rb b/test/unit/helpers/application_helper_test.rb
index d44bc0045..83a508594 100644
--- a/test/unit/helpers/application_helper_test.rb
+++ b/test/unit/helpers/application_helper_test.rb
@@ -79,6 +79,8 @@ class ApplicationHelperTest < ActionView::TestCase
'http://example.net/path!602815048C7B5C20!302.html' => '<a class="external" href="http://example.net/path!602815048C7B5C20!302.html">http://example.net/path!602815048C7B5C20!302.html</a>',
# escaping
'http://foo"bar' => '<a class="external" href="http://foo&quot;bar">http://foo"bar</a>',
+ # wrap in angle brackets
+ '<http://foo.bar>' => '&lt;<a class="external" href="http://foo.bar">http://foo.bar</a>&gt;'
}
to_test.each { |text, result| assert_equal "<p>#{result}</p>", textilizable(text) }
end