Merge pull request #104 from jekyll/pr/image

Merge pull request 104
jekyll · Mar 22, 2016 · e25c27d · e25c27d
2 parents 680ded5 + 9ac99bd
commit e25c27d
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 0 deletions.
diff --git a/README.md b/README.md
@@ -112,6 +112,8 @@ There are several ways to convey author-specific information. Author information
   author: benbalter
   ```
 
+* `image` - URL of an image that is representative of the post.  
+
 ### Meta tags
 
 The plugin exposes a helper tag to expose the appropriate meta tags to support automated discovery of your feed. Simply place `{% feed_meta %}` someplace in your template's `<head>` section, to output the necessary metadata.

diff --git a/lib/feed.xml b/lib/feed.xml
@@ -95,6 +95,14 @@
       {% if post.excerpt and post.excerpt != empty %}
         <summary>{{ post.excerpt | strip_html | replace: '\n', ' ' | strip | xml_escape }}</summary>
       {% endif %}
+
+      {% assign post_image = post.image %}
+      {% if post_image %}
+        {% unless post_image contains "://" %}
+          {% assign post_image = post_image | prepend: url_base | xml_escape  %}
+        {% endunless %}
+        <media:thumbnail xmlns:media="http://search.yahoo.com/mrss/" url="{{ post_image }}" />
+      {% endif %}
     </entry>
   {% endunless %}
   {% endfor %}

diff --git a/spec/fixtures/_posts/2013-12-12-dec-the-second.md b/spec/fixtures/_posts/2013-12-12-dec-the-second.md
@@ -1,5 +1,6 @@
 ---
 excerpt: "Foo"
+image: "/image.png"
 ---
 
 # December the twelfth, actually.
diff --git a/spec/fixtures/_posts/2014-03-02-march-the-second.md b/spec/fixtures/_posts/2014-03-02-march-the-second.md
@@ -1,4 +1,5 @@
 ---
+image: "https://cdn.example.org/absolute.png"
 ---
 
 March the second!
diff --git a/spec/jekyll-feed_spec.rb b/spec/jekyll-feed_spec.rb
@@ -85,6 +85,11 @@
     expect(contents).not_to match /Liquid is not rendered\./
   end
 
+  it "includes the item image" do
+    expect(contents).to include('<media:thumbnail xmlns:media="http://search.yahoo.com/mrss/" url="http://example.org/image.png" />')
+    expect(contents).to include('<media:thumbnail xmlns:media="http://search.yahoo.com/mrss/" url="https://cdn.example.org/absolute.png" />')
+  end
+
   context "parsing" do
     let(:feed) { RSS::Parser.parse(contents) }