]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/bugs/html5_time_element__39__s_pubdate_wrong_when_using_xhtml5___34__mode__34__.mdwn
respond re pubdate attribute
[git.ikiwiki.info.git] / doc / bugs / html5_time_element__39__s_pubdate_wrong_when_using_xhtml5___34__mode__34__.mdwn
1 Hi,
3 XML error:
5     Created <time datetime="2009-03-24T18:02:14Z" pubdate class="relativedate" title="Tue, 24 Mar 2009 14:02:14 -0400">2009-03-24</time>
7 The pubdate REQUIRES a date, so e.g. `pubdate="2009-03-24T18:02:14Z"`
9 > No, `pubdate="pubdate"`. It's a boolean attribute. applied && [[done]]
10 > --[[Joey]] 
11 >> awesome, thanks for fixing my fix ;) --[[simonraven]]
12 >>> This seems to be happening either still or again with version 3.20200202.3-1. I'm getting strings generated like
13 >>>
14 >>> Posted &lt;time datetime="2007-12-06T05:00:00Z" pubdate="pubdate"&gt;Thu 06 Dec 2007 12:00:00 AM EST&lt;/time&gt;
15 >>>
16 >>> which shows up as an error on https://validator.w3.org/ --Luke Schierer 
18 >>>> My reading of Joey's response, above, was that (according to the spec at the time), `pubdate="pubdate"` is what
19 >>>> should be generated, *not* `pubdate="timestamp"`, and so what you are seeing is expected. However, looking at
20 >>>> the *current* Spec (linked elsewhere in this page), `pubdate` is not actually a valid attribute any more at
21 >>>> all. And indeed, running my own blog through the Validator, I see:
22 >>>>> `Error: Attribute pubdate not allowed on element time at this point.`
23 >>>> *— [[Jon]], 2020-10-05*
27 Otherwise the XML parser chokes.
29 <http://www.whatwg.org/specs/web-apps/current-work/multipage/text-level-semantics.html#attr-time-pubdate>
31 (indented exactly 4 spaces)
33 <pre>
34     diff --git a/IkiWiki.pm b/IkiWiki.pm
35     index 1f2ab07..6ab5b56 100644
36     --- a/IkiWiki.pm
37     +++ b/IkiWiki.pm
38     @@ -1004,7 +1004,7 @@ sub displaytime ($;$$) {
39         my $time=formattime($_[0], $_[1]);
40         if ($config{html5}) {
41             return '&lt;time datetime="'.date_3339($_[0]).'"'.
42     -         ($_[2] ? ' pubdate' : '').
43     +           ($_[2] ? ' pubdate="'.date_3339($_[0]).'"' : '').
44                 '>'.$time.'&lt;/time&gt;';
45         }
46         else {
47     diff --git a/IkiWiki/Plugin/relativedate.pm b/IkiWiki/Plugin/relativedate.pm
48     index fe8ef09..8c4a1b4 100644
49     --- a/IkiWiki/Plugin/relativedate.pm
50     +++ b/IkiWiki/Plugin/relativedate.pm
51     @@ -59,7 +59,7 @@ sub mydisplaytime ($;$$) {
52      
53         if ($config{html5}) {
54             return '&lt;time datetime="'.IkiWiki::date_3339($time).'"'.
55     -         ($pubdate ? ' pubdate' : '').$mid.'&lt;/time&gt;';
56     +           ($pubdate ? ' pubdate="'.IkiWiki::date_3339($time).'"' : '').$mid.'&lt;/time&gt;';
57         }
58         else {
59             return '&lt;span'.$mid.'&lt;/span&gt;';
60 </pre>