found it
authorhttp://smcv.pseudorandom.co.uk/ <smcv@web>
Sat, 4 Jun 2011 15:40:24 +0000 (15:40 +0000)
committeradmin <admin@branchable.com>
Sat, 4 Jun 2011 15:40:24 +0000 (15:40 +0000)
doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn

index a656a33a0386269804e7c0507014f5b3959a5892..39c744f9fc1fcc44b7dc82b142c04b9e960f1244 100644 (file)
@@ -46,3 +46,19 @@ The presentation of the resulting comments is not sorted by this date, which I w
 >>> Sorting by filename would only be useful with
 >>> [[!cpan Sort::Naturally]], since normal `cmp` ordering would break pages
 >>> with more than 9 comments. --s
+
+>>> OK, I've found the bug. As internal pages, comments are not preprocessed
+>>> (and so their date attributes do not have a chance to take effect) until
+>>> they are already being inlined, by which time they have already been
+>>> sorted by the files' ctimes. Possible fixes include:
+>>>
+>>> * scan any modified/created internal pages (including preprocessing them
+>>>   in scan mode) before rendering any page, fixing all possible bugs of this
+>>>   form but causing a slower scan phase
+>>>
+>>> * list and scan the comments below a given page in its `pagetemplate`
+>>>   hook before inlining them, causing a slower build phase for pages with
+>>>   many comments
+>>>
+>>> I think the former is a more general fix, and I'll try to implement
+>>> it. --[[smcv]]