Added "Reply to" field to mail header

mpalmer / jekyll-static-comments

A plugin for Jekyll to implement a static-file based comments system

http://theshed.hezmatt.org/jekyll-static-comments/

GNU General Public License v3.0

195 stars 33 forks source link

Added "Reply to" field to mail header #8

Open IQAndreas opened 12 years ago

IQAndreas commented 12 years ago

This fix allows blog owners to reply to the user's comment by replying to their email (if they included an email address in the comment) if they want a private reply rather than a reply as a comment on the blog.

Second version. This time I use the recommended header cleaning function from http://stackoverflow.com/questions/8071916/escape-string-to-use-in-mail

mpalmer commented 12 years ago

My comments from #6 regarding 8e6b17b are appropriate here also. The brace placement on c35c45e is... unique. Far better to stick with the existing style, though.

IQAndreas commented 12 years ago

Third version. Does this look better?

I switched to a different cleaning function for mail input which seems to suit the purpose much better. http://mattgeri.com/blog/2012/01/escaping-input-to-the-php-mail-function/