otwcode · sarken · Nov 4, 2017 · Nov 2, 2017 · Nov 2, 2017 · Nov 3, 2017
diff --git a/config/initializers/gem-plugin_config/sanitizer_config.rb b/config/initializers/gem-plugin_config/sanitizer_config.rb
@@ -27,6 +27,10 @@ module Config
         'th' => ['abbr', 'axis', 'colspan', 'height', 'rowspan', 'scope', 'width'],
         'ul' => ['type'],
       },
+      
+      add_attributes: {
+        'a' => {'rel' => 'nofollow'}
+      },
 
       protocols: {
         'a' => {'href' => ['ftp', 'http', 'https', 'mailto', :relative]},
diff --git a/lib/html_cleaner.rb b/lib/html_cleaner.rb
@@ -25,7 +25,6 @@ def close_tag(node, attributes=nil)
       self_closing = node.children.empty?
     rescue NameError
       name = node
-      attributes ||= {}
       self_closing = false
     end
 
@@ -124,13 +123,6 @@ def fix_bad_characters(text)
     # convert carriage returns to newlines
     text.gsub!(/\r\n?/, "\n")
 
-    # replace curlyquotes
-    # note: turns out not to be necessary?
-    # text.gsub! "\xE2\x80\x98", "'"
-    # text.gsub! "\xE2\x80\x99", "'"
-    # text.gsub! "\xE2\x80\x9C", '"'
-    # text.gsub! "\xE2\x80\x9D", '"'
-
     # argh, get rid of ____spacer____ inserts
     text.gsub! "____spacer____", ""
 
@@ -271,7 +263,7 @@ def traverse_nodes(node, stack=nil, out_html=nil)
       return [stack, out_html]
     end
 
-    # Don't decend into node if we don't want to touch the content of
+    # Don't descend into node if we don't want to touch the content of
     # this kind of tag
     if dont_touch_content_tag?(node.name)
       if put_inside_p_tag?(node.name) && !stack.inside_paragraph?