From bb811b0692b0d2366fd6befc2b5b4ccbfd773b57 Mon Sep 17 00:00:00 2001
From: Tunghsiao Liu <t@sparanoid.com>
Date: Fri, 6 Jan 2017 15:16:09 +0800
Subject: [PATCH] feat(template): simplify thumbnail logic

---
 TODOS.md                                      |  1 -
 _app/_includes/_amsf.html                     | 31 ++++++++++++++-----
 .../themes/curtana/includes/open-graph.html   |  2 +-
 3 files changed, 24 insertions(+), 10 deletions(-)

diff --git a/TODOS.md b/TODOS.md
index 00cce28b0..2c9bde98f 100644
--- a/TODOS.md
+++ b/TODOS.md
@@ -1,4 +1,3 @@
 # Todos
 
 - Try load-grunt-config
-- Cleanup thumb fetching method for `amsf_page_thumb` in `_amsf.html`
diff --git a/_app/_includes/_amsf.html b/_app/_includes/_amsf.html
index beea2a650..275481101 100644
--- a/_app/_includes/_amsf.html
+++ b/_app/_includes/_amsf.html
@@ -199,11 +199,11 @@
 {% endcapture %}
 
 {% comment %}
-  Page thumbnail
+  First image in post
   TODO: https://regex101.com/r/N2hoxV/1
 {% endcomment %}
 {% assign amsf_site_logo = "/logo.png" | prepend: amsf_site_base %}
-{% capture amsf_post_first_image %}
+{% capture amsf_page_first_image %}
   {% assign hero_image = page.content | match_regex: '<img .*?src="([^"]+)"' %}
   {% if hero_image == nil %}
     {% assign hero_image = amsf_site_logo %}
@@ -228,17 +228,32 @@
   {% endif %}
 {% endcapture %}
 
-{% capture amsf_page_thumb %}
+{% comment %}
+  Page thumbnail
+  Regex: https://regex101.com/r/AmwvYE/2
+{% endcomment %}
+{% capture amsf_page_thumbnail_src %}
+  {% assign protocal = page.thumbnail | match_regex: '^((?:https?:)?\/\/)' %}
+  {% if protocal == null %}
+    {% if page.thumbnail-local %}
+      {{ amsf_user_assets | prepend: amsf_site_base }}/{{ page.thumbnail }}
+    {% else %}
+      {{ site.file }}/{{ page.thumbnail }}
+    {% endif %}
+  {% else %}
+    {{ page.thumbnail }}
+  {% endif %}
+{% endcapture %}
+
+{% capture amsf_page_thumbnail %}
   {% if page.thumb_force_logo %}
     {{ amsf_site_logo }}
-  {% elsif page.thumb %}
-    {{ site.file }}/thumb/{{ page.thumb }}
-  {% elsif page.thumb_url %}
-    {{ page.thumb_url }}
+  {% elsif page.thumbnail %}
+    {{ amsf_page_thumbnail_src }}
   {% elsif page.heading-bg %}
     {{ amsf_page_heading_bg_image_src }}
   {% else %}
-    {{ amsf_post_first_image }}
+    {{ amsf_page_first_image }}
   {% endif %}
 {% endcapture %}
 
diff --git a/_app/_includes/themes/curtana/includes/open-graph.html b/_app/_includes/themes/curtana/includes/open-graph.html
index 4042d6d3b..908ac23da 100644
--- a/_app/_includes/themes/curtana/includes/open-graph.html
+++ b/_app/_includes/themes/curtana/includes/open-graph.html
@@ -7,7 +7,7 @@
 <meta property="og:title" content="{{ amsf_page_title }}">
 <meta property="og:url" content="{{ amsf_page_url }}">
 <meta property="og:description" content="{{ amsf_site_desc | strip }}">
-<meta property="og:image" content="{{ amsf_page_thumb | strip }}">
+<meta property="og:image" content="{{ amsf_page_thumbnail | strip }}">
 
 <meta name="twitter:card" content="{{ amsf_page_type_twitter | strip }}">