Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Exception handler in Convertible references nonexisting method .name #419

Merged
merged 1 commit into from

4 participants

@vjt

When an Exception in a liquid template happens, the exception handler is invoked, but it raises another exception (oh, no, not another!), because Post does not quack .name, and the almighty web master finds difficult to understand what the error means.

No need to wonder, but yeah the attached commit fixes the issue. Yay!

:-)

@lastknight

for testing the issue just use a non.existend Liquid Tag (in my case {% youtbe ID %}

@mojombo mojombo merged commit 3c2519f into jekyll:master
@eob
eob commented

Related to this bug, however -- is there a way to get an informational message about which template tag is causing the exception? I'm running into this problem right now and can't find what in the post is causing the server process to barf.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 0 deletions.
  1. +2 −0  lib/jekyll/post.rb
View
2  lib/jekyll/post.rb
@@ -22,6 +22,8 @@ def self.valid?(name)
attr_accessor :data, :content, :output, :ext
attr_accessor :date, :slug, :published, :tags, :categories
+ attr_reader :name
+
# Initialize this Post instance.
# +site+ is the Site
# +base+ is the String path to the dir containing the post file
Something went wrong with that request. Please try again.