From c9b19c478d11da03c062e2dc0e2e7290154dd76d Mon Sep 17 00:00:00 2001
From: Pat Hawks <pat@pathawks.com>
Date: Fri, 29 Jan 2016 10:35:40 -0800
Subject: [PATCH] Add default filter

---
 lib/jekyll-seo-tag.rb         |  4 +++-
 lib/jekyll-seo-tag/filters.rb | 14 ++++++++++++++
 lib/template.html             | 30 +++++-------------------------
 3 files changed, 22 insertions(+), 26 deletions(-)
 create mode 100644 lib/jekyll-seo-tag/filters.rb

diff --git a/lib/jekyll-seo-tag.rb b/lib/jekyll-seo-tag.rb
index 2575fc4..1b6f62b 100644
--- a/lib/jekyll-seo-tag.rb
+++ b/lib/jekyll-seo-tag.rb
@@ -1,3 +1,5 @@
+require 'jekyll-seo-tag/filters'
+
 module Jekyll
   class SeoTag < Liquid::Tag
 
@@ -22,7 +24,7 @@ module Jekyll
     def info
       {
         :registers => context.registers,
-        :filters   => [Jekyll::Filters]
+        :filters   => [Jekyll::Filters, JekyllSeoTag::Filters]
       }
     end
 
diff --git a/lib/jekyll-seo-tag/filters.rb b/lib/jekyll-seo-tag/filters.rb
new file mode 100644
index 0000000..0f0d6ef
--- /dev/null
+++ b/lib/jekyll-seo-tag/filters.rb
@@ -0,0 +1,14 @@
+module JekyllSeoTag
+  module Filters
+
+    # This is available in Liquid from version 3 which is required by Jekyll 3
+    # Provided here for compatibility with Jekyll 2.x
+    def default(input, default_value = ''.freeze)
+      if !input || input.respond_to?(:empty?) && input.empty?
+        default_value
+      else
+        input
+      end
+    end
+  end
+end
diff --git a/lib/template.html b/lib/template.html
index 4326fa3..165df98 100644
--- a/lib/template.html
+++ b/lib/template.html
@@ -2,15 +2,10 @@
 
 {% if site.url %}
   {% assign seo_url = site.url | append: site.baseurl %}
-{% elsif site.github.url %}
-  {% assign seo_url = site.github.url %}
 {% endif %}
+{% assign seo_url = seo_url | default: site.github.url %}
 
-{% if site.title %}
-  {% assign seo_site_title = site.title %}
-{% elsif site.name %}
-  {% assign seo_site_title = site.name %}
-{% endif %}
+{% assign seo_site_title = site.title | default: site.name %}
 
 {% if page.title %}
   {% assign seo_title = page.title %}
@@ -35,29 +30,14 @@
   {% assign seo_page_title = seo_page_title | markdownify | strip_html | strip_newlines | escape_once %}
 {% endif %}
 
-{% if page.description %}
-  {% assign seo_description = page.description %}
-{% elsif page.excerpt %}
-  {% assign seo_description = page.excerpt %}
-{% elsif site.description %}
-  {% assign seo_description = site.description %}
-{% endif %}
+{% assign seo_description = page.description | default: page.excerpt | default: site.description %}
 {% if seo_description %}
   {% assign seo_description = seo_description | markdownify | strip_html | strip_newlines | escape_once %}
 {% endif %}
 
 {% if page.author %}
-  {% if page.author.name %}
-    {% assign seo_author_name = page.author.name %}
-  {% else %}
-    {% assign seo_author_name = page.author %}
-  {% endif %}
-
-  {% if page.author.twitter %}
-    {% assign seo_author_twitter = page.author.twitter %}
-  {% else %}
-    {% assign seo_author_twitter = page.author %}
-  {% endif %}
+  {% assign seo_author_name = page.author.name | default: page.author %}
+  {% assign seo_author_twitter = page.author.twitter | default: page.author %}
 {% endif %}
 {% if seo_author_twitter %}
   {% assign seo_author_twitter = seo_author_twitter | replace:"@","" | prepend:"@" %}