Report bug (and patch) with conditional preprocessing and scan
[ikiwiki] / doc / bugs / templateForRecentChangesMissingCloseSpan.mdwn
1 In the template for ikiwiki's recent changes page
2
3      /usr/share/ikiwiki/templates/change.tmpl
4
5 there is a missing </span> tag after the
6
7      <span class="changedate"><TMPL_VAR COMMITDATE>
8
9 This results in the recentchanges/ page being invalid and rendering quite horrifyingly in Internet Exploder.
10
11 [I'm running](http://wiki.shlrm.org) (linked so you can see the one I'm running if you need to) the latest version of ikiwiki, and I note that it's broken on [ikiwiki.info](http://validator.w3.org/check?uri=http%3A%2F%2Fikiwiki.info%2Frecentchanges%2F&charset=%28detect+automatically%29&doctype=Inline&group=0&user-agent=W3C_Validator%2F1.767) too :)
12
13 [This one on debian](https://www.icanttype.org/recentchanges/) is somehow [valid](http://validator.w3.org/check?uri=https%3A%2F%2Fwww.icanttype.org%2F%2Frecentchanges%2F&charset=%28detect+automatically%29&doctype=Inline&group=0&user-agent=W3C_Validator%2F1.767), although it's using the same template. Perhaps there's an additional scrubbing going on his end.
14
15 Thanks,
16 David
17
18 PS: I have fixed the template by hand on my server, so it will validate, however ikiwiki.info will not.
19
20 > [[!template id="gitbranch" branch=smcv/trivia author="[[smcv]]"]] [[!tag patch]]
21 > Enabling either [[plugins/htmltidy]] or [[plugins/htmlbalance]] will automatically fix unbalanced
22 > markup like this; using [[plugins/comments]] without having one or other of those is a bad idea
23 > from the point of view of avoiding comment forgery, which is probably why icanttype.org works
24 > correctly. Anyway, I've fixed this in a branch: Joey, care to review smcv/trivia? --[[smcv]]
25
26 [[done]], thanks guys --[[Joey]]