Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Friendly display of article having a damaged "front matter" #71

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants

thenrio commented Jan 24, 2011

Hello !

Having a damaged file under /articles prevents site from displaying '/'

TypeError at /
can't convert String into Hash

Damaged?

By damaged, I mean front matter does not parse to Hash using YAML.load

(I had a quick search on ticket 'convert' keyword, and did not find)

So here is a topic branch for this functionality

Why do I think it is useful ?

One mistake means no site

I made a demo with such an article ... (git checkout saved me after 30s of bewilderment)

What may be better ?

reduce list of article to undamaged articles (ignore damaged article)

Cheers

ixti commented on 3903d57 Aug 10, 2011

I think load_front should be renamed to load_meta
Also I guess error message should say "Article's meta data is missing or corrupted".

ixti commented Aug 10, 2011

That's a great proposal I think. The only I would like to see is more obvious method names and error messages
thenrio/toto@3903d57#commitcomment-523641

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment