]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/todo/directive_docs.mdwn
Merge branch 'master' into po
[git.ikiwiki.info.git] / doc / todo / directive_docs.mdwn
1 The current basewiki is not [[self-documenting|todo/basewiki_should_be_self_documenting]]. In particular, if
2 [[plugins/listdirectives]] is used, it creates a list with a bunch of
3 broken links to directives/*, pages that do not currently exist in the
4 docwiki or basewiki.
6 This could be fixed by adding a page for each directive under to
7 `ikiwiki/directives`, and put those into a new underlay, which the plugin
8 could enable. Rather a lot of work and maintenance to document all the
9 directives like that.
11 I also considered having it link to the plugin that defined the
12 directive. Then all the plugins can be included in a new underlay, which
13 both [[plugins/listdirectives]] and [[plugins/websetup]] could enable.
14 (The latter could be improved by making the plugin names in the web setup
15 be links to docs about each plugin..) 
17 The problem I ran into doing that is that the existing plugin pages have a
18 lot of stuff on them you probably don't want an underlay doing. The biggest
19 issues were wikilinks to other pages in the docwiki (which would end up
20 broken if the plugins were used as an underlay), and plugin pages that
21 include examples of the plugin in use, which are sometimes rather expensive
22 (eg, brokenlinks).
24 Either way requires a lot of reorganisation/doc work, and an onging
25 maintenance load.
27 > Which has now been [[done]].  -- [[Will]]
29 BTW, this patch would be needed for the second approach, to allow
30 listdirectives to map from preprocessor directives back to the plugin that
31 defined them:  --[[Joey]]
33     commit 0486b46a629cae19ce89492d5ac498bbf9b84f5f
34     Author: Joey Hess <joey@kodama.kitenet.net>
35     Date:   Mon Aug 25 15:38:51 2008 -0400
36     
37         record which plugins registered which hooks
38     
39     diff --git a/IkiWiki.pm b/IkiWiki.pm
40     index e476521..afe982a 100644
41     --- a/IkiWiki.pm
42     +++ b/IkiWiki.pm
43     @@ -493,6 +493,7 @@ sub loadplugins () {
44         return 1;
45      }
46      
47     +my $loading_plugin;
48      sub loadplugin ($) {
49         my $plugin=shift;
50      
51     @@ -502,14 +503,18 @@ sub loadplugin ($) {
52                          "$installdir/lib/ikiwiki") {
53                 if (defined $dir && -x "$dir/plugins/$plugin") {
54                         require IkiWiki::Plugin::external;
55     +                   $loading_plugin=$plugin;
56                         import IkiWiki::Plugin::external "$dir/plugins/$plugin";
57     +                   $loading_plugin=undef;
58                         $loaded_plugins{$plugin}=1;
59                         return 1;
60                 }
61         }
62      
63         my $mod="IkiWiki::Plugin::".possibly_foolish_untaint($plugin);
64     +   $loading_plugin=$plugin;
65         eval qq{use $mod};
66     +   $loading_plugin=undef;
67         if ($@) {
68                 error("Failed to load plugin $mod: $@");
69         }
70     @@ -1429,6 +1434,9 @@ sub hook (@) {
71      
72         return if $param{no_override} && exists $hooks{$param{type}}{$param{id}};
73         
74     +   # Record which plugin was being loaded when the hook was defined.
75     +   $param{plugin}=$loading_plugin if defined $loading_plugin;
76     +
77         $hooks{$param{type}}{$param{id}}=\%param;
78         return 1;
79      }