]> sipb.mit.edu Git - ikiwiki.git/commitdiff
comments on templatebody and phases
authorchrysn <chrysn@fsfe.org>
Sat, 5 Jul 2014 17:11:47 +0000 (19:11 +0200)
committerchrysn <chrysn@fsfe.org>
Sat, 5 Jul 2014 17:11:47 +0000 (19:11 +0200)
doc/bugs/template_creation_error.mdwn

index 98d1cf2353c874841c51d60a2eddbd452e2edad1..d3c0bcca5c9fcdbe2539c86e164faa9da7875ebc 100644 (file)
@@ -255,3 +255,10 @@ same logic as IkiWiki itself. I don't think that's serious. --[[smcv]]
 >>> is a start towards that; the docwiki builds successfully, but
 >>> the tests that use IkiWiki internals also need updating to
 >>> set `$phase = PHASE_RENDER` before they start preprocessing. --s
+
+>>>> reviewing those modifications, i think this is a good way to go. along
+>>>> with warning about pagespecs evaluated in scan phase, i think it should be
+>>>> an error to invoke scan in the render phase; that would mean that
+>>>> `readtemplate` needs to check whether it's invoked as a scan or not to
+>>>> decide whether to scan the template page, but would be generally more
+>>>> robust for future plugin writing. --[[chrysn]]