modify patch based on feedback
authorhttp://www.cse.unsw.edu.au/~willu/ <http://www.cse.unsw.edu.au/~willu/@web>
Mon, 25 Aug 2008 05:50:27 +0000 (01:50 -0400)
committerJoey Hess <joey@kitenet.net>
Mon, 25 Aug 2008 05:50:27 +0000 (01:50 -0400)
doc/todo/Add_a_plugin_to_list_available_pre-processor_commands.mdwn

index e311addcddd37f7d0306f43c9eebb0cff1072fe4..0524627d45ed6e91d03116af52629bb21fc3213b 100644 (file)
@@ -135,7 +135,7 @@ I've found myself wanting to know which [[plugins]] are switched on so I know wh
 >>>>>>>> Only code change I'd suggest is using `htmllink` rather than 
 >>>>>>>> generating a wikilink.
 
->>>>>>>>> Yeah - that would make sense.  Will do. -- [[Will]]
+>>>>>>>>> Yeah - that would make sense.  done. -- [[Will]]
 
     #!/usr/bin/perl
     # Ikiwiki listpreprocessors plugin.
@@ -207,7 +207,7 @@ I've found myself wanting to know which [[plugins]] are switched on so I know wh
        
        my @pluginlist;
        
-       if (defined $params{generated}) {
+       if (defined $params{generated}) {
                @pluginlist = @fullPluginList;
        } else {
                @pluginlist = @earlyPluginList;
@@ -216,13 +216,14 @@ I've found myself wanting to know which [[plugins]] are switched on so I know wh
        my $result = '<ul class="listpreprocessors">';
        
        foreach my $plugin (@pluginlist) {
-               $result .= '<li class="listpreprocessors">[[' . $config{plugin_description_dir} . $plugin . ']]</li>';
+               $result .= '<li class="listpreprocessors">';
+               $result .= htmllink($params{page}, $params{destpage}, IkiWiki::linkpage($config{plugin_description_dir} . $plugin));
+               $result .= '</li>';
        }
        
        $result .= "</ul>";
-       
-       return IkiWiki::preprocess($params{page}, $params{destpage}, 
-               IkiWiki::filter($params{page}, $params{destpage}, $result));
+    
+       return $result;
     } # }}}
     
     1