]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/todo/turn_edittemplate_verbosity_off_by_default.mdwn
optimize for common case where list is not changed
[git.ikiwiki.info.git] / doc / todo / turn_edittemplate_verbosity_off_by_default.mdwn
1 `edittemplate` replaces its directive with a note like "edittemplate person
2 registered for people/*". It would be nice if this were dependent on
3 a `verbose` parameter and default to off. I don't see the value in it, and by
4 disabling the output, I can keep template registration as close as possible to
5 the action as needed.
7 I think this (untested) patch might just do the trick:
9     --- a/IkiWiki/Plugin/edittemplate.pm
10     +++ b/IkiWiki/Plugin/edittemplate.pm
11     @@ -46,8 +46,13 @@ sub preprocess (@) {
13       $pagestate{$params{page}}{edittemplate}{$params{match}}=$params{template};
15     -       return sprintf(gettext("edittemplate %s registered for %s"),
16     -               $params{template}, $params{match});
17     +       if (yesno($params{verbose})) {
18     +               return sprintf(gettext("edittemplate %s registered for %s"),
19     +                       $params{template}, $params{match});
20     +       }
21     +       else {
22     +               return '';
23     +       }
24     }
26     sub formbuilder (@) {
28 --[[madduck]]
30 [[!tags wishlist patch]]
32 [[done]], though the patch I eventually applied uses "silent" as the
33 parameter name. Sorry for forgetting about this patch until someone else
34 implemented it too. --[[Joey]]