feature: use nokogiri for strip tags

abak-press · Sep 23, 2019 · f495195 · f495195
1 parent 2054e2a
commit f495195
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/lib/string_tools/core_ext/string.rb b/lib/string_tools/core_ext/string.rb
@@ -4,7 +4,6 @@
 require 'active_support/version'
 require 'action_pack'
 require 'string_tools/string'
-require 'action_controller'
 
 class String
   %w[auto_link excerpt highlight sanitize simple_format word_wrap].each do |method|
@@ -19,7 +18,7 @@ def #{method}(*args)
   # возвращает строку из которой удалены HTML-теги
   # символы <>&"  остаются без изменения
   def strip_tags
-    ActionController::Base.helpers.strip_tags(self).to_str.gsub(/<!--/, '<--')
+    Nokogiri::HTML5.parse(self).content
   end
 
   # '11,3'.to_f

diff --git a/spec/string_spec.rb b/spec/string_spec.rb
@@ -38,9 +38,9 @@
       'some<!-- comment -->text' => 'sometext',
       'some<ul>text' => 'sometext',
       'some<script>text</script>' => 'sometext',
-      'some<!-- comment' => 'some<-- comment',
+      'some<!-- comment' => 'some',
       '<h1 style="coller:red">>text</h1>>' => '>text>',
-      " <ul>list<li>text</li>\n  <li>text2</li> " => " listtext\n  text2 "
+      " <ul>list<li>text</li>\n  <li>text2</li> " => "listtext\n  text2 "
     }.each do |example_string, result|
       context "when string '#{example_string}'" do
         it { expect(example_string.dup.strip_tags).to eq result }