Merge commit 'bahuvrihi/master'

lib/jekyll/convertible.rb

@@ -17,12 +17,14 @@ module Jekyll
     # Returns nothing
     def read_yaml(base, name)
       self.content = File.read(File.join(base, name))
-
+      
-      if self.content =~ /^(---\s*\n.*?)\r?\n---\s*\n/m
+      if self.content =~ /^(---\s*\n.*?\n?)(---.*?\n)/m
-        self.content = self.content[($1.size + 5)..-1]
+        self.content = self.content[($1.size + $2.size)..-1]
-
+      
-        self.data = YAML.load($1.dup)
+        self.data = YAML.load($1)
       end
+      
+      self.data ||= {}
     end
 
     # Transform the contents based on the file extension.

lib/jekyll/page.rb

@@ -20,8 +20,6 @@ module Jekyll
       @dir  = dir
       @name = name
 
-      self.data = {}
-
       self.process(name)
       self.read_yaml(File.join(base, dir), name)
     end

test/source/_posts/2009-06-22-empty-yaml.textile

@@ -0,0 +1,3 @@
+---
+---
+Empty YAML.

test/source/_posts/2009-06-22-no-yaml.textile

@@ -0,0 +1 @@
+No YAML.

test/test_post.rb

@@ -77,7 +77,7 @@ class TestPost < Test::Unit::TestCase
           @post.read_yaml(@source, @real_file)
 
           assert_equal({"title" => "Test title", "layout" => "post", "tag" => "Ruby"}, @post.data)
-          assert_equal "\r\n\r\nThis is the content", @post.content
+          assert_equal "\r\nThis is the content", @post.content
         end
       end
 
@@ -235,6 +235,16 @@ class TestPost < Test::Unit::TestCase
         assert post.tags.include?('cooking')
         assert post.tags.include?('pizza')
       end
+      
+      should "allow no yaml" do
+        post = setup_post("2009-06-22-no-yaml.textile")
+        assert_equal "No YAML.", post.content
+      end
+
+      should "allow empty yaml" do
+        post = setup_post("2009-06-22-empty-yaml.textile")
+        assert_equal "Empty YAML.", post.content
+      end
 
       context "rendering" do
         setup do