Merge pull request #21 from benbalter/entities

Do not escape smart quotes

Author: benbalter

lib/jekyll-seo-tag.rb

@@ -3,19 +3,10 @@ class SeoTag < Liquid::Tag
 
     attr_accessor :context
 
-    HTML_ESCAPE = {
-      "\u201c".freeze => '&ldquo;'.freeze,
-      "\u201d".freeze => '&rdquo;'.freeze
-    }
-    HTML_ESCAPE_REGEX = Regexp.union(HTML_ESCAPE.keys).freeze
-
     def render(context)
       @context = context
       output = template.render!(payload, info)
 
-      # Encode smart quotes. See https://github.com/benbalter/jekyll-seo-tag/pull/6
-      output.gsub!(HTML_ESCAPE_REGEX, HTML_ESCAPE)
-
       output
     end
 

spec/jekyll_seo_tag_spec.rb

@@ -32,19 +32,6 @@
     expect(subject.render(context)).to match(/<title>foo<\/title>/)
   end
 
-  it "escapes titles" do
-    site = site({"title" => 'Jekyll & "Hyde"'})
-    context = context({ :site => site })
-    expect(subject.render(context)).to match(/<title>Jekyll &amp; &ldquo;Hyde&rdquo;<\/title>/)
-  end
-
-  it "escapes descriptions" do
-    site = site({"description" => 'Jekyll & "Hyde"'})
-    context = context({ :site => site })
-    expected = /<meta name="description" content="Jekyll &amp; &ldquo;Hyde&rdquo;" \/>/
-    expect(subject.render(context)).to match(expected)
-  end
-
   it "uses the page description" do
     page = page({"description" => "foo"})
     context = context({ :page => page })