From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: Wed, 11 Nov 2015 07:35:44 +0000 (+0000)
Subject: Merged r14863 (#6969).
X-Git-Tag: 2.6.8~2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=f2e03946a5cc216b3f6c01084801ad1373ef83cc;p=redmine.git

Merged r14863 (#6969).

git-svn-id: http://svn.redmine.org/redmine/branches/2.6-stable@14866 e93f8b46-1217-0410-a6f0-8f06a7374b81
---

diff --git a/lib/redcloth3.rb b/lib/redcloth3.rb
index dc1058026..31a558799 100644
--- a/lib/redcloth3.rb
+++ b/lib/redcloth3.rb
@@ -1202,7 +1202,7 @@ class RedCloth3 < String
     ALLOWED_TAGS = %w(redpre pre code notextile)
     
     def escape_html_tags(text)
-      text.gsub!(%r{<(\/?([!\w]+)[^<>\n]*)?(>?)}) {|m| $2 && ALLOWED_TAGS.include?($2) ? "<#{$1}#{$3}" : "&lt;#{$1}#{'&gt;' unless $3.blank?}" }
+      text.gsub!(%r{<(\/?([!\w]+)[^<>\n]*)(>?)}) {|m| ALLOWED_TAGS.include?($2) ? "<#{$1}#{$3}" : "&lt;#{$1}#{'&gt;' unless $3.blank?}" }
     end
 end
 
diff --git a/test/unit/lib/redmine/wiki_formatting/textile_formatter_test.rb b/test/unit/lib/redmine/wiki_formatting/textile_formatter_test.rb
index a505b6661..5c7735547 100644
--- a/test/unit/lib/redmine/wiki_formatting/textile_formatter_test.rb
+++ b/test/unit/lib/redmine/wiki_formatting/textile_formatter_test.rb
@@ -154,19 +154,12 @@ EXPECTED
     assert_equal expected.gsub(%r{\s+}, ''), to_html(raw).gsub(%r{\s+}, '')
   end
 
-  def test_should_escape_unallowed_tags
+  def test_escaping
     assert_html_output(
       'this is a <script>'      => 'this is a &lt;script&gt;'
     )
   end
 
-  def test_should_escape_less_than_signs
-    assert_html_output(
-      '<'                     => '&lt;',
-      '1 < 2'                 => '1 &lt; 2'
-    )
-  end
-
   def test_use_of_backslashes_followed_by_numbers_in_headers
     assert_html_output({
       'h1. 2009\02\09'      => '<h1>2009\02\09</h1>'