minor
[ikiwiki.git] / doc / patchqueue / meta_plugin_should_not_escape_title.mdwn
index 0edc4db6c2e5c9c10f5b31a2ca5a7571c82fd9c8..2cf8cf9b2835d569307cd5175a2439e65b476ff2 100644 (file)
 The meta plugin escapes the overridden title of a page, which makes the title doubly escaped in RSS feeds.
+
+> 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.
 
-       === 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 @@
-        <head>
-        <base href="<TMPL_VAR BASEURL>" />
-        <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
-       -<title><TMPL_VAR TITLE></title>
-       +<title><TMPL_VAR TITLE ESCAPE=HTML></title>
-        <link rel="stylesheet" href="<TMPL_VAR BASEURL>style.css" type="text/css" />
-        <link rel="stylesheet" href="<TMPL_VAR BASEURL>local.css" type="text/css" />
-        <TMPL_IF NAME="FAVICON">
-       @@ -15,7 +15,7 @@
-        
-        <div class="header">
-        <span>
-       -<TMPL_VAR INDEXLINK>/ <span class="title"><TMPL_VAR TITLE></span>
-       +<TMPL_VAR INDEXLINK>/ <span class="title"><TMPL_VAR TITLE ESCAPE=HTML></span>
-        </span>
-        </div>
-        
-       === templates/inlinepage.tmpl
-       ==================================================================
-       --- templates/inlinepage.tmpl   (revision 6730)
-       +++ templates/inlinepage.tmpl   (revision 6731)
-       @@ -10,9 +10,9 @@
-        </TMPL_IF>
-        <span class="header">
-        <TMPL_IF NAME="PERMALINK">
-       -<a href="<TMPL_VAR PERMALINK>"><TMPL_VAR TITLE></a>
-       +<a href="<TMPL_VAR PERMALINK>"><TMPL_VAR TITLE ESCAPE=HTML></a>
-        <TMPL_ELSE>
-       -<a href="<TMPL_VAR PAGEURL>"><TMPL_VAR TITLE></a>
-       +<a href="<TMPL_VAR PAGEURL>"><TMPL_VAR TITLE ESCAPE=HTML></a>
-        </TMPL_IF>
-        </span>
-        <TMPL_VAR CONTENT>
-       === templates/misc.tmpl
-       ==================================================================
-       --- templates/misc.tmpl (revision 6730)
-       +++ templates/misc.tmpl (revision 6731)
-       @@ -4,7 +4,7 @@
-        <head>
-        <base href="<TMPL_VAR BASEURL>" />
-        <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
-       -<title><TMPL_VAR TITLE></title>
-       +<title><TMPL_VAR TITLE ESCAPE=HTML></title>
-        <link rel="stylesheet" href="<TMPL_VAR BASEURL>style.css" type="text/css" />
-        <link rel="stylesheet" href="<TMPL_VAR BASEURL>local.css" type="text/css" />
-        <TMPL_IF NAME="FAVICON">
-       @@ -15,7 +15,7 @@
-        
-        <div class="header">
-        <span>
-       -<TMPL_VAR INDEXLINK>/ <span class="title"><TMPL_VAR TITLE></span>
-       +<TMPL_VAR INDEXLINK>/ <span class="title"><TMPL_VAR TITLE ESCAPE=HTML></span>
-        </span>
-        </div>
-        
-       === templates/page.tmpl
-       ==================================================================
-       --- templates/page.tmpl (revision 6730)
-       +++ templates/page.tmpl (revision 6731)
-       @@ -3,7 +3,7 @@
-        <html>
-        <head>
-        <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
-       -<title><TMPL_VAR TITLE></title>
-       +<title><TMPL_VAR TITLE ESCAPE=HTML></title>
-        <TMPL_IF NAME="FAVICON">
-        <link rel="icon" href="<TMPL_VAR BASEURL><TMPL_VAR FAVICON>" type="image/x-icon" />
-        </TMPL_IF>
-       @@ -18,7 +18,7 @@
-        <span>
-        <span class="parentlinks"><TMPL_LOOP NAME="PARENTLINKS">
-        <a href="<TMPL_VAR NAME=URL>"><TMPL_VAR NAME=PAGE></a>/ 
-       -</TMPL_LOOP></span><span class="title"><TMPL_VAR TITLE></span>
-       +</TMPL_LOOP></span><span class="title"><TMPL_VAR TITLE ESCAPE=HTML></span>
-        </span>
-        <TMPL_IF NAME="SEARCHFORM">
-        <TMPL_VAR SEARCHFORM>
-       === templates/archivepage.tmpl
-       ==================================================================
-       --- templates/archivepage.tmpl  (revision 6730)
-       +++ templates/archivepage.tmpl  (revision 6731)
-       @@ -1,4 +1,4 @@
-        <p>
-        <i><TMPL_VAR CTIME></i>
-       -<a href="<TMPL_VAR PAGEURL>"><TMPL_VAR TITLE></a>
-       +<a href="<TMPL_VAR PAGEURL>"><TMPL_VAR TITLE ESCAPE=HTML></a>
-        </p>
-       === templates/titlepage.tmpl
-       ==================================================================
-       --- templates/titlepage.tmpl    (revision 6730)
-       +++ templates/titlepage.tmpl    (revision 6731)
-       @@ -1 +1 @@
-       -<p><a href="<TMPL_VAR PAGEURL>"><TMPL_VAR TITLE></a></p>
-       +<p><a href="<TMPL_VAR PAGEURL>"><TMPL_VAR TITLE ESCAPE=HTML></a></p>
+> That doesn't work; now nonoverridden titles will be double escaped. And
+> if you look at removing the code that escapes them the first time, things
+> become complex.. --[[Joey]]