aboutsummaryrefslogtreecommitdiff
path: root/doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn
diff options
context:
space:
mode:
authorhttp://smcv.pseudorandom.co.uk/ <smcv@web>2011-06-04 15:40:24 +0000
committeradmin <admin@branchable.com>2011-06-04 15:40:24 +0000
commit118e6f718be1932dfbe553d8a844a6407659cba3 (patch)
treee4364a840462348544ec48945061af40dfcf5be4 /doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn
parentd5a3335e15cc2203dd357e6e55fa9d15bc39a181 (diff)
downloadikiwiki-118e6f718be1932dfbe553d8a844a6407659cba3.tar
ikiwiki-118e6f718be1932dfbe553d8a844a6407659cba3.tar.gz
found it
Diffstat (limited to 'doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn')
-rw-r--r--doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn16
1 files changed, 16 insertions, 0 deletions
diff --git a/doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn b/doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn
index a656a33a0..39c744f9f 100644
--- a/doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn
+++ b/doc/bugs/Comments_are_not_sorted_by_their_date_attribute.mdwn
@@ -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]]