make revert hooks optional

[git.ikiwiki.info.git] / IkiWiki / Plugin / tag.pm

diff --git a/IkiWiki/Plugin/tag.pm b/IkiWiki/Plugin/tag.pm
index 957b012efcde187d7255d0801ddeda50bc9502cc..55064a9a3cef72c92f3ebc3cb43fa9ef7646d256 100644 (file)
--- a/IkiWiki/Plugin/tag.pm
+++ b/IkiWiki/Plugin/tag.pm
@@ -36,17 +36,17 @@ sub getsetup () {
                },
                tag_autocreate => {
                        type => "boolean",
                },
                tag_autocreate => {
                        type => "boolean",

-                       example => 0,
+                       example => 1,

                        description => "autocreate new tag pages?",
                        safe => 1,
                        rebuild => undef,
                },
 }
 
                        description => "autocreate new tag pages?",
                        safe => 1,
                        rebuild => undef,
                },
 }
 

-sub tagpage ($) {
+sub taglink ($) {

        my $tag=shift;
        my $tag=shift;

-                       
-       if ($tag !~ m{^\.?/} &&
+       
+       if ($tag !~ m{^/} &&

            defined $config{tagbase}) {
                $tag="/".$config{tagbase}."/".$tag;
                $tag=~y#/#/#s; # squash dups
            defined $config{tagbase}) {
                $tag="/".$config{tagbase}."/".$tag;
                $tag=~y#/#/#s; # squash dups


@@ -55,27 +55,44 @@ sub tagpage ($) {
        return $tag;
 }
 
        return $tag;
 }
 

-sub taglink ($$$;@) {
+sub htmllink_tag ($$$;@) {

        my $page=shift;
        my $destpage=shift;
        my $tag=shift;
        my %opts=@_;
 
        my $page=shift;
        my $destpage=shift;
        my $tag=shift;
        my %opts=@_;
 

-       return htmllink($page, $destpage, tagpage($tag), %opts);
+       return htmllink($page, $destpage, taglink($tag), %opts);

 }
 
 sub gentag ($) {
        my $tag=shift;
 }
 
 sub gentag ($) {
        my $tag=shift;

-       if ($config{tag_autocreate}) {
-               my $tagfile = newpagefile(tagpage($tag), $config{default_pageext});
-               $tagfile=~s/^\///;
+
+       if ($config{tag_autocreate} ||
+           ($config{tagbase} && ! defined $config{tag_autocreate})) {
+               my $tagpage=taglink($tag);
+               if ($tagpage=~/^\.\/(.*)/) {
+                       $tagpage=$1;
+               }
+               else {
+                       $tagpage=~s/^\///;
+               }
+
+               my $tagfile = newpagefile($tagpage, $config{default_pageext});

 
                add_autofile($tagfile, "tag", sub {
 
                add_autofile($tagfile, "tag", sub {

-                       debug(sprintf(gettext("creating tag page %s"), $tag));
+                       my $message=sprintf(gettext("creating tag page %s"), $tagpage);
+                       debug($message);

 
                        my $template=template("autotag.tmpl");
 
                        my $template=template("autotag.tmpl");

+                       $template->param(tagname => IkiWiki::basename($tag));

                        $template->param(tag => $tag);
                        writefile($tagfile, $config{srcdir}, $template->output);
                        $template->param(tag => $tag);
                        writefile($tagfile, $config{srcdir}, $template->output);

+                       if ($config{rcs}) {
+                               IkiWiki::disable_commit_hook();
+                               IkiWiki::rcs_add($tagfile);
+                               IkiWiki::rcs_commit_staged(message => $message);
+                               IkiWiki::enable_commit_hook();
+                       }

                });
        }
 }
                });
        }
 }


@@ -94,9 +111,8 @@ sub preprocess_tag (@) {
                $tag=linkpage($tag);
                
                # hidden WikiLink
                $tag=linkpage($tag);
                
                # hidden WikiLink

-               add_link($page, tagpage($tag), 'tag');
+               add_link($page, taglink($tag), 'tag');

                
                

-               # add tagpage if necessary

                gentag($tag);
        }
                
                gentag($tag);
        }
                


@@ -111,14 +127,16 @@ sub preprocess_taglink (@) {
        return join(" ", map {
                if (/(.*)\|(.*)/) {
                        my $tag=linkpage($2);
        return join(" ", map {
                if (/(.*)\|(.*)/) {
                        my $tag=linkpage($2);

-                       add_link($params{page}, tagpage($tag), 'tag');
-                       return taglink($params{page}, $params{destpage}, $tag,
+                       add_link($params{page}, taglink($tag), 'tag');
+                       gentag($tag);
+                       return htmllink_tag($params{page}, $params{destpage}, $tag,

                                linktext => pagetitle($1));
                }
                else {
                        my $tag=linkpage($_);
                                linktext => pagetitle($1));
                }
                else {
                        my $tag=linkpage($_);

-                       add_link($params{page}, tagpage($tag), 'tag');
-                       return taglink($params{page}, $params{destpage}, $tag);
+                       add_link($params{page}, taglink($tag), 'tag');
+                       gentag($tag);
+                       return htmllink_tag($params{page}, $params{destpage}, $tag);

                }
        }
        grep {
                }
        }
        grep {


@@ -136,7 +154,7 @@ sub pagetemplate (@) {
 
        $template->param(tags => [
                map { 
 
        $template->param(tags => [
                map { 

-                       link => taglink($page, $destpage, $_, rel => "tag")
+                       link => htmllink_tag($page, $destpage, $_, rel => "tag")

                }, sort keys %$tags
        ]) if defined $tags && %$tags && $template->query(name => "tags");
 
                }, sort keys %$tags
        ]) if defined $tags && %$tags && $template->query(name => "tags");
 


@@ -152,7 +170,9 @@ sub pagetemplate (@) {
 package IkiWiki::PageSpec;
 
 sub match_tagged ($$;@) {
 package IkiWiki::PageSpec;
 
 sub match_tagged ($$;@) {

-       return match_link($_[0], IkiWiki::Plugin::tag::tagpage($_[1]), linktype => 'tag');
+       my $page=shift;
+       my $glob=IkiWiki::Plugin::tag::taglink(shift);
+       return match_link($page, $glob, linktype => 'tag', @_);

 }
 
 1
 }
 
 1