From d8b70fa39b371739b90fa1c907718f39315ad6e1 Mon Sep 17 00:00:00 2001 From: chrysn Date: Sun, 13 Jul 2014 16:53:27 +0200 Subject: [PATCH] phases are a good idea --- doc/bugs/template_creation_error.mdwn | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/doc/bugs/template_creation_error.mdwn b/doc/bugs/template_creation_error.mdwn index d3c0bcca5..d1fb788f5 100644 --- a/doc/bugs/template_creation_error.mdwn +++ b/doc/bugs/template_creation_error.mdwn @@ -261,4 +261,10 @@ same logic as IkiWiki itself. I don't think that's serious. --[[smcv]] >>>> 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]] +>>>> robust for future plugin writing. +>>>> +>>>> **addendum**: if the new phase state is used to create warnings/errors +>>>> about improper ikiwiki api use of plugins (which is something i'd +>>>> advocate), that should likewise warn if `add_link` actually adds a link in +>>>> the render phase. such a warning would have helped spotting the +>>>> link-related [[template evaluation oddities]] earlier. --[[chrysn]] -- 2.39.5