Close meta tag for redir properly.
authorJoey Hess <joey@kodama.kitenet.net>
Fri, 21 Mar 2008 04:24:06 +0000 (00:24 -0400)
committerJoey Hess <joey@kodama.kitenet.net>
Fri, 21 Mar 2008 04:24:06 +0000 (00:24 -0400)
IkiWiki/Plugin/meta.pm
debian/changelog
doc/bugs/RecentChanges_contains_invalid_XHTML.mdwn

index 8a9a5382b00823ffdcd2825f5db6ae9214490480..0afe1c3622561654bf682df6ea2e6d5b9551f135 100644 (file)
@@ -186,7 +186,7 @@ sub preprocess (@) { #{{{
                        $value=encode_entities($value);
                }
                my $delay=int(exists $params{delay} ? $params{delay} : 0);
                        $value=encode_entities($value);
                }
                my $delay=int(exists $params{delay} ? $params{delay} : 0);
-               my $redir="<meta http-equiv=\"refresh\" content=\"$delay; URL=$value\">";
+               my $redir="<meta http-equiv=\"refresh\" content=\"$delay; URL=$value\" />";
                if (! $safe) {
                        $redir=scrub($redir);
                }
                if (! $safe) {
                        $redir=scrub($redir);
                }
index 5909f237e2ad24cafa62f7821d6d8ff2f49a9d0d..ef16cbfaf8c240e01e1158af9b08b185376cdda0 100644 (file)
@@ -54,6 +54,7 @@ ikiwiki (2.41) UNRELEASED; urgency=low
   * Time::Duration is no longer used, remove from docs and recommends.
   * Store userinfo in network byte order for easy portability.
     (Old files will be automatically converted.)
   * Time::Duration is no longer used, remove from docs and recommends.
   * Store userinfo in network byte order for easy portability.
     (Old files will be automatically converted.)
+  * Close meta tag for redir properly.
 
  -- martin f. krafft <madduck@debian.org>  Sun, 02 Mar 2008 17:46:38 +0100
 
 
  -- martin f. krafft <madduck@debian.org>  Sun, 02 Mar 2008 17:46:38 +0100
 
index 4d65066570245d35a73b5834f9a2cdac9f9f418c..fe536676071fce26f3d5d0a61322daa80d219261 100644 (file)
@@ -4,11 +4,17 @@ the end of the [[RecentChanges]] page
 
     <p></div></p>
 
 
     <p></div></p>
 
+> I'll bet this is fixed if you use the markdown 1.2 prerelease, which has
+> a much less buggy html parser. (Ah, I see below that was the case.) 
+> --[[Joey]]
+
 Also, there is a problem with the `<img>` tags generated by the smiley
 plugin which end up wrapped in a `<pre>` tag in the inline diff output.
 `<img>` tags is not allowed within a `<pre>` block.  Maybe the smiley
 plugin should be disabled on [[RecentChanges]]?
 
 Also, there is a problem with the `<img>` tags generated by the smiley
 plugin which end up wrapped in a `<pre>` tag in the inline diff output.
 `<img>` tags is not allowed within a `<pre>` block.  Maybe the smiley
 plugin should be disabled on [[RecentChanges]]?
 
+> See [[Smileys_in_the_block_code]] --[[Joey]]
+
 See the [validator output][validate] for more details.
 
  [tmpl]: http://git.ikiwiki.info/?p=ikiwiki;a=blob_plain;f=templates/recentchanges.tmpl;hb=HEAD
 See the [validator output][validate] for more details.
 
  [tmpl]: http://git.ikiwiki.info/?p=ikiwiki;a=blob_plain;f=templates/recentchanges.tmpl;hb=HEAD
@@ -31,24 +37,10 @@ processing and throw an error. &mdash;[Jason Blevins](http://jblevins.org/)
 Here is a simple patch for the refresh problem.  I haven't figured out
 what's causing the recentchanges bug yet.
 
 Here is a simple patch for the refresh problem.  I haven't figured out
 what's causing the recentchanges bug yet.
 
-
-    diff --git a/IkiWiki/Plugin/meta.pm b/IkiWiki/Plugin/meta.pm
-    index 8a9a538..0afe1c3 100644
-    --- a/IkiWiki/Plugin/meta.pm
-    +++ b/IkiWiki/Plugin/meta.pm
-    @@ -186,7 +186,7 @@ sub preprocess (@) { #{{{
-                       $value=encode_entities($value);
-               }
-               my $delay=int(exists $params{delay} ? $params{delay} : 0);
-    -          my $redir="<meta http-equiv=\"refresh\" content=\"$delay; URL=$value\">";
-    +          my $redir="<meta http-equiv=\"refresh\" content=\"$delay; URL=$value\" />";
-               if (! $safe) {
-                       $redir=scrub($redir);
-               }
-
-
 --[[JasonBlevins]]
 
 --[[JasonBlevins]]
 
+> Thanks, applied that patch. --[[Joey]]
+
 - - -
 
 It turns out that the invalid XHTML on the recent changes page is due to
 - - -
 
 It turns out that the invalid XHTML on the recent changes page is due to