]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/commitdiff
* Work on firming up the plugin interface:
authorjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Sat, 9 Sep 2006 22:50:27 +0000 (22:50 +0000)
committerjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Sat, 9 Sep 2006 22:50:27 +0000 (22:50 +0000)
  - Plugins should not need to load IkiWiki::Render to get commonly
    used functions, so moved some functions from there to IkiWiki.
  - Picked out the set of functions and variables that most plugins
    use, documented them, and made IkiWiki export them by default,
    like a proper perl module should.
  - Use the other functions at your own risk.
  - This is not quite complete, I still have to decide whether to
    export some other things.
* Changed all plugins included in ikiwiki to not use "IkiWiki::" when
  referring to stuff now exported by the IkiWiki module.
* Anyone with a third-party ikiwiki plugin is strongly enrouraged
  to make like changes to it and avoid use of non-exported symboles from
  "IkiWiki::".
* Link debian/changelog and debian/news to NEWS and CHANGELOG.
* Support hyperestradier version 1.4.2, which adds a new required phraseform
  setting.

48 files changed:
CHANGELOG [new symlink]
IkiWiki.pm
IkiWiki/CGI.pm
IkiWiki/Plugin/aggregate.pm
IkiWiki/Plugin/brokenlinks.pm
IkiWiki/Plugin/camelcase.pm
IkiWiki/Plugin/ddate.pm
IkiWiki/Plugin/fortune.pm
IkiWiki/Plugin/googlecalendar.pm
IkiWiki/Plugin/haiku.pm
IkiWiki/Plugin/html.pm
IkiWiki/Plugin/htmlscrubber.pm
IkiWiki/Plugin/htmltidy.pm
IkiWiki/Plugin/inline.pm
IkiWiki/Plugin/linkmap.pm
IkiWiki/Plugin/map.pm
IkiWiki/Plugin/mdwn.pm
IkiWiki/Plugin/meta.pm
IkiWiki/Plugin/orphans.pm
IkiWiki/Plugin/otl.pm
IkiWiki/Plugin/pagecount.pm
IkiWiki/Plugin/pagestats.pm
IkiWiki/Plugin/polygen.pm
IkiWiki/Plugin/rst.pm
IkiWiki/Plugin/search.pm
IkiWiki/Plugin/sidebar.pm
IkiWiki/Plugin/skeleton.pm
IkiWiki/Plugin/smiley.pm
IkiWiki/Plugin/tag.pm
IkiWiki/Plugin/template.pm
IkiWiki/Plugin/toc.pm
IkiWiki/Plugin/wikitext.pm
IkiWiki/Rcs/git.pm
IkiWiki/Render.pm
NEWS [new symlink]
debian/changelog
doc/plugins/write.mdwn
doc/roadmap.mdwn
doc/todo/firm_up_plugin_interface.mdwn [new file with mode: 0644]
t/bestlink.t
t/crazy-badass-perl-bug.t
t/html.t
t/htmlize.t
t/linkify.t
t/pagespec_match.t
t/pagespec_merge.t
t/readfile.t
templates/estseek.conf

diff --git a/CHANGELOG b/CHANGELOG
new file mode 120000 (symlink)
index 0000000..d526672
--- /dev/null
+++ b/CHANGELOG
@@ -0,0 +1 @@
+debian/changelog
\ No newline at end of file
index 990836f8e461226442933064d2d62defb457e27d..1cbe975c058cc7a455dc4127067d1bf943776651 100644 (file)
@@ -7,14 +7,19 @@ use Encode;
 use HTML::Entities;
 use open qw{:utf8 :std};
 
+use vars qw{%config %links %oldlinks %oldpagemtime %pagectime %pagecase
+            %renderedfiles %pagesources %depends %hooks %forcerebuild};
+
+use Exporter q{import};
+our @EXPORT = qw(hook debug error template htmlpage add_depends pagespec_match
+                 bestlink htmllink readfile writefile pagetype srcfile pagename
+                 %config %links %renderedfiles %pagesources);
+
 # Optimisation.
 use Memoize;
 memoize("abs2rel");
 memoize("pagespec_translate");
 
-use vars qw{%config %links %oldlinks %oldpagemtime %pagectime %pagecase
-            %renderedfiles %pagesources %depends %hooks %forcerebuild};
-
 my $installdir=''; # INSTALLDIR_AUTOREPLACE done by Makefile, DNE
 
 sub defaultconfig () { #{{{
@@ -254,11 +259,6 @@ sub writefile ($$$;$) { #{{{
 } #}}}
 
 sub bestlink ($$) { #{{{
-       # Given a page and the text of a link on the page, determine which
-       # existing page that link best points to. Prefers pages under a
-       # subdirectory with the same name as the source page, failing that
-       # goes down the directory tree to the base looking for matching
-       # pages.
        my $page=shift;
        my $link=shift;
        
@@ -329,6 +329,16 @@ sub abs2rel ($$) { #{{{
        return $ret;
 } #}}}
 
+sub displaytime ($) { #{{{
+       my $time=shift;
+
+       eval q{use POSIX};
+       # strftime doesn't know about encodings, so make sure
+       # its output is properly treated as utf8
+       return decode_utf8(POSIX::strftime(
+                       $config{timeformat}, localtime($time)));
+} #}}}
+
 sub htmllink ($$$;$$$) { #{{{
        my $lpage=shift; # the page doing the linking
        my $page=shift; # the page that will contain the link (different for inline)
@@ -370,6 +380,117 @@ sub htmllink ($$$;$$$) { #{{{
        return "<a href=\"$bestlink\">$linktext</a>";
 } #}}}
 
+sub htmlize ($$$) { #{{{
+       my $page=shift;
+       my $type=shift;
+       my $content=shift;
+
+       if (exists $hooks{htmlize}{$type}) {
+               $content=$hooks{htmlize}{$type}{call}->(
+                       page => $page,
+                       content => $content,
+               );
+       }
+       else {
+               error("htmlization of $type not supported");
+       }
+
+       run_hooks(sanitize => sub {
+               $content=shift->(
+                       page => $page,
+                       content => $content,
+               );
+       });
+
+       return $content;
+} #}}}
+
+sub linkify ($$$) { #{{{
+       my $lpage=shift; # the page containing the links
+       my $page=shift; # the page the link will end up on (different for inline)
+       my $content=shift;
+
+       $content =~ s{(\\?)$config{wiki_link_regexp}}{
+               $2 ? ( $1 ? "[[$2|$3]]" : htmllink($lpage, $page, titlepage($3), 0, 0, pagetitle($2)))
+                  : ( $1 ? "[[$3]]" :    htmllink($lpage, $page, titlepage($3)))
+       }eg;
+       
+       return $content;
+} #}}}
+
+my %preprocessing;
+sub preprocess ($$$) { #{{{
+       my $page=shift; # the page the data comes from
+       my $destpage=shift; # the page the data will appear in (different for inline)
+       my $content=shift;
+
+       my $handle=sub {
+               my $escape=shift;
+               my $command=shift;
+               my $params=shift;
+               if (length $escape) {
+                       return "[[$command $params]]";
+               }
+               elsif (exists $hooks{preprocess}{$command}) {
+                       # Note: preserve order of params, some plugins may
+                       # consider it significant.
+                       my @params;
+                       while ($params =~ /(?:(\w+)=)?(?:"""(.*?)"""|"([^"]+)"|(\S+))(?:\s+|$)/sg) {
+                               my $key=$1;
+                               my $val;
+                               if (defined $2) {
+                                       $val=$2;
+                                       $val=~s/\r\n/\n/mg;
+                                       $val=~s/^\n+//g;
+                                       $val=~s/\n+$//g;
+                               }
+                               elsif (defined $3) {
+                                       $val=$3;
+                               }
+                               elsif (defined $4) {
+                                       $val=$4;
+                               }
+
+                               if (defined $key) {
+                                       push @params, $key, $val;
+                               }
+                               else {
+                                       push @params, $val, '';
+                               }
+                       }
+                       if ($preprocessing{$page}++ > 3) {
+                               # Avoid loops of preprocessed pages preprocessing
+                               # other pages that preprocess them, etc.
+                               return "[[$command preprocessing loop detected on $page at depth $preprocessing{$page}]]";
+                       }
+                       my $ret=$hooks{preprocess}{$command}{call}->(
+                               @params,
+                               page => $page,
+                               destpage => $destpage,
+                       );
+                       $preprocessing{$page}--;
+                       return $ret;
+               }
+               else {
+                       return "[[$command $params]]";
+               }
+       };
+       
+       $content =~ s{(\\?)\[\[(\w+)\s+((?:(?:\w+=)?(?:""".*?"""|"[^"]+"|[^\s\]]+)\s*)*)\]\]}{$handle->($1, $2, $3)}seg;
+       return $content;
+} #}}}
+
+sub filter ($$) {
+       my $page=shift;
+       my $content=shift;
+
+       run_hooks(filter => sub {
+               $content=shift->(page => $page, content => $content);
+       });
+
+       return $content;
+}
+
 sub indexlink () { #{{{
        return "<a href=\"$config{url}\">$config{wikiname}</a>";
 } #}}}
@@ -478,7 +599,7 @@ sub misctemplate ($$) { #{{{
                indexlink => indexlink(),
                wikiname => $config{wikiname},
                pagebody => $pagebody,
-               baseurl => baseurl(),
+       baseurl => baseurl(),
        );
        return $template->output;
 }#}}}
@@ -594,6 +715,18 @@ sub pagespec_translate ($) { #{{{
        return $code;
 } #}}}
 
+sub add_depends ($$) { #{{{
+       my $page=shift;
+       my $pagespec=shift;
+       
+       if (! exists $depends{$page}) {
+               $depends{$page}=$pagespec;
+       }
+       else {
+               $depends{$page}=pagespec_merge($depends{$page}, $pagespec);
+       }
+} # }}}
+
 sub pagespec_match ($$) { #{{{
        my $page=shift;
        my $spec=shift;
index eb89752ec2084a03e522c6f94c3001e07af0e925..074971be1d19e4710ba953ab39003fab1de96094 100644 (file)
@@ -431,7 +431,6 @@ sub cgi_editpage ($$) { #{{{
                return;
        }
        elsif ($form->submitted eq "Preview") {
-               require IkiWiki::Render;
                my $content=$form->field('editcontent');
                my $comments=$form->field('comments');
                $form->field(name => "editcontent",
index 22f46aac0e47dcced33fe08ee67e660423d570db..ac55dcb1daa5ec5daf959929e97bd6d614625819 100644 (file)
@@ -14,30 +14,24 @@ my %feeds;
 my %guids;
 
 sub import { #{{{
-       IkiWiki::hook(type => "getopt", id => "aggregate", 
-               call => \&getopt);
-       IkiWiki::hook(type => "checkconfig", id => "aggregate",
-               call => \&checkconfig);
-       IkiWiki::hook(type => "filter", id => "aggregate", 
-               call => \&filter);
-       IkiWiki::hook(type => "preprocess", id => "aggregate",
-               call => \&preprocess);
-        IkiWiki::hook(type => "delete", id => "aggregate",
-                call => \&delete);
-       IkiWiki::hook(type => "savestate", id => "aggregate",
-               call => \&savestate);
+       hook(type => "getopt", id => "aggregate", call => \&getopt);
+       hook(type => "checkconfig", id => "aggregate", call => \&checkconfig);
+       hook(type => "filter", id => "aggregate", call => \&filter);
+       hook(type => "preprocess", id => "aggregate", call => \&preprocess);
+        hook(type => "delete", id => "aggregate", call => \&delete);
+       hook(type => "savestate", id => "aggregate", call => \&savestate);
 } # }}}
 
 sub getopt () { #{{{
         eval q{use Getopt::Long};
         Getopt::Long::Configure('pass_through');
-        GetOptions("aggregate" => \$IkiWiki::config{aggregate});
+        GetOptions("aggregate" => \$config{aggregate});
 } #}}}
 
 sub checkconfig () { #{{{
        IkiWiki::lockwiki();
        loadstate();
-       if ($IkiWiki::config{aggregate}) {
+       if ($config{aggregate}) {
                IkiWiki::loadindex();
                aggregate();
                savestate();
@@ -78,7 +72,7 @@ sub preprocess (@) { #{{{
        $feed->{url}=$params{url};
        my $dir=exists $params{dir} ? $params{dir} : $params{page}."/".IkiWiki::titlepage($params{name});
        $dir=~s/^\/+//;
-       ($dir)=$dir=~/$IkiWiki::config{wiki_file_regexp}/;
+       ($dir)=$dir=~/$config{wiki_file_regexp}/;
        $feed->{dir}=$dir;
        $feed->{feedurl}=defined $params{feedurl} ? $params{feedurl} : "";
        $feed->{updateinterval}=defined $params{updateinterval} ? $params{updateinterval} * 60 : 15 * 60;
@@ -109,15 +103,15 @@ sub delete (@) { #{{{
 
        # Remove feed data for removed pages.
        foreach my $file (@files) {
-               my $page=IkiWiki::pagename($file);
+               my $page=pagename($file);
                remove_feeds($page);
        }
 } #}}}
 
 sub loadstate () { #{{{
-       if (-e "$IkiWiki::config{wikistatedir}/aggregate") {
-               open (IN, "$IkiWiki::config{wikistatedir}/aggregate" ||
-                       die "$IkiWiki::config{wikistatedir}/aggregate: $!");
+       if (-e "$config{wikistatedir}/aggregate") {
+               open (IN, "$config{wikistatedir}/aggregate" ||
+                       die "$config{wikistatedir}/aggregate: $!");
                while (<IN>) {
                        $_=IkiWiki::possibly_foolish_untaint($_);
                        chomp;
@@ -151,8 +145,8 @@ sub loadstate () { #{{{
 sub savestate () { #{{{
        eval q{use HTML::Entities};
        die $@ if $@;
-       open (OUT, ">$IkiWiki::config{wikistatedir}/aggregate" ||
-               die "$IkiWiki::config{wikistatedir}/aggregate: $!");
+       open (OUT, ">$config{wikistatedir}/aggregate" ||
+               die "$config{wikistatedir}/aggregate: $!");
        foreach my $data (values %feeds, values %guids) {
                if ($data->{remove}) {
                        if ($data->{name}) {
@@ -193,19 +187,19 @@ sub aggregate () { #{{{
        die $@ if $@;
 
        foreach my $feed (values %feeds) {
-               next unless $IkiWiki::config{rebuild} || 
+               next unless $config{rebuild} || 
                        time - $feed->{lastupdate} >= $feed->{updateinterval};
                $feed->{lastupdate}=time;
                $feed->{newposts}=0;
                $IkiWiki::forcerebuild{$feed->{sourcepage}}=1;
 
-               IkiWiki::debug("checking feed ".$feed->{name}." ...");
+               debug("checking feed ".$feed->{name}." ...");
 
                if (! length $feed->{feedurl}) {
                        my @urls=XML::Feed->find_feeds($feed->{url});
                        if (! @urls) {
                                $feed->{message}="could not find feed at ".$feed->{feedurl};
-                               IkiWiki::debug($feed->{message});
+                               debug($feed->{message});
                                next;
                        }
                        $feed->{feedurl}=pop @urls;
@@ -213,12 +207,12 @@ sub aggregate () { #{{{
                my $f=eval{XML::Feed->parse(URI->new($feed->{feedurl}))};
                if ($@) {
                        $feed->{message}="feed crashed XML::Feed! $@";
-                       IkiWiki::debug($feed->{message});
+                       debug($feed->{message});
                        next;
                }
                if (! $f) {
                        $feed->{message}=XML::Feed->errstr;
-                       IkiWiki::debug($feed->{message});
+                       debug($feed->{message});
                        next;
                }
 
@@ -234,7 +228,7 @@ sub aggregate () { #{{{
                }
 
                $feed->{message}="processed ok at ".
-                       IkiWiki::displaytime($feed->{lastupdate});
+                       displaytime($feed->{lastupdate});
        }
 
        # TODO: expiry
@@ -264,7 +258,7 @@ sub add_page (@) { #{{{
                # directory name or trigger ".." disallowing code.
                $page=~s!([/.])!"__".ord($1)."__"!eg;
                $page=$feed->{dir}."/".$page;
-               ($page)=$page=~/$IkiWiki::config{wiki_file_regexp}/;
+               ($page)=$page=~/$config{wiki_file_regexp}/;
                if (! defined $page || ! length $page) {
                        $page=$feed->{dir}."/item";
                }
@@ -274,7 +268,7 @@ sub add_page (@) { #{{{
                        $c++
                }
                $guid->{page}=$page;
-               IkiWiki::debug("creating new page $page");
+               debug("creating new page $page");
        }
        $guid->{feed}=$feed->{name};
        
@@ -284,11 +278,11 @@ sub add_page (@) { #{{{
        eval q{use Digest::MD5 'md5_hex'};
        require Encode;
        my $digest=md5_hex(Encode::encode_utf8($params{content}));
-       return unless ! exists $guid->{md5} || $guid->{md5} ne $digest || $IkiWiki::config{rebuild};
+       return unless ! exists $guid->{md5} || $guid->{md5} ne $digest || $config{rebuild};
        $guid->{md5}=$digest;
 
        # Create the page.
-       my $template=IkiWiki::template("aggregatepost.tmpl", blind_cache => 1);
+       my $template=template("aggregatepost.tmpl", blind_cache => 1);
        $template->param(title => $params{title})
                if defined $params{title} && length($params{title});
        $template->param(content => htmlescape(htmlabs($params{content}, $feed->{feedurl})));
@@ -299,7 +293,7 @@ sub add_page (@) { #{{{
        if (ref $feed->{tags}) {
                $template->param(tags => [map { tag => $_ }, @{$feed->{tags}}]);
        }
-       IkiWiki::writefile(IkiWiki::htmlpage($guid->{page}), $IkiWiki::config{srcdir},
+       writefile(htmlpage($guid->{page}), $config{srcdir},
                $template->output);
 
        # Set the mtime, this lets the build process get the right creation
@@ -374,7 +368,7 @@ sub remove_feeds () { #{{{
 sub pagefile ($) { #{{{
        my $page=shift;
 
-       return "$IkiWiki::config{srcdir}/".IkiWiki::htmlpage($page);
+       return "$config{srcdir}/".htmlpage($page);
 } #}}}
 
 1
index 7fae9a2f2e5ceae2b5f11191bf83f6c835beee0a..225d8d829f28ee842024934c6038443b01c5f27f 100644 (file)
@@ -7,8 +7,7 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "brokenlinks",
-               call => \&preprocess);
+       hook(type => "preprocess", id => "brokenlinks", call => \&preprocess);
 } # }}}
 
 sub preprocess (@) { #{{{
@@ -17,19 +16,19 @@ sub preprocess (@) { #{{{
        
        # Needs to update whenever a page is added or removed, so
        # register a dependency.
-       IkiWiki::add_depends($params{page}, $params{pages});
+       add_depends($params{page}, $params{pages});
        
        my @broken;
-       foreach my $page (keys %IkiWiki::links) {
-               if (IkiWiki::pagespec_match($page, $params{pages})) {
-                       foreach my $link (@{$IkiWiki::links{$page}}) {
-                               next if $link =~ /.*\/discussion/i && $IkiWiki::config{discussion};
-                               my $bestlink=IkiWiki::bestlink($page, $link);
+       foreach my $page (keys %links) {
+               if (pagespec_match($page, $params{pages})) {
+                       foreach my $link (@{$links{$page}}) {
+                               next if $link =~ /.*\/discussion/i && $config{discussion};
+                               my $bestlink=bestlink($page, $link);
                                next if length $bestlink;
                                push @broken,
-                                       IkiWiki::htmllink($page, $params{destpage}, $link, 1).
+                                       htmllink($page, $params{destpage}, $link, 1).
                                        " in ".
-                                       IkiWiki::htmllink($params{page}, $params{destpage}, $page, 1);
+                                       htmllink($params{page}, $params{destpage}, $page, 1);
                        }
                }
        }
index 68d399d77698bb5be52ba48cd3ddaab21bb5b964..0bb35b6e26249294fb7d3aa57c38b17a59663e69 100644 (file)
@@ -2,11 +2,12 @@
 # CamelCase links
 package IkiWiki::Plugin::camelcase;
 
+use IkiWiki;
 use warnings;
 use strict;
 
 sub import { #{{{
-       IkiWiki::hook(type => "filter", id => "camelcase", call => \&filter);
+       hook(type => "filter", id => "camelcase", call => \&filter);
 } # }}}
 
 sub filter (@) { #{{{
index 4e9d919a3bd182f88227a650616de48b94a2cf25..b677026558f457f8da0e48c605006771361eb905 100644 (file)
@@ -2,18 +2,16 @@
 # Discordian date support fnord ikiwiki.
 package IkiWiki::Plugin::ddate;
 use IkiWiki;
-use IkiWiki::Render; # so we can redefine it here
 no warnings;
 
 sub import { #{{{
-       IkiWiki::hook(type => "checkconfig", id => "skeleton", 
-               call => \&checkconfig);
+       hook(type => "checkconfig", id => "skeleton", call => \&checkconfig);
 } # }}}
 
 sub checkconfig () { #{{{
-       if (! defined $IkiWiki::config{timeformat} ||
-           $IkiWiki::config{timeformat} eq '%c') {
-               $IkiWiki::config{timeformat}='on %{%A, the %e of %B%}, %Y. %N%nCelebrate %H';
+       if (! defined $config{timeformat} ||
+           $config{timeformat} eq '%c') {
+               $config{timeformat}='on %{%A, the %e of %B%}, %Y. %N%nCelebrate %H';
        }
 } #}}}
 
@@ -21,7 +19,7 @@ sub IkiWiki::displaytime ($) { #{{{
        my $time=shift;
         eval q{use POSIX};
         my $gregorian=POSIX::strftime("%d %m %Y", localtime($time));
-       my $date=`ddate +'$IkiWiki::config{timeformat}' $gregorian`;
+       my $date=`ddate +'$config{timeformat}' $gregorian`;
        chomp $date;
        if ($? || ! length $date) {
                return "some time or other (hail Eris!)";
index 8e96bdbbd1e442be73b45586f3fd276b2ea9b0e2..43f4526af3a99812ccfad101526b3242aabed088 100644 (file)
@@ -2,12 +2,12 @@
 # Include a fortune in a page
 package IkiWiki::Plugin::fortune;
 
+use IkiWiki;
 use warnings;
 use strict;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "fortune",
-               call => \&preprocess);
+       hook(type => "preprocess", id => "fortune", call => \&preprocess);
 } # }}}
 
 sub preprocess (@) { #{{{
index c99563d9591f856ed7e01661534733ea812b206e..c6abeb717d96ca7cdc16bd3e7f15e130b2d99d55 100644 (file)
@@ -7,9 +7,9 @@ use IkiWiki;
 use IPC::Open2;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "googlecalendar",
+       hook(type => "preprocess", id => "googlecalendar",
                call => \&preprocess);
-       IkiWiki::hook(type => "format", id => "googlecalendar",
+       hook(type => "format", id => "googlecalendar",
                call => \&format);
 } # }}}
 
index e2873ec6f56f7a65cf51f6b23813eed4b3926099..fe8a782fad2ffdf5c83c4ecb0c86f269195dd2d7 100644 (file)
@@ -7,8 +7,7 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "haiku",
-               call => \&preprocess);
+       hook(type => "preprocess", id => "haiku", call => \&preprocess);
 } # }}}
 
 sub preprocess (@) { #{{{
@@ -16,7 +15,7 @@ sub preprocess (@) { #{{{
 
        my $haiku;
        eval q{use Coy};
-       if ($@) {
+       if ($@ || ! Coy->can("Coy::with_haiku")) {
                my @canned=(
                        "The lack of a Coy:
                         No darting, subtle haiku.
index 83720b4b59c84e5b317f46e77e4ca9e82b29a02e..4270a7eb67c247123cacf2a4fd486157ccbdb780 100644 (file)
@@ -7,12 +7,12 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "htmlize", id => "html", call => \&htmlize);
-       IkiWiki::hook(type => "htmlize", id => "htm", call => \&htmlize);
+       hook(type => "htmlize", id => "html", call => \&htmlize);
+       hook(type => "htmlize", id => "htm", call => \&htmlize);
 
        # ikiwiki defaults to skipping .html files as a security measure;
        # make it process them so this plugin can take effect
-       $IkiWiki::config{wiki_file_prune_regexp} =~ s/\|\\\.x\?html\?\$//;
+       $config{wiki_file_prune_regexp} =~ s/\|\\\.x\?html\?\$//;
 } # }}}
 
 sub htmlize (@) { #{{{
index d77ab809b7f67ae0e7d3d86842e12ea06a00736b..3035307c246001acefd8e3bd945a6f3b8280520b 100644 (file)
@@ -6,8 +6,7 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "sanitize", id => "htmlscrubber",
-               call => \&sanitize);
+       hook(type => "sanitize", id => "htmlscrubber", call => \&sanitize);
 } # }}}
 
 sub sanitize (@) { #{{{
index c626250cba0c8efcecc21b7f82b29abcb2aaed96..906c677dc1302391eebd9266d74aea20fe5b1d9c 100644 (file)
@@ -13,7 +13,7 @@ use IkiWiki;
 use IPC::Open2;
 
 sub import { #{{{
-       IkiWiki::hook(type => "sanitize", id => "tidy", call => \&sanitize);
+       hook(type => "sanitize", id => "tidy", call => \&sanitize);
 } # }}}
 
 sub sanitize (@) { #{{{
@@ -28,7 +28,7 @@ sub sanitize (@) { #{{{
                last unless $@;
                $tries--;
                if ($tries < 1) {
-                       IkiWiki::debug("failed to run tidy: $@");
+                       debug("failed to run tidy: $@");
                        return $params{content};
                }
        }
index 6661ed2153cb25e3df7f4c85562a51b6bd7566c8..79e64beac72a16fd5019d23eac3614ea848afbbf 100644 (file)
@@ -5,17 +5,18 @@ package IkiWiki::Plugin::inline;
 use warnings;
 use strict;
 use IkiWiki;
+use IkiWiki::Render; # for displaytime
 use URI;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "inline", 
+       hook(type => "preprocess", id => "inline", 
                call => \&IkiWiki::preprocess_inline);
-       IkiWiki::hook(type => "pagetemplate", id => "inline",
+       hook(type => "pagetemplate", id => "inline",
                call => \&IkiWiki::pagetemplate_inline);
        # Hook to change to do pinging since it's called late.
        # This ensures each page only pings once and prevents slow
        # pings interrupting page builds.
-       IkiWiki::hook(type => "change", id => "inline", 
+       hook(type => "change", id => "inline", 
                call => \&IkiWiki::pingurl);
 } # }}}
 
index 2405cae7f52c4199042642c9d2103a2fae1a38ae..6b16097442cc468abcdb4f88754cf498eafab2c8 100644 (file)
@@ -7,10 +7,8 @@ use IkiWiki;
 use IPC::Open2;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "linkmap",
-               call => \&preprocess);
-       IkiWiki::hook(type => "format", id => "linkmap",
-               call => \&format);
+       hook(type => "preprocess", id => "linkmap", call => \&preprocess);
+       hook(type => "format", id => "linkmap", call => \&format);
 } # }}}
 
 my $mapnum=0;
@@ -23,7 +21,7 @@ sub preprocess (@) { #{{{
        
        # Needs to update whenever a page is added or removed, so
        # register a dependency.
-       IkiWiki::add_depends($params{page}, $params{pages});
+       add_depends($params{page}, $params{pages});
        
        # Can't just return the linkmap here, since the htmlscrubber
        # scrubs out all <object> tags (with good reason!)
@@ -49,9 +47,9 @@ sub genmap ($) { #{{{
 
        # Get all the items to map.
        my %mapitems = ();
-       foreach my $item (keys %IkiWiki::links) {
-               if (IkiWiki::pagespec_match($item, $params{pages})) {
-                       my $link=IkiWiki::htmlpage($item);
+       foreach my $item (keys %links) {
+               if (pagespec_match($item, $params{pages})) {
+                       my $link=htmlpage($item);
                        $link=IkiWiki::abs2rel($link, IkiWiki::dirname($params{page}));
                        $mapitems{$item}=$link;
                }
@@ -59,7 +57,7 @@ sub genmap ($) { #{{{
 
        # Use ikiwiki's function to create the file, this makes sure needed
        # subdirs are there and does some sanity checking.
-       IkiWiki::writefile("$params{page}.png", $IkiWiki::config{destdir}, "");
+       writefile("$params{page}.png", $config{destdir}, "");
 
        # Run dot to create the graphic and get the map data.
        # TODO: should really add the png to renderedfiles and call
@@ -69,7 +67,7 @@ sub genmap ($) { #{{{
        my $pid;
        while (1) {
                eval {
-                       $pid=open2(*IN, *OUT, "dot -Tpng -o '$IkiWiki::config{destdir}/$params{page}.png' -Tcmapx");
+                       $pid=open2(*IN, *OUT, "dot -Tpng -o '$config{destdir}/$params{page}.png' -Tcmapx");
                };
                last unless $@;
                $tries--;
@@ -88,7 +86,7 @@ sub genmap ($) { #{{{
                if defined $params{width} and defined $params{height};
        foreach my $item (keys %mapitems) {
                print OUT "\"$item\" [shape=box,href=\"$mapitems{$item}\"];\n";
-               foreach my $link (map { IkiWiki::bestlink($item, $_) } @{$IkiWiki::links{$item}}) {
+               foreach my $link (map { bestlink($item, $_) } @{$links{$item}}) {
                        print OUT "\"$item\" -> \"$link\";\n"
                                if $mapitems{$link};
                }
index 27ae0c6a37eb6dce47ec37460b63243bb176c2af..96daf39fcdf9debd255eec245bba57d1432747fc 100644 (file)
@@ -12,8 +12,7 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "map",
-               call => \&preprocess);
+       hook(type => "preprocess", id => "map", call => \&preprocess);
 } # }}}
 
 sub preprocess (@) { #{{{
@@ -22,12 +21,12 @@ sub preprocess (@) { #{{{
        
        # Needs to update whenever a page is added or removed, so
        # register a dependency.
-       IkiWiki::add_depends($params{page}, $params{pages});
+       add_depends($params{page}, $params{pages});
        
        # Get all the items to map.
        my @mapitems = ();
-       foreach my $page (keys %IkiWiki::links) {
-               if (IkiWiki::pagespec_match($page, $params{pages})) {
+       foreach my $page (keys %links) {
+               if (pagespec_match($page, $params{pages})) {
                        push @mapitems, $page;
                }
        }
@@ -50,7 +49,7 @@ sub preprocess (@) { #{{{
                }
                $map .= "</li>\n" if $openli;
                $map .= "<li>"
-                       .IkiWiki::htmllink($params{page}, $params{destpage}, $item) ."\n";
+                       .htmllink($params{page}, $params{destpage}, $item) ."\n";
                $openli=1;
        }
        while ($indent > 0) {
index 2274fea72489b3256cd3d05ee9b0ba7f0dd82d42..e5228c3299c5f98b3b8ac8b3cda0b2d11d4e6ce5 100644 (file)
@@ -7,7 +7,7 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "htmlize", id => "mdwn", call => \&htmlize);
+       hook(type => "htmlize", id => "mdwn", call => \&htmlize);
 } # }}}
 
 my $markdown_loaded=0;
@@ -26,7 +26,7 @@ sub htmlize (@) { #{{{
                eval q{use Markdown};
                if ($@) {
                        do "/usr/bin/markdown" ||
-                               IkiWiki::error("failed to load Markdown.pm perl module ($@) or /usr/bin/markdown ($!)");
+                               error("failed to load Markdown.pm perl module ($@) or /usr/bin/markdown ($!)");
                }
                $markdown_loaded=1;
                require Encode;
index 9b026fffa3c3c57480643bb424834abb6e50efbc..5bcd658378f37c630fe4b389f7bf25dfc95faeb7 100644 (file)
@@ -13,12 +13,9 @@ my %author;
 my %authorurl;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "meta", 
-               call => \&preprocess);
-       IkiWiki::hook(type => "filter", id => "meta", 
-               call => \&filter);
-       IkiWiki::hook(type => "pagetemplate", id => "meta", 
-               call => \&pagetemplate);
+       hook(type => "preprocess", id => "meta", call => \&preprocess);
+       hook(type => "filter", id => "meta", call => \&filter);
+       hook(type => "pagetemplate", id => "meta", call => \&pagetemplate);
 } # }}}
 
 sub filter (@) { #{{{
@@ -54,7 +51,7 @@ sub preprocess (@) { #{{{
                }
                else {
                        # hidden WikiLink
-                       push @{$IkiWiki::links{$page}}, $value;
+                       push @{$links{$page}}, $value;
                }
        }
        elsif ($key eq 'title') {
index 2219634caa3a4c28d2a07276712847be48f8363b..3a8a7954fe1dd6e5c49b5ba12a7f5ec6557e6fa6 100644 (file)
@@ -7,8 +7,7 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "orphans",
-               call => \&preprocess);
+       hook(type => "preprocess", id => "orphans", call => \&preprocess);
 } # }}}
 
 sub preprocess (@) { #{{{
@@ -17,30 +16,30 @@ sub preprocess (@) { #{{{
        
        # Needs to update whenever a page is added or removed, so
        # register a dependency.
-       IkiWiki::add_depends($params{page}, $params{pages});
+       add_depends($params{page}, $params{pages});
        
        my %linkedto;
-       foreach my $p (keys %IkiWiki::links) {
-               map { $linkedto{IkiWiki::bestlink($p, $_)}=1 if length $_ }
-                       @{$IkiWiki::links{$p}};
+       foreach my $p (keys %links) {
+               map { $linkedto{bestlink($p, $_)}=1 if length $_ }
+                       @{$links{$p}};
        }
        
        my @orphans;
-       foreach my $page (keys %IkiWiki::renderedfiles) {
+       foreach my $page (keys %renderedfiles) {
                next if $linkedto{$page};
-               next unless IkiWiki::pagespec_match($page, $params{pages});
+               next unless pagespec_match($page, $params{pages});
                # If the page has a link to some other page, it's
                # indirectly linked to a page via that page's backlinks.
                next if grep { 
                        length $_ &&
-                       ($_ !~ /\/Discussion$/i || ! $IkiWiki::config{discussion}) &&
-                       IkiWiki::bestlink($page, $_) !~ /^($page|)$/ 
-               } @{$IkiWiki::links{$page}};
+                       ($_ !~ /\/Discussion$/i || ! $config{discussion}) &&
+                       bestlink($page, $_) !~ /^($page|)$/ 
+               } @{$links{$page}};
                push @orphans, $page;
        }
        
        return "All pages are linked to by other pages." unless @orphans;
-       return "<ul>\n".join("\n", map { "<li>".IkiWiki::htmllink($params{page}, $params{destpage}, $_, 1)."</li>" } sort @orphans)."</ul>\n";
+       return "<ul>\n".join("\n", map { "<li>".htmllink($params{page}, $params{destpage}, $_, 1)."</li>" } sort @orphans)."</ul>\n";
 } # }}}
 
 1
index b10d5e80724a315c5c277087f99aa0464d743c22..3f34133906fec3c181ee25aee42682305c0fe720 100644 (file)
@@ -7,8 +7,8 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "filter", id => "otl", call => \&filter);
-       IkiWiki::hook(type => "htmlize", id => "otl", call => \&htmlize);
+       hook(type => "filter", id => "otl", call => \&filter);
+       hook(type => "htmlize", id => "otl", call => \&htmlize);
 
 } # }}}
 
@@ -16,9 +16,9 @@ sub filter (@) { #{{{
        my %params=@_;
         
        # Munge up check boxes to look a little bit better. This is a hack.
-       my $checked=IkiWiki::htmllink($params{page}, $params{page},
+       my $checked=htmllink($params{page}, $params{page},
                "smileys/star_on.png", 0, 0, "[X]");
-       my $unchecked=IkiWiki::htmllink($params{page}, $params{page},
+       my $unchecked=htmllink($params{page}, $params{page},
                "smileys/star_off.png", 0, 0, "[_]");
        $params{content}=~s/^(\s*)\[X\]\s/${1}$checked /mg;
        $params{content}=~s/^(\s*)\[_\]\s/${1}$unchecked /mg;
@@ -40,7 +40,7 @@ sub htmlize (@) { #{{{
                unless (defined $pid) {
                        $tries--;
                        if ($tries < 1) {
-                               IkiWiki::debug("failed to fork: $@");
+                               debug("failed to fork: $@");
                                return $params{content};
                        }
                }
@@ -55,7 +55,7 @@ sub htmlize (@) { #{{{
                        unless (defined $pid) {
                                $tries--;
                                if ($tries < 1) {
-                                       IkiWiki::debug("failed to fork: $@");
+                                       debug("failed to fork: $@");
                                        print $params{content};
                                        exit;
                                }
@@ -64,7 +64,7 @@ sub htmlize (@) { #{{{
 
                if (! $pid) {
                        if (! exec 'otl2html', '-S', '/dev/null', '-T', '/dev/stdin') {
-                               IkiWiki::debug("failed to run otl2html: $@");
+                               debug("failed to run otl2html: $@");
                                print $params{content};
                                exit;
                        }
index cfc962b6c4df0172fca409033326b3153602fc3f..7ced159823fff8cf8139392d9b6d81e903f3390f 100644 (file)
@@ -7,8 +7,7 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "pagecount", 
-               call => \&preprocess);
+       hook(type => "preprocess", id => "pagecount", call => \&preprocess);
 } # }}}
 
 sub preprocess (@) { #{{{
@@ -17,13 +16,13 @@ sub preprocess (@) { #{{{
        
        # Needs to update count whenever a page is added or removed, so
        # register a dependency.
-       IkiWiki::add_depends($params{page}, $params{pages});
+       add_depends($params{page}, $params{pages});
        
-       my @pages=keys %IkiWiki::pagesources;
+       my @pages=keys %pagesources;
        return $#pages+1 if $params{pages} eq "*"; # optimisation
        my $count=0;
        foreach my $page (@pages) {
-               $count++ if IkiWiki::pagespec_match($page, $params{pages});
+               $count++ if pagespec_match($page, $params{pages});
        }
        return $count;
 } # }}}
index 8ce563fc5ac489260db3122dc842621a8a51ae8b..0958f5af6f173e673aa25f0109c0e80ee91aa08a 100644 (file)
@@ -18,8 +18,7 @@ use IkiWiki;
 our @classes = ('smallestPC', 'smallPC', 'normalPC', 'bigPC', 'biggestPC' );
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "pagestats",
-               call => \&preprocess);
+       hook(type => "preprocess", id => "pagestats", call => \&preprocess);
 } # }}}
 
 sub preprocess (@) { #{{{
@@ -29,12 +28,12 @@ sub preprocess (@) { #{{{
        
        # Needs to update whenever a page is added or removed, so
        # register a dependency.
-       IkiWiki::add_depends($params{page}, $params{pages});
+       add_depends($params{page}, $params{pages});
        
        my %counts;
        my $max = 0;
-       foreach my $page (keys %IkiWiki::links) {
-               if (IkiWiki::pagespec_match($page, $params{pages})) {
+       foreach my $page (keys %links) {
+               if (pagespec_match($page, $params{pages})) {
                        my @bl = IkiWiki::backlinks($page);
                        $counts{$page} = scalar(@bl);
                        $max = $counts{$page} if $counts{$page} > $max;
@@ -45,7 +44,7 @@ sub preprocess (@) { #{{{
                return "<table class='pageStats'>\n".
                        join("\n", map {
                                "<tr><td>".
-                               IkiWiki::htmllink($params{page}, $params{destpage}, $_, 1).
+                               htmllink($params{page}, $params{destpage}, $_, 1).
                                "</td><td>".$counts{$_}."</td></tr>"
                        }
                        sort { $counts{$b} <=> $counts{$a} } keys %counts).
@@ -57,7 +56,7 @@ sub preprocess (@) { #{{{
                foreach my $page (sort keys %counts) {
                        my $class = $classes[$counts{$page} * scalar(@classes) / ($max + 1)];
                        $res .= "<span class=\"$class\">".
-                               IkiWiki::htmllink($params{page}, $params{destpage}, $page).
+                               htmllink($params{page}, $params{destpage}, $page).
                                "</span>\n";
                }
                $res .= "</div>\n";
index 82fd575a78e5b921e1c73977626ff11d44825b4f..05ad4416ec58c7ad4962305a20dfef6374a47dfa 100644 (file)
@@ -11,8 +11,7 @@ use IkiWiki;
 use File::Find;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "polygen",
-               call => \&preprocess);
+       hook(type => "preprocess", id => "polygen", call => \&preprocess);
 } # }}}
 
 sub preprocess (@) { #{{{
index 789a70ac752b25c2c7aec04d0c79dd61a66f1775..1fd13d1f5ceca92262cbbfb70d86e979c9ff7d88 100644 (file)
@@ -36,7 +36,7 @@ print html[html.find('<body>')+6:html.find('</body>')].strip();
 ";
 
 sub import { #{{{
-       IkiWiki::hook(type => "htmlize", id => "rst", call => \&htmlize);
+       hook(type => "htmlize", id => "rst", call => \&htmlize);
 } # }}}
 
 sub htmlize (@) { #{{{
@@ -54,7 +54,7 @@ sub htmlize (@) { #{{{
                last unless $@;
                $tries--;
                if ($tries < 1) {
-                       IkiWiki::debug("failed to run python to convert rst: $@");
+                       debug("failed to run python to convert rst: $@");
                        return $content;
                }
        }
index c8e0dd580224a7dc810208c1ac3f294633b1f5ab..b6fca66ffe0f8655486abc414a9e3b952c5667aa 100644 (file)
@@ -7,22 +7,22 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "checkconfig", id => "hyperestraier",
+       hook(type => "checkconfig", id => "hyperestraier",
                call => \&checkconfig);
-       IkiWiki::hook(type => "pagetemplate", id => "hyperestraier",
+       hook(type => "pagetemplate", id => "hyperestraier",
                call => \&pagetemplate);
-       IkiWiki::hook(type => "delete", id => "hyperestraier",
+       hook(type => "delete", id => "hyperestraier",
                call => \&delete);
-       IkiWiki::hook(type => "change", id => "hyperestraier",
+       hook(type => "change", id => "hyperestraier",
                call => \&change);
-       IkiWiki::hook(type => "cgi", id => "hyperestraier",
+       hook(type => "cgi", id => "hyperestraier",
                call => \&cgi);
 } # }}}
 
 sub checkconfig () { #{{{
        foreach my $required (qw(url cgiurl)) {
-               if (! length $IkiWiki::config{$required}) {
-                       IkiWiki::error("Must specify $required when using the search plugin\n");
+               if (! length $config{$required}) {
+                       error("Must specify $required when using the search plugin\n");
                }
        }
 } #}}}
@@ -36,8 +36,8 @@ sub pagetemplate (@) { #{{{
        # Add search box to page header.
        if ($template->query(name => "searchform")) {
                if (! defined $form) {
-                       my $searchform = IkiWiki::template("searchform.tmpl", blind_cache => 1);
-                       $searchform->param(searchaction => $IkiWiki::config{cgiurl});
+                       my $searchform = template("searchform.tmpl", blind_cache => 1);
+                       $searchform->param(searchaction => $config{cgiurl});
                        $form=$searchform->output;
                }
 
@@ -46,19 +46,19 @@ sub pagetemplate (@) { #{{{
 } #}}}
 
 sub delete (@) { #{{{
-       IkiWiki::debug("cleaning hyperestraier search index");
-       IkiWiki::estcmd("purge -cl");
-       IkiWiki::estcfg();
+       debug("cleaning hyperestraier search index");
+       estcmd("purge -cl");
+       estcfg();
 } #}}}
 
 sub change (@) { #{{{
-       IkiWiki::debug("updating hyperestraier search index");
-       IkiWiki::estcmd("gather -cm -bc -cl -sd",
+       debug("updating hyperestraier search index");
+       estcmd("gather -cm -bc -cl -sd",
                map {
-                       Encode::encode_utf8($IkiWiki::config{destdir}."/".$IkiWiki::renderedfiles{IkiWiki::pagename($_)})
+                       Encode::encode_utf8($config{destdir}."/".$renderedfiles{pagename($_)})
                } @_
        );
-       IkiWiki::estcfg();
+       estcfg();
 } #}}}
 
 sub cgi ($) { #{{{
@@ -66,25 +66,22 @@ sub cgi ($) { #{{{
 
        if (defined $cgi->param('phrase')) {
                # only works for GET requests
-               chdir("$IkiWiki::config{wikistatedir}/hyperestraier") || IkiWiki::error("chdir: $!");
-               exec("./".IkiWiki::basename($IkiWiki::config{cgiurl})) || IkiWiki::error("estseek.cgi failed");
+               chdir("$config{wikistatedir}/hyperestraier") || error("chdir: $!");
+               exec("./".IkiWiki::basename($config{cgiurl})) || error("estseek.cgi failed");
        }
 } #}}}
 
-# Easier to keep these in the IkiWiki namespace.
-package IkiWiki;
-
 my $configured=0;
 sub estcfg () { #{{{
        return if $configured;
        $configured=1;
        
        my $estdir="$config{wikistatedir}/hyperestraier";
-       my $cgi=basename($config{cgiurl});
+       my $cgi=IkiWiki::basename($config{cgiurl});
        $cgi=~s/\..*$//;
        open(TEMPLATE, ">$estdir/$cgi.tmpl") ||
                error("write $estdir/$cgi.tmpl: $!");
-       print TEMPLATE misctemplate("search", 
+       print TEMPLATE IkiWiki::misctemplate("search", 
                "<!--ESTFORM-->\n\n<!--ESTRESULT-->\n\n<!--ESTINFO-->\n\n");
        close TEMPLATE;
        open(TEMPLATE, ">$estdir/$cgi.conf") ||
@@ -94,12 +91,12 @@ sub estcfg () { #{{{
        $template->param(
                index => $estdir,
                tmplfile => "$estdir/$cgi.tmpl",
-               destdir => abs_path($config{destdir}),
+               destdir => IkiWiki::abs_path($config{destdir}),
                url => $config{url},
        );
        print TEMPLATE $template->output;
        close TEMPLATE;
-       $cgi="$estdir/".basename($config{cgiurl});
+       $cgi="$estdir/".IkiWiki::basename($config{cgiurl});
        unlink($cgi);
        symlink("/usr/lib/estraier/estseek.cgi", $cgi) ||
                error("symlink $cgi: $!");
index 0b006f110b20a963bba9b4acc575db1c95e9fcce..da1f82b82ba87365c902337718d1b29ecc7ffde9 100644 (file)
@@ -9,24 +9,23 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "pagetemplate", id => "sidebar", 
-               call => \&pagetemplate);
+       hook(type => "pagetemplate", id => "sidebar", call => \&pagetemplate);
 } # }}}
 
 sub sidebar_content ($) { #{{{
        my $page=shift;
        
-       my $sidebar_page=IkiWiki::bestlink($page, "sidebar") || return;
-       my $sidebar_file=$IkiWiki::pagesources{$sidebar_page} || return;
-       my $sidebar_type=IkiWiki::pagetype($sidebar_file);
+       my $sidebar_page=bestlink($page, "sidebar") || return;
+       my $sidebar_file=$pagesources{$sidebar_page} || return;
+       my $sidebar_type=pagetype($sidebar_file);
        
        if (defined $sidebar_type) {
                # FIXME: This isn't quite right; it won't take into account
                # adding a new sidebar page. So adding such a page
                # currently requires a wiki rebuild.
-               IkiWiki::add_depends($page, $sidebar_page);
+               add_depends($page, $sidebar_page);
 
-               my $content=IkiWiki::readfile(IkiWiki::srcfile($sidebar_file));
+               my $content=readfile(srcfile($sidebar_file));
                return unless length $content;
                return IkiWiki::htmlize($page, $sidebar_type,
                       IkiWiki::linkify($sidebar_page, $page,
index 5947ebb4c2fdc974dfa852ddb1add9b628a0e2d5..3e1e5700b7e84734805639de744c764cebac226a 100644 (file)
@@ -9,38 +9,26 @@ use strict;
 use IkiWiki;
 
 sub import { #{{{
-       IkiWiki::hook(type => "getopt", id => "skeleton", 
-               call => \&getopt);
-       IkiWiki::hook(type => "checkconfig", id => "skeleton", 
-               call => \&checkconfig);
-       IkiWiki::hook(type => "preprocess", id => "skeleton", 
-               call => \&preprocess);
-       IkiWiki::hook(type => "filter", id => "skeleton", 
-               call => \&filter);
-       IkiWiki::hook(type => "htmlize", id => "skeleton",
-               call => \&htmlize);
-       IkiWiki::hook(type => "sanitize", id => "skeleton", 
-               call => \&sanitize);
-       IkiWiki::hook(type => "format", id => "skeleton", 
-               call => \&format);
-       IkiWiki::hook(type => "pagetemplate", id => "skeleton", 
-               call => \&pagetemplate);
-       IkiWiki::hook(type => "delete", id => "skeleton", 
-               call => \&delete);
-       IkiWiki::hook(type => "change", id => "skeleton", 
-               call => \&change);
-       IkiWiki::hook(type => "cgi", id => "skeleton", 
-               call => \&cgi);
-       IkiWiki::hook(type => "savestate", id => "savestate", 
-               call => \&savestate);
+       hook(type => "getopt", id => "skeleton",  call => \&getopt);
+       hook(type => "checkconfig", id => "skeleton", call => \&checkconfig);
+       hook(type => "preprocess", id => "skeleton", call => \&preprocess);
+       hook(type => "filter", id => "skeleton", call => \&filter);
+       hook(type => "htmlize", id => "skeleton", call => \&htmlize);
+       hook(type => "sanitize", id => "skeleton", call => \&sanitize);
+       hook(type => "format", id => "skeleton", call => \&format);
+       hook(type => "pagetemplate", id => "skeleton", call => \&pagetemplate);
+       hook(type => "delete", id => "skeleton", call => \&delete);
+       hook(type => "change", id => "skeleton", call => \&change);
+       hook(type => "cgi", id => "skeleton", call => \&cgi);
+       hook(type => "savestate", id => "savestate", call => \&savestate);
 } # }}}
 
 sub getopt () { #{{{
-       IkiWiki::debug("skeleton plugin getopt");
+       debug("skeleton plugin getopt");
 } #}}}
 
 sub checkconfig () { #{{{
-       IkiWiki::debug("skeleton plugin checkconfig");
+       debug("skeleton plugin checkconfig");
 } #}}}
 
 sub preprocess (@) { #{{{
@@ -52,7 +40,7 @@ sub preprocess (@) { #{{{
 sub filter (@) { #{{{
        my %params=@_;
        
-       IkiWiki::debug("skeleton plugin running as filter");
+       debug("skeleton plugin running as filter");
 
        return $params{content};
 } # }}}
@@ -60,7 +48,7 @@ sub filter (@) { #{{{
 sub htmlize (@) { #{{{
        my %params=@_;
 
-       IkiWiki::debug("skeleton plugin running as htmlize");
+       debug("skeleton plugin running as htmlize");
 
        return $params{content};
 } # }}}
@@ -68,7 +56,7 @@ sub htmlize (@) { #{{{
 sub sanitize (@) { #{{{
        my %params=@_;
        
-       IkiWiki::debug("skeleton plugin running as a sanitizer");
+       debug("skeleton plugin running as a sanitizer");
 
        return $params{content};
 } # }}}
@@ -76,7 +64,7 @@ sub sanitize (@) { #{{{
 sub format (@) { #{{{
        my %params=@_;
        
-       IkiWiki::debug("skeleton plugin running as a formatter");
+       debug("skeleton plugin running as a formatter");
 
        return $params{content};
 } # }}}
@@ -86,29 +74,29 @@ sub pagetemplate (@) { #{{{
        my $page=$params{page};
        my $template=$params{template};
        
-       IkiWiki::debug("skeleton plugin running as a pagetemplate hook");
+       debug("skeleton plugin running as a pagetemplate hook");
 } # }}}
 
 sub delete (@) { #{{{
        my @files=@_;
 
-       IkiWiki::debug("skeleton plugin told that files were deleted: @files");
+       debug("skeleton plugin told that files were deleted: @files");
 } #}}}
 
 sub change (@) { #{{{
        my @files=@_;
 
-       IkiWiki::debug("skeleton plugin told that changed files were rendered: @files");
+       debug("skeleton plugin told that changed files were rendered: @files");
 } #}}}
 
 sub cgi ($) { #{{{
        my $cgi=shift;
 
-       IkiWiki::debug("skeleton plugin running in cgi");
+       debug("skeleton plugin running in cgi");
 } #}}}
 
 sub savestate () { #{{{
-       IkiWiki::debug("skeleton plugin running in savestate");
+       debug("skeleton plugin running in savestate");
 } #}}}
 
 1
index f49c9b62f6e27b8be7665e7b78c3bfe6e77b60ba..0c6f5d4272cba5fb8aff8ea68bf76935fca0b299 100644 (file)
@@ -9,33 +9,33 @@ my %smileys;
 my $smiley_regexp;
 
 sub import { #{{{
-       IkiWiki::hook(type => "checkconfig", id => "smiley", call => \&setup);
+       hook(type => "checkconfig", id => "smiley", call => \&setup);
 } # }}}
 
 sub setup () { #{{{
-       my $list=IkiWiki::readfile(IkiWiki::srcfile("smileys.mdwn"));
+       my $list=readfile(srcfile("smileys.mdwn"));
        while ($list =~ m/^\s*\*\s+\\([^\s]+)\s+\[\[([^]]+)\]\]/mg) {
                $smileys{$1}=$2;
        }
        
        if (! %smileys) {
-               IkiWiki::debug("failed to parse any smileys, disabling plugin");
+               debug("failed to parse any smileys, disabling plugin");
                return;
        }
        
-       IkiWiki::hook(type => "filter", id => "smiley", call => \&filter);
+       hook(type => "filter", id => "smiley", call => \&filter);
        # sort and reverse so that substrings come after longer strings
        # that contain them, in most cases.
        $smiley_regexp='('.join('|', map { quotemeta }
                reverse sort keys %smileys).')';
-       #IkiWiki::debug($smiley_regexp);
+       #debug($smiley_regexp);
 } #}}}
 
 sub filter (@) { #{{{
        my %params=@_;
        
        $params{content} =~ s{(?<=\s)(\\?)$smiley_regexp(?=\s)}{
-               $1 ? $2 : IkiWiki::htmllink($params{page}, $params{page}, $smileys{$2}, 0, 0, $2)
+               $1 ? $2 : htmllink($params{page}, $params{page}, $smileys{$2}, 0, 0, $2)
        }egs;
        
        return $params{content};
index 3de09a7672641ec471eb147038fd6a9672a3e0e3..7a1be6bec6b983c1ec8abb332d9a197df08a26cc 100644 (file)
@@ -9,26 +9,23 @@ use IkiWiki;
 my %tags;
 
 sub import { #{{{
-       IkiWiki::hook(type => "getopt", id => "tag",
-               call => \&getopt);
-       IkiWiki::hook(type => "preprocess", id => "tag",
-               call => \&preprocess);
-       IkiWiki::hook(type => "pagetemplate", id => "tag",
-               call => \&pagetemplate);
+       hook(type => "getopt", id => "tag", call => \&getopt);
+       hook(type => "preprocess", id => "tag", call => \&preprocess);
+       hook(type => "pagetemplate", id => "tag", call => \&pagetemplate);
 } # }}}
 
 sub getopt () { #{{{
        eval q{use Getopt::Long};
        Getopt::Long::Configure('pass_through');
-       GetOptions("tagbase=s" => \$IkiWiki::config{tagbase});
+       GetOptions("tagbase=s" => \$config{tagbase});
 } #}}}
 
 sub tagpage ($) { #{{{
        my $tag=shift;
                        
-       if (exists $IkiWiki::config{tagbase} &&
-           defined $IkiWiki::config{tagbase}) {
-               $tag=$IkiWiki::config{tagbase}."/".$tag;
+       if (exists $config{tagbase} &&
+           defined $config{tagbase}) {
+               $tag=$config{tagbase}."/".$tag;
        }
 
        return $tag;
@@ -47,7 +44,7 @@ sub preprocess (@) { #{{{
        foreach my $tag (keys %params) {
                push @{$tags{$page}}, $tag;
                # hidden WikiLink
-               push @{$IkiWiki::links{$page}}, tagpage($tag);
+               push @{$links{$page}}, tagpage($tag);
        }
                
        return "";
@@ -61,7 +58,7 @@ sub pagetemplate (@) { #{{{
 
        $template->param(tags => [
                map { 
-                       link => IkiWiki::htmllink($page, $destpage, tagpage($_))
+                       link => htmllink($page, $destpage, tagpage($_))
                }, @{$tags{$page}}
        ]) if exists $tags{$page} && @{$tags{$page}} && $template->query(name => "tags");
 
index 7e23306cb6a96f1ffd1e075760966b0039ca2164..1a4faf05f03515c30e85f3d0d4a9359d9ab87c37 100644 (file)
@@ -9,8 +9,7 @@ use HTML::Template;
 use Encode;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "template", 
-               call => \&preprocess);
+       hook(type => "preprocess", id => "template", call => \&preprocess);
 } # }}}
 
 sub preprocess (@) { #{{{
@@ -21,11 +20,11 @@ sub preprocess (@) { #{{{
        }
 
        my $template_page="templates/$params{id}";
-       IkiWiki::add_depends($params{page}, $template_page);
+       add_depends($params{page}, $template_page);
 
-       my $template_file=$IkiWiki::pagesources{$template_page};
+       my $template_file=$pagesources{$template_page};
        return "[[template ".
-              IkiWiki::htmllink($params{page}, $params{destpage}, $template_page).
+              htmllink($params{page}, $params{destpage}, $template_page).
               " not found]]"
                unless defined $template_file;
 
@@ -34,7 +33,7 @@ sub preprocess (@) { #{{{
                         my $text_ref = shift;
                         $$text_ref=&Encode::decode_utf8($$text_ref);
                 },
-                filename => IkiWiki::srcfile($template_file),
+                filename => srcfile($template_file),
                 die_on_bad_params => 0,
                no_includes => 1,
                blind_cache => 1,
index 063e66f3babafaa674bcd9a3d5ef472dfd2e4f20..279eef00837a66c901056096c89af1c1180e6a42 100644 (file)
@@ -8,10 +8,8 @@ use IkiWiki;
 use HTML::Parser;
 
 sub import { #{{{
-       IkiWiki::hook(type => "preprocess", id => "toc",
-               call => \&preprocess);
-       IkiWiki::hook(type => "format", id => "toc",
-               call => \&format);
+       hook(type => "preprocess", id => "toc", call => \&preprocess);
+       hook(type => "format", id => "toc", call => \&format);
 } # }}}
 
 my %tocpages;
index 5c46c565cc7b95100c3124fa9a3c496bb4afc502..95182f870c671a8c758e35231df93494464ad889 100644 (file)
@@ -8,7 +8,7 @@ use IkiWiki;
 use Text::WikiFormat;
 
 sub import { #{{{
-       IkiWiki::hook(type => "htmlize", id => "wiki", call => \&htmlize);
+       hook(type => "htmlize", id => "wiki", call => \&htmlize);
 } # }}}
 
 sub htmlize (@) { #{{{
index dc8b6b9fecea22bd182f47031c4c6bd05690a00b..949bbde0ae602a46eafd10f3a44484ee8e203358 100644 (file)
@@ -48,7 +48,7 @@ sub _safe_git (&@) { #{{{
        return wantarray ? @lines : ($? == 0);
 }
 # Convenient wrappers.
-sub run_or_die ($@) { _safe_git(\&IkiWiki::error, @_) }
+sub run_or_die ($@) { _safe_git(\&error, @_) }
 sub run_or_cry ($@) { _safe_git(sub { warn @_ },  @_) }
 sub run_or_non ($@) { _safe_git(undef,            @_) }
 #}}}
index 2c1ea3eb7c0b73f507fb51c70499f01c10c51ccc..c16877b3deb9117573962d7a061843ecf26e5067 100644 (file)
@@ -7,44 +7,6 @@ use strict;
 use IkiWiki;
 use Encode;
 
-sub linkify ($$$) { #{{{
-       my $lpage=shift; # the page containing the links
-       my $page=shift; # the page the link will end up on (different for inline)
-       my $content=shift;
-
-       $content =~ s{(\\?)$config{wiki_link_regexp}}{
-               $2 ? ( $1 ? "[[$2|$3]]" : htmllink($lpage, $page, titlepage($3), 0, 0, pagetitle($2)))
-                  : ( $1 ? "[[$3]]" :    htmllink($lpage, $page, titlepage($3)))
-       }eg;
-       
-       return $content;
-} #}}}
-
-sub htmlize ($$$) { #{{{
-       my $page=shift;
-       my $type=shift;
-       my $content=shift;
-       
-       if (exists $hooks{htmlize}{$type}) {
-               $content=$hooks{htmlize}{$type}{call}->(
-                       page => $page,
-                       content => $content,
-               );
-       }
-       else {
-               error("htmlization of $type not supported");
-       }
-
-       run_hooks(sanitize => sub {
-               $content=shift->(
-                       page => $page,
-                       content => $content,
-               );
-       });
-       
-       return $content;
-} #}}}
-
 sub backlinks ($) { #{{{
        my $page=shift;
 
@@ -91,80 +53,6 @@ sub parentlinks ($) { #{{{
        return @ret;
 } #}}}
 
-my %preprocessing;
-sub preprocess ($$$) { #{{{
-       my $page=shift; # the page the data comes from
-       my $destpage=shift; # the page the data will appear in (different for inline)
-       my $content=shift;
-
-       my $handle=sub {
-               my $escape=shift;
-               my $command=shift;
-               my $params=shift;
-               if (length $escape) {
-                       return "[[$command $params]]";
-               }
-               elsif (exists $hooks{preprocess}{$command}) {
-                       # Note: preserve order of params, some plugins may
-                       # consider it significant.
-                       my @params;
-                       while ($params =~ /(?:(\w+)=)?(?:"""(.*?)"""|"([^"]+)"|(\S+))(?:\s+|$)/sg) {
-                               my $key=$1;
-                               my $val;
-                               if (defined $2) {
-                                       $val=$2;
-                                       $val=~s/\r\n/\n/mg;
-                                       $val=~s/^\n+//g;
-                                       $val=~s/\n+$//g;
-                               }
-                               elsif (defined $3) {
-                                       $val=$3;
-                               }
-                               elsif (defined $4) {
-                                       $val=$4;
-                               }
-
-                               if (defined $key) {
-                                       push @params, $key, $val;
-                               }
-                               else {
-                                       push @params, $val, '';
-                               }
-                       }
-                       if ($preprocessing{$page}++ > 3) {
-                               # Avoid loops of preprocessed pages preprocessing
-                               # other pages that preprocess them, etc.
-                               return "[[$command preprocessing loop detected on $page at depth $preprocessing{$page}]]";
-                       }
-                       my $ret=$hooks{preprocess}{$command}{call}->(
-                               @params,
-                               page => $page,
-                               destpage => $destpage,
-                       );
-                       $preprocessing{$page}--;
-                       return $ret;
-               }
-               else {
-                       return "[[$command $params]]";
-               }
-       };
-       
-       $content =~ s{(\\?)\[\[(\w+)\s+((?:(?:\w+=)?(?:""".*?"""|"[^"]+"|[^\s\]]+)\s*)*)\]\]}{$handle->($1, $2, $3)}seg;
-       return $content;
-} #}}}
-
-sub add_depends ($$) { #{{{
-       my $page=shift;
-       my $pagespec=shift;
-       
-       if (! exists $depends{$page}) {
-               $depends{$page}=$pagespec;
-       }
-       else {
-               $depends{$page}=pagespec_merge($depends{$page}, $pagespec);
-       }
-} # }}}
-
 sub genpage ($$$) { #{{{
        my $page=shift;
        my $content=shift;
@@ -236,16 +124,6 @@ sub check_overwrite ($$) { #{{{
        }
 } #}}}
 
-sub displaytime ($) { #{{{
-       my $time=shift;
-
-       eval q{use POSIX};
-       # strftime doesn't know about encodings, so make sure
-       # its output is properly treated as utf8
-       return decode_utf8(POSIX::strftime(
-                       $config{timeformat}, localtime($time)));
-} #}}}
-
 sub mtime ($) { #{{{
        my $file=shift;
        
@@ -270,17 +148,6 @@ sub findlinks ($$) { #{{{
        }
 } #}}}
 
-sub filter ($$) {
-       my $page=shift;
-       my $content=shift;
-
-       run_hooks(filter => sub {
-               $content=shift->(page => $page, content => $content);
-       });
-
-       return $content;
-}
-
 sub render ($) { #{{{
        my $file=shift;
        
diff --git a/NEWS b/NEWS
new file mode 120000 (symlink)
index 0000000..798088b
--- /dev/null
+++ b/NEWS
@@ -0,0 +1 @@
+debian/NEWS
\ No newline at end of file
index 5f9190e68b65a0e5e9bef9c7f1bff188a00f3a65..a54a18fa8bedbae7149444a32aabe932ff3762c6 100644 (file)
@@ -3,8 +3,25 @@ ikiwiki (1.27) UNRELEASED; urgency=low
   * Add a googlecalendar plugin. A bit special-purpose, but it shows
     one way to to deal with user-supplied content that could cause XSS
     issues w/o the htmlscrubber, and won't survive the scrubber.
-
- -- Joey Hess <joeyh@debian.org>  Sat,  9 Sep 2006 03:00:45 -0400
+  * Work on firming up the plugin interface:
+    - Plugins should not need to load IkiWiki::Render to get commonly
+      used functions, so moved some functions from there to IkiWiki.
+    - Picked out the set of functions and variables that most plugins
+      use, documented them, and made IkiWiki export them by default,
+      like a proper perl module should.
+    - Use the other functions at your own risk.
+    - This is not quite complete, I still have to decide whether to
+      export some other things.
+  * Changed all plugins included in ikiwiki to not use "IkiWiki::" when
+    referring to stuff now exported by the IkiWiki module.
+  * Anyone with a third-party ikiwiki plugin is strongly enrouraged
+    to make like changes to it and avoid use of non-exported symboles from
+    "IkiWiki::".
+  * Link debian/changelog and debian/news to NEWS and CHANGELOG.
+  * Support hyperestradier version 1.4.2, which adds a new required phraseform
+    setting.
+
+ -- Joey Hess <joeyh@debian.org>  Sat,  9 Sep 2006 18:38:19 -0400
 
 ikiwiki (1.26) unstable; urgency=low
 
index dd0ab451fdc30be5bf71c0fc01237be5b7c21dfe..8886bca29051c68e9b363b4e3e22bc72d73f231b 100644 (file)
@@ -19,11 +19,13 @@ being edited.
 
 ## Registering plugins
 
-Plugins should, when imported, call IkiWiki::hook to hook into ikiwiki's
+All plugins should `use IkiWiki` to import the ikiwiki plugin interface.
+
+Plugins should, when imported, call `hook()` to hook into ikiwiki's
 processing. The function uses named parameters, and use varies depending on
-the type of plugin being registered. Note that a plugin can call the
-function more than once to register multiple hooks. All calls to
-IkiWiki::hook should be passed a "type" parameter, which gives the type of
+the type of hook being registered -- see below. Note that a plugin can call
+the function more than once to register multiple hooks. All calls to
+`hook()` should be passed a "type" parameter, which gives the type of
 hook, a "id" paramter, which should be a unique string for this plugin, and
 a "call" parameter, which is a reference to a function to call for the
 hook.
@@ -34,29 +36,29 @@ In roughly the order they are called.
 
 ### getopt
 
-       IkiWiki::hook(type => "getopt", id => "foo", call => \&getopt);
+       hook(type => "getopt", id => "foo", call => \&getopt);
 
 This allows for plugins to perform their own processing of command-line
 options and so add options to the ikiwiki command line. It's called during
 command line processing, with @ARGV full of any options that ikiwiki was
 not able to process on its own. The function should process any options it
 can, removing them from @ARGV, and probably recording the configuration
-settings in %IkiWiki::config. It should take care not to abort if it sees
+settings in %config. It should take care not to abort if it sees
 an option it cannot process, and should just skip over those options and
 leave them in @ARGV.
 
 ### checkconfig
 
-       IkiWiki::hook(type => "checkconfig", id => "foo", call => \&checkconfig);
+       hook(type => "checkconfig", id => "foo", call => \&checkconfig);
 
 This is useful if the plugin needs to check for or modify ikiwiki's
 configuration. It's called early in the startup process. The
 function is passed no values. It's ok for the function to call
-IkiWiki::error if something isn't configured right.
+`error()` if something isn't configured right.
 
 ### filter
 
-       IkiWiki::hook(type => "filter", id => "foo", call => \&filter);
+       hook(type => "filter", id => "foo", call => \&filter);
 
 Runs on the raw source of a page, before anything else touches it, and can
 make arbitrary changes. The function is passed named parameters `page` and
@@ -67,7 +69,7 @@ make arbitrary changes. The function is passed named parameters `page` and
 Adding a [[PreProcessorDirective]] is probably the most common use of a
 plugin.
 
-        IkiWiki::hook(type => "preprocess", id => "foo", call => \&preprocess);
+        hook(type => "preprocess", id => "foo", call => \&preprocess);
 
 Replace "foo" with the command name that will be used inside brackets for
 the preprocessor directive.
@@ -89,7 +91,7 @@ the page) along with the rest of the page.
 
 ### htmlize
 
-       IkiWiki::hook(type => "htmlize", id => "ext", call => \&htmlize);
+       hook(type => "htmlize", id => "ext", call => \&htmlize);
 
 Runs on the raw source of a page and turns it into html. The id parameter
 specifies the filename extension that a file must have to be htmlized using
@@ -101,7 +103,7 @@ return the htmlized content.
 
 ### pagetemplate
 
-       IkiWiki::hook(type => "pagetemplate", id => "foo", call => \&pagetemplate);
+       hook(type => "pagetemplate", id => "foo", call => \&pagetemplate);
 
 Each time a page (or part of a blog page, or an rss feed) is rendered, a
 [[template|templates]] is filled out. This hook allows modifying that
@@ -116,7 +118,7 @@ a new custom parameter to the template.
 
 ### sanitize
 
-       IkiWiki::hook(type => "sanitize", id => "foo", call => \&sanitize);
+       hook(type => "sanitize", id => "foo", call => \&sanitize);
 
 Use this to implement html sanitization or anything else that needs to
 modify the body of a page after it has been fully converted to html.
@@ -126,7 +128,7 @@ should return the sanitized content.
 
 ### format
 
-       IkiWiki::hook(type => "format", id => "foo", call => \&format);
+       hook(type => "format", id => "foo", call => \&format);
 
 The difference between format and sanitize is that sanitize only acts on
 the page body, while format can modify the entire html page including the
@@ -137,14 +139,14 @@ should return the formatted content.
 
 ### delete
 
-       IkiWiki::hook(type => "delete", id => "foo", call => \&delete);
+       hook(type => "delete", id => "foo", call => \&delete);
 
 Each time a page or pages is removed from the wiki, the referenced function
 is called, and passed the names of the source files that were removed.
 
 ### change
 
-       IkiWiki::hook(type => "change", id => "foo", call => \&render);
+       hook(type => "change", id => "foo", call => \&render);
 
 Each time ikiwiki renders a change or addition (but not deletion) to the
 wiki, the referenced function is called, and passed the names of the
@@ -152,7 +154,7 @@ source files that were rendered.
 
 ### cgi
 
-       IkiWiki::hook(type => "cgi", id => "foo", call => \&cgi);
+       hook(type => "cgi", id => "foo", call => \&cgi);
 
 Use this to hook into ikiwiki's cgi script. Each registered cgi hook is
 called in turn, and passed a CGI object. The hook should examine the
@@ -161,64 +163,156 @@ terminate the program.
 
 ### savestate
 
-       IkiWiki::hook(type => "savestate", id => "foo", call => \&savestate);
+       hook(type => "savestate", id => "foo", call => \&savestate);
 
 This hook is called wheneven ikiwiki normally saves its state, just before
 the state is saved. The function can save other state, modify values before
 they're saved, etc.
 
-## Error handing
+## Plugin interface
+
+To import the ikiwiki plugin interface:
+
+       use IkiWiki;
 
-While a plugin can call ikiwiki's `error` routine for a fatal error, for
-errors that aren't intended to halt the entire wiki build, including bad
-parameters passed to a [[PreProcessorDirective]], etc, it's better to just
-return the error message as the output of the plugin.
+This will import several variables and functions into your plugin's
+namespace. These variables and functions are the ones most plugins need,
+and a special effort will be made to avoid changing them in incompatible
+ways, and to document any changes that have to be made in the future.
 
-## Wiki configuration
+Note that IkiWiki also provides other variables functions that are not
+exported by default. No guarantee is made about these in the future, so if
+it's not exported, the wise choice is to not use it.
 
-A plugin can access the wiki's configuration via the `%IkiWiki::config`
+### %config
+
+A plugin can access the wiki's configuration via the `%config`
 hash. The best way to understand the contents of the hash is to look at
 [[ikiwiki.setup]], which sets the hash content to configure the wiki.
 
-## Wiki data
+### Other variables
 
 If your plugin needs to access data about other pages in the wiki. It can
 use the following hashes, using a page name as the key:
 
-* `%IkiWiki::links` lists the names of each page
-  that a page links to, in an array reference.
-* `%IkiWiki::pagemtime` contains the last modification time of each page
-* `%IkiWiki::pagectime` contains the creation time of each page
-* `%IkiWiki::renderedfiles` contains the name of the file rendered by a
-  page
-* `%IkiWiki::pagesources` contains the name of the source file for a page.
-* `%IkiWiki::depends` contains a [[PageSpec]] that is used to specify other
-  pages that a page depends on. If one of its dependencies is updated, the
-  page will also get rebuilt. 
-  
-  Many plugins will need to add dependencies to this hash; the best way to do
-  it is by using the IkiWiki::add_depends function, which takes as its
-  parameters the page name and a [[PageSpec]] of dependencies to add.
-* `%IkiWiki::forcerebuild` any pages set as the keys to this hash will be
-  treated as if they're modified and rebuilt.
-
-## Generating html links
+* `%links` lists the names of each page that a page links to, in an array
+  reference.
+* `%renderedfiles` contains the name of the file rendered by a page.
+* `%pagesources` contains the name of the source file for a page.
+
+### Library functions
+
+#### `hook(@)`
+
+Hook into ikiwiki's processing. See the discussion of hooks above.
+
+#### `debug($)`
+
+Logs a debugging message. These are supressed unless verbose mode is turned
+on.
+
+#### `error($)`
+
+Aborts with an error message.
+
+Note that while any plugin can use this for a fatal error, plugins should
+try to avoid dying on bad input, as that will halt the entire wiki build
+and make the wiki unusable. So for example, if a [[PreProcessorDirective]]
+is passed bad parameters, it's better to return an error message, which can
+appear on the wiki page, rather than calling error().
+
+#### `template($;@)`
+
+Creates and returns a HTML::Template object. The first parameter is the
+name of the file in the template directory. The optional remaining
+parameters are passed to HTML::Template->new.
+
+#### `htmlpage($)`
+
+Passed a page name, returns the base name that will be used for a the html
+page created from it. (Ie, it appends ".html".)
+
+#### `add_depends($$)`
+
+Makes the specified page depend on the specified [[PageSpec]].
+
+#### `pagespec_match($$)`
+
+Passed a page name, and a [[PageSpec]], returns true if the [[PageSpec]]
+matches the page.
+
+#### `bestlink($$)`
+
+Given a page and the text of a link on the page, determine which
+existing page that link best points to. Prefers pages under a
+subdirectory with the same name as the source page, failing that
+goes down the directory tree to the base looking for matching
+pages, as described in [[SubPage/LinkingRules]].
+
+#### `htmllink($$$;$$$)`
 
 Many plugins need to generate html links and add them to a page. This is
-done by using the `IkiWiki::htmllink` function. The usual way to call
-htmlllink is:
+done by using the `htmllink` function. The usual way to call
+`htmlllink` is:
 
        htmllink($page, $page, $link)
 
-Why is $page repeated? Because if a page is inlined inside another, and a
+Why is `$page` repeated? Because if a page is inlined inside another, and a
 link is placed on it, the right way to make that link is actually:
 
        htmllink($page, $destpage, $link)
 
-Here $destpage is the inlining page. A destpage parameter is passed to some
-of the hook functions above; the ones that are not passed it are not used
+Here `$destpage` is the inlining page. A `destpage` parameter is passed to
+some of the hook functions above; the ones that are not passed it are not used
 during inlining and don't need to worry about this issue.
 
+The remaining three optional parameters to `htmllink` are:
+
+1. noimageinline - set to true to avoid turning links into inline html images
+1. forcesubpage  - set to force a link to a subpage
+1. linktext - set to force the link text to something
+
+#### `readfile($;$)`
+
+Given a filename, reads and returns the entire file.
+
+The optional second parameter, if set to a true value, makes the file be read
+in binary mode.
+
+A failure to read the file will result in it dying with an error.
+
+#### `writefile($$$;$)`
+
+Given a filename, a directory to put it in, and the file's content,
+writes a file. 
+
+The optional second parameter, if set to a true value, makes the file be
+written in binary mode.
+
+A failure to write the file will result in it dying with an error.
+
+If the destination directory doesn't exist, it will first be created.
+
+#### `pagetype($)`
+
+Given the name of a source file, returns the type of page it is, if it's
+a type that ikiwiki knowns how to htmlize. Otherwise, returns undef.
+
+#### `pagename($)`
+
+Given the name of a source file, returns the name of the wiki page
+that corresponds to that file.
+
+#### `srcfile($)`
+
+Given the name of a source file in the wiki, searches for the file in
+the source directory and the underlay directory, and returns the full
+path to the first file found.
+
+#### `displaytime($)`
+
+Given a time, formats it for display.
+
 ## RCS plugins
 
 ikiwiki's support for revision control systems also uses pluggable perl
index a4b4919d4cac88ac72711e3760ed8d887eb3d79d..7a3193308b3ff52eb8ab413adaa66cf2a60f3130 100644 (file)
@@ -14,7 +14,7 @@ Released 29 April 2006.
 
 * Unit test suite (with tests of at least core stuff like
   [[PageSpec]]). _(status: exists, could of course use more tests)_
-* [[Plugins]] _(status: done, interface still not quite stable)_
+* [[Plugins]] _(status: done, interface still not [[quite_stable|todo/firm_up_plugin_interface]])_
 * [[Tags]] _(status: fair)_
 * Should have fully working [[todo/utf8]] support. _(status: good)_
 * [[Optimised_rendering|todo/optimisations]] if possible. Deal with other
diff --git a/doc/todo/firm_up_plugin_interface.mdwn b/doc/todo/firm_up_plugin_interface.mdwn
new file mode 100644 (file)
index 0000000..e23f2f3
--- /dev/null
@@ -0,0 +1,72 @@
+The plugin interface is currently that they can register hooks, and can
+call any other ikiwiki internal function they desire, generally through
+Ikiwiki::function calls. Also, some Ikiwiki::config etc variables can be
+used.
+
+This is a very weak interface, and should be firmed up to something more
+like a proper perl library. I've been waiting to get some idea of what bits
+of ikiwiki are most useful to plugins before doing it; there are plenty of
+plugins to know that now.
+
+IkiWiki will now  export some function calls and variables when loaded.
+
+Functions used by many plugins, which I'm sure should be exported:
+
+* hook
+* debug
+* error
+* template
+* htmlpage
+* add_depends
+* pagespec_match
+* bestlink
+* htmllink
+* readfile
+* writefile
+* pagetype
+* srcfile
+* pagename
+* displaytime
+
+Functions used by only some plugins, undecided:
+
+* lockwiki, unlockwiki (aggregate)
+       Too internal to ever be exported.
+* loadindex (aggregate)
+       Too internal to ever be exported.
+* titlepage (aggregate)
+       Not until more than one thing uses it.
+* basename (polygen, inline, search, polygen)
+* dirname (linkmap, inline)
+       For basename and dirname, they could just use standard perl library
+       stuff. Howevever, ikiwiki's versions are slightly different and I'd
+       need to check if the standard versions work for the uses made in
+       these plugins. Inclined not to export.
+* abs2rel (linkmap, inline)
+       This *is* the library version, just optimised to work around a bug.
+       Don't export this.
+* possibly_foolish_untaint (aggregate, polygen)
+       Probably better to implement yourself.
+* htmlize
+* linkify
+* preprocess
+* filter
+       Used by several, but problimatic since plugins typically define
+       functions with these names..
+
+Variables used by plugins:
+
+* %IkiWiki::config (various values; probably not worth locking down any
+  more, export it)
+* %IkiWiki::links (many, seems clearcut to export)
+* %IkiWiki::renderedfiles (orphans, inline, search)
+* %IkiWiki::pagesources (pagecount, sidebar, template, inline)
+* %IkiWiki::pagecase (aggregate.. will not export yet)
+* %IkiWIki::backlinks (pagestats.. will not export yet)
+
+I don't want this interface to be too firm; it's ok for a plugin like
+`ddate` to redefine an internal function like IkiWiki::displaytime if it
+wants to.. But plugins that still access stuff through IkiWiki:: should be
+aware that that stuff can change at any time and break them. Possibly without
+perl's type checking catching the breakage, in some cases. Plugins that
+only use exported symbols should not break by future ikiwiki changes.
index ece900152e0b0648ba686895baa2036e18f1945f..9358bf3764b8b70afebab980b3316904f5602a62 100755 (executable)
@@ -3,23 +3,23 @@ use warnings;
 use strict;
 use Test::More tests => 9;
 
+BEGIN { use_ok("IkiWiki"); }
+
 sub test ($$$) {
        my $page=shift;
        my $link=shift;
        my @existing_pages=@{shift()};
        
        %IkiWiki::pagecase=();
-       %IkiWiki::links=();
+       %links=();
        foreach my $page (@existing_pages) {
                $IkiWiki::pagecase{lc $page}=$page;
-               $IkiWiki::links{$page}=[];
+               $links{$page}=[];
        }
 
-       return IkiWiki::bestlink($page, $link);
+       return bestlink($page, $link);
 }
 
-BEGIN { use_ok("IkiWiki"); }
-
 is(test("bar", "foo", ["bar"]), "", "broken link");
 is(test("bar", "foo", ["bar", "foo"]), "foo", "simple link");
 is(test("bar", "FoO", ["bar", "foo"]), "foo", "simple link with different input case");
index 78f0b50112885d44e490925270058620f705c477..27812559b4556ea7e99e74c773ba8fb53230aed6 100755 (executable)
@@ -4,16 +4,15 @@
 # to htmlize are changed.
 use warnings;
 use strict;
-use Test::More tests => 103;
+use Test::More tests => 102;
 use Encode;
 
 BEGIN { use_ok("IkiWiki"); }
-BEGIN { use_ok("IkiWiki::Render"); }
 
 # Initialize htmlscrubber plugin
-%IkiWiki::config=IkiWiki::defaultconfig();
-$IkiWiki::config{srcdir}=$IkiWiki::config{destdir}="/dev/null";
+%config=IkiWiki::defaultconfig();
+$config{srcdir}=$config{destdir}="/dev/null";
 IkiWiki::loadplugins(); IkiWiki::checkconfig();
-ok(IkiWiki::htmlize("foo", "mdwn", IkiWiki::readfile("t/test1.mdwn")));
-ok(IkiWiki::htmlize("foo", "mdwn", IkiWiki::readfile("t/test3.mdwn")),
+ok(IkiWiki::htmlize("foo", "mdwn", readfile("t/test1.mdwn")));
+ok(IkiWiki::htmlize("foo", "mdwn", readfile("t/test3.mdwn")),
        "wtf?") for 1..100;
index fc6888dca7de7832767d6f16eb9aa036a4ee5752..66e90d8e3bc91d8323283be6c10d313fa603beef 100755 (executable)
--- a/t/html.t
+++ b/t/html.t
@@ -11,10 +11,9 @@ BEGIN {
                plan skip_all => "/usr/bin/validate html validator not present";
        }
        else {
-               plan(tests => int @pages + 3);
+               plan(tests => int @pages + 2);
        }
        use_ok("IkiWiki");
-       use_ok("IkiWiki::Render");
 }
 
 # Have to build the html pages first.
index fc9ca5b84ec64b03b6c61ad55201c7ab34b4303e..a9ccfedcbb184b9a31165b63d9009f012416cbf5 100755 (executable)
@@ -1,22 +1,21 @@
 #!/usr/bin/perl
 use warnings;
 use strict;
-use Test::More tests => 5;
+use Test::More tests => 4;
 use Encode;
 
 BEGIN { use_ok("IkiWiki"); }
-BEGIN { use_ok("IkiWiki::Render"); }
 
 # Initialize htmlscrubber plugin
-%IkiWiki::config=IkiWiki::defaultconfig();
-$IkiWiki::config{srcdir}=$IkiWiki::config{destdir}="/dev/null";
+%config=IkiWiki::defaultconfig();
+$config{srcdir}=$config{destdir}="/dev/null";
 IkiWiki::loadplugins();
 IkiWiki::checkconfig();
 
 is(IkiWiki::htmlize("foo", "mdwn", "foo\n\nbar\n"), "<p>foo</p>\n\n<p>bar</p>\n",
        "basic");
-is(IkiWiki::htmlize("foo", "mdwn", IkiWiki::readfile("t/test1.mdwn")),
+is(IkiWiki::htmlize("foo", "mdwn", readfile("t/test1.mdwn")),
        Encode::decode_utf8(qq{<p><img src="../images/o.jpg" alt="o" title="&oacute;" />\nóóóóó</p>\n}),
        "utf8; bug #373203");
-ok(IkiWiki::htmlize("foo", "mdwn", IkiWiki::readfile("t/test2.mdwn")),
+ok(IkiWiki::htmlize("foo", "mdwn", readfile("t/test2.mdwn")),
        "this file crashes markdown if it's fed in as decoded utf-8");
index 2ed3face30db096b4ade26ccf9f93544393d3cb5..74747e1270339104e53f0edfd2e9b80c301a1adc 100755 (executable)
@@ -3,6 +3,8 @@ use warnings;
 use strict;
 use Test::More tests => 13;
 
+BEGIN { use_ok("IkiWiki"); }
+
 sub linkify ($$$$) {
        my $lpage=shift;
        my $page=shift;
@@ -13,13 +15,13 @@ sub linkify ($$$$) {
        # This is what linkify and htmllink need set right now to work.
        # This could change, if so, update it..
        %IkiWiki::pagecase=();
-       %IkiWiki::links=();
+       %links=();
        foreach my $page (@existing_pages) {
                $IkiWiki::pagecase{lc $page}=$page;
-               $IkiWiki::links{$page}=[];
-               $IkiWiki::renderedfiles{"$page.mdwn"}=$page;
+               $links{$page}=[];
+               $renderedfiles{"$page.mdwn"}=$page;
        }
-       %IkiWiki::config=IkiWiki::defaultconfig();
+       %config=IkiWiki::defaultconfig();
 
        return IkiWiki::linkify($lpage, $page, $content);
 }
@@ -64,8 +66,6 @@ sub links_text ($$) {
 }
 
 
-BEGIN { use_ok("IkiWiki::Render"); }
-
 ok(links_to("bar", linkify("foo", "foo", "link to [[bar]] ok", ["foo", "bar"])), "ok link");
 ok(not_links_to("bar", linkify("foo", "foo", "link to \\[[bar]] ok", ["foo", "bar"])), "escaped link");
 ok(links_to("page=bar", linkify("foo", "foo", "link to [[bar]] ok", ["foo"])), "broken link");
index 20243dfa69855ab2c84539262a1f16940d320014..bf6c8cbdf90bd1e3ce8273fcdedacf43058b9bd0 100755 (executable)
@@ -5,44 +5,44 @@ use Test::More tests => 35;
 
 BEGIN { use_ok("IkiWiki"); }
 
-ok(IkiWiki::pagespec_match("foo", "*"));
-ok(IkiWiki::pagespec_match("page", "?ag?"));
-ok(! IkiWiki::pagespec_match("page", "?a?g?"));
-ok(IkiWiki::pagespec_match("foo.png", "*.*"));
-ok(! IkiWiki::pagespec_match("foo", "*.*"));
-ok(IkiWiki::pagespec_match("foo", "foo or bar"), "simple list");
-ok(IkiWiki::pagespec_match("bar", "foo or bar"), "simple list 2");
-ok(IkiWiki::pagespec_match("foo", "f?? and !foz"));
-ok(! IkiWiki::pagespec_match("foo", "f?? and !foo"));
-ok(! IkiWiki::pagespec_match("foo", "* and !foo"));
-ok(! IkiWiki::pagespec_match("foo", "foo and !foo"));
-ok(! IkiWiki::pagespec_match("foo.png", "* and !*.*"));
-ok(IkiWiki::pagespec_match("foo", "(bar or ((meep and foo) or (baz or foo) or beep))"));
+ok(pagespec_match("foo", "*"));
+ok(pagespec_match("page", "?ag?"));
+ok(! pagespec_match("page", "?a?g?"));
+ok(pagespec_match("foo.png", "*.*"));
+ok(! pagespec_match("foo", "*.*"));
+ok(pagespec_match("foo", "foo or bar"), "simple list");
+ok(pagespec_match("bar", "foo or bar"), "simple list 2");
+ok(pagespec_match("foo", "f?? and !foz"));
+ok(! pagespec_match("foo", "f?? and !foo"));
+ok(! pagespec_match("foo", "* and !foo"));
+ok(! pagespec_match("foo", "foo and !foo"));
+ok(! pagespec_match("foo.png", "* and !*.*"));
+ok(pagespec_match("foo", "(bar or ((meep and foo) or (baz or foo) or beep))"));
 
-$IkiWiki::links{foo}=[qw{bar baz}];
-ok(IkiWiki::pagespec_match("foo", "link(bar)"));
-ok(! IkiWiki::pagespec_match("foo", "link(quux)"));
-ok(IkiWiki::pagespec_match("bar", "backlink(foo)"));
-ok(! IkiWiki::pagespec_match("quux", "backlink(foo)"));
+$links{foo}=[qw{bar baz}];
+ok(pagespec_match("foo", "link(bar)"));
+ok(! pagespec_match("foo", "link(quux)"));
+ok(pagespec_match("bar", "backlink(foo)"));
+ok(! pagespec_match("quux", "backlink(foo)"));
 
 $IkiWiki::pagectime{foo}=1154532692; # Wed Aug  2 11:26 EDT 2006
 $IkiWiki::pagectime{bar}=1154532695; # after
-ok(IkiWiki::pagespec_match("foo", "created_before(bar)"));
-ok(! IkiWiki::pagespec_match("foo", "created_after(bar)"));
-ok(! IkiWiki::pagespec_match("bar", "created_before(foo)"));
-ok(IkiWiki::pagespec_match("bar", "created_after(foo)"));
-ok(IkiWiki::pagespec_match("foo", "creation_year(2006)"), "year");
-ok(! IkiWiki::pagespec_match("foo", "creation_year(2005)"), "other year");
-ok(IkiWiki::pagespec_match("foo", "creation_month(8)"), "month");
-ok(! IkiWiki::pagespec_match("foo", "creation_month(9)"), "other month");
-ok(IkiWiki::pagespec_match("foo", "creation_day(2)"), "day");
-ok(! IkiWiki::pagespec_match("foo", "creation_day(3)"), "other day");
+ok(pagespec_match("foo", "created_before(bar)"));
+ok(! pagespec_match("foo", "created_after(bar)"));
+ok(! pagespec_match("bar", "created_before(foo)"));
+ok(pagespec_match("bar", "created_after(foo)"));
+ok(pagespec_match("foo", "creation_year(2006)"), "year");
+ok(! pagespec_match("foo", "creation_year(2005)"), "other year");
+ok(pagespec_match("foo", "creation_month(8)"), "month");
+ok(! pagespec_match("foo", "creation_month(9)"), "other month");
+ok(pagespec_match("foo", "creation_day(2)"), "day");
+ok(! pagespec_match("foo", "creation_day(3)"), "other day");
 
 # old style globlists
-ok(IkiWiki::pagespec_match("foo", "foo bar"), "simple list");
-ok(IkiWiki::pagespec_match("bar", "foo bar"), "simple list 2");
-ok(IkiWiki::pagespec_match("foo", "f?? !foz"));
-ok(! IkiWiki::pagespec_match("foo", "f?? !foo"));
-ok(! IkiWiki::pagespec_match("foo", "* !foo"));
-ok(! IkiWiki::pagespec_match("foo", "foo !foo"));
-ok(! IkiWiki::pagespec_match("foo.png", "* !*.*"));
+ok(pagespec_match("foo", "foo bar"), "simple list");
+ok(pagespec_match("bar", "foo bar"), "simple list 2");
+ok(pagespec_match("foo", "f?? !foz"));
+ok(! pagespec_match("foo", "f?? !foo"));
+ok(! pagespec_match("foo", "* !foo"));
+ok(! pagespec_match("foo", "foo !foo"));
+ok(! pagespec_match("foo.png", "* !*.*"));
index c2860709b3f44b32940504ccf7670f392e53b4aa..cbb06219c15184d82caf320b5842f16de74daf6a 100755 (executable)
@@ -3,20 +3,20 @@ use warnings;
 use strict;
 use Test::More tests => 25;
 
+BEGIN { use_ok("IkiWiki"); }
+
 sub same {
        my $a=shift;
        my $b=shift;
        my $match=shift;
        
-       my $imatch=(IkiWiki::pagespec_match($match, $a) ||
-                   IkiWiki::pagespec_match($match, $b));
-       my $cmatch=IkiWiki::pagespec_match($match, IkiWiki::pagespec_merge($a, $b));
+       my $imatch=(pagespec_match($match, $a) ||
+                   pagespec_match($match, $b));
+       my $cmatch=pagespec_match($match, IkiWiki::pagespec_merge($a, $b));
        
        return $imatch == $cmatch;
 }
 
-BEGIN { use_ok("IkiWiki"); }
-
 ok(same("foo", "bar", "foo"), "basic match 1");
 ok(same("foo", "bar", "bar"), "basic match 2");
 ok(same("foo", "bar", "foobar"), "basic failed match");
@@ -36,7 +36,7 @@ ok(same("f?? !f??", "!bar", "bar"), "matching glob and matching inverted glob");
 ok(same("b??", "!b?z", "bar"), "matching glob and non-matching inverted glob");
 ok(same("f?? !f?z", "!bar", "bar"), "matching glob and non-matching inverted glob");
 ok(same("!foo bar baz", "!bar", "bar"), "matching list and matching inversion");
-ok(IkiWiki::pagespec_match("foo/Discussion",
+ok(pagespec_match("foo/Discussion",
        IkiWiki::pagespec_merge("* !*/Discussion", "*/Discussion")), "should match");
 ok(same("* !*/Discussion", "*/Discussion", "foo/Discussion"), "Discussion merge 1");
 ok(same("*/Discussion", "* !*/Discussion", "foo/Discussion"), "Discussion merge 2");
index 5332f4c352a0abc603ab8786febf6780820997da..bb1c6bae3fa0dc75b3dc0145c7c1357e3bc27e94 100755 (executable)
@@ -7,6 +7,6 @@ use Encode;
 BEGIN { use_ok("IkiWiki"); }
 
 # should read files as utf8
-ok(Encode::is_utf8(IkiWiki::readfile("t/test1.mdwn"), 1));
-is(IkiWiki::readfile("t/test1.mdwn"),
+ok(Encode::is_utf8(readfile("t/test1.mdwn"), 1));
+is(readfile("t/test1.mdwn"),
        Encode::decode_utf8('![o](../images/o.jpg "ó")'."\n".'óóóóó'."\n"));
index 599a3cd0bd52c68d27126fedb388f6d12b84d7ba..33a4fa9196bafb4535e470e6082285185687054d 100644 (file)
@@ -15,6 +15,7 @@ condgstep: 2
 dotfidf: true
 scancheck: false
 smplphrase: true
+phraseform: 2
 candetail: true
 smlrvnum: 0
 smlrtune: 16 1024 4096