diff --git a/doc/patchqueue/meta_plugin_should_not_escape_title.mdwn b/doc/patchqueue/meta_plugin_should_not_escape_title.mdwn index fa7e26d77..2cf8cf9b2 100644 --- a/doc/patchqueue/meta_plugin_should_not_escape_title.mdwn +++ b/doc/patchqueue/meta_plugin_should_not_escape_title.mdwn @@ -2,121 +2,10 @@ The meta plugin escapes the overridden title of a page, which makes the title do > Er, didn't that already get fixed in svn? +>> Right, sorry, I should have seen [[bugs/HTML-escaped_titles_in_Atom__44___RSS_feeds_don__39__t_validate]]. + Instead, the meta plugin should not escape the overridden title, but the templates should escape the title (overridden or not) whenever needed. > That doesn't work; now nonoverridden titles will be double escaped. And -> if you look at removing the code that esapes them the first time, things +> if you look at removing the code that escapes them the first time, things > become complex.. --[[Joey]] - - === perl/IkiWiki/Plugin/meta.pm - ================================================================== - --- perl/IkiWiki/Plugin/meta.pm (revision 6730) - +++ perl/IkiWiki/Plugin/meta.pm (revision 6731) - @@ -122,7 +122,7 @@ - $template->param(meta => $meta{$page}) - if exists $meta{$page} && $template->query(name => "meta"); - if (exists $title{$page} && $template->query(name => "title")) { - - $template->param(title => $title{$page}); - + $template->param(title => decode_entities($title{$page})); - $template->param(title_overridden => 1); - } - $template->param(permalink => $permalink{$page}) - === templates/recentchanges.tmpl - ================================================================== - --- templates/recentchanges.tmpl (revision 6730) - +++ templates/recentchanges.tmpl (revision 6731) - @@ -4,7 +4,7 @@ -
-