properly escape smart quotes in title and description

Author: benbalter

lib/jekyll-seo-tag.rb

@@ -5,7 +5,16 @@ class SeoTag < Liquid::Tag
 
     def render(context)
       @context = context
-      Liquid::Template.parse(template_contents).render!(payload, info).gsub(/[\n\s]{2,}/, "\n")
+      output = Liquid::Template.parse(template_contents).render!(payload, info)
+
+      # Minify
+      output.gsub!(/[\n\s]{2,}/, "\n")
+
+      # Encode smart quotes. See https://github.com/benbalter/jekyll-seo-tag/pull/6
+      output.gsub!("\u201c", "&ldquo;")
+      output.gsub!("\u201d", "&rdquo;")
+
+      output
     end
 
     private

lib/template.html

@@ -18,7 +18,7 @@
 	{% endif %}
 {% endif %}
 {% if seo_title %}
-  {% assign seo_title = seo_title | escape | markdownify | strip_html | strip_newlines | replace:'"','"' | escape_once %}
+  {% assign seo_title = seo_title | markdownify | strip_html | strip_newlines | escape_once %}
 {% endif %}
 
 {% if page.description %}
@@ -27,7 +27,7 @@
   {% assign seo_description = site.description %}
 {% endif %}
 {% if seo_description %}
-  {% assign seo_description = seo_description | escape | markdownify | strip_html | strip_newlines | replace:'"','"' | escape_once %}
+  {% assign seo_description = seo_description | markdownify | strip_html | strip_newlines | escape_once %}
 {% endif %}
 
 {% if seo_title %}

spec/jekyll_seo_tag_spec.rb

@@ -35,13 +35,13 @@
   it "escapes titles" do
     site = site({"title" => 'Jekyll & "Hyde"'})
     context = context({ :site => site })
-    expect(subject.render(context)).to match(/<title>Jekyll &amp; &quot;Hyde&quot;<\/title>/)
+    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; &quot;Hyde&quot;" \/>/
+    expected = /<meta name="description" content="Jekyll &amp; &ldquo;Hyde&rdquo;" \/>/
     expect(subject.render(context)).to match(expected)
   end