Merge branch 'ready/tag-test' into transient-tag

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

diff --git a/IkiWiki/Plugin/tag.pm b/IkiWiki/Plugin/tag.pm
index c6c99ae45cd008469b9918af315c2a2b4ffda132..fd5ce1e8a91488ce5d5b20709001fd3d177def97 100644 (file)
--- a/IkiWiki/Plugin/tag.pm
+++ b/IkiWiki/Plugin/tag.pm
@@ -6,8 +6,6 @@ use warnings;
 use strict;
 use IkiWiki 3.00;
 
 use strict;
 use IkiWiki 3.00;
 

-my %tags;
-

 sub import {
        hook(type => "getopt", id => "tag", call => \&getopt);
        hook(type => "getsetup", id => "tag", call => \&getsetup);
 sub import {
        hook(type => "getopt", id => "tag", call => \&getopt);
        hook(type => "getsetup", id => "tag", call => \&getsetup);


@@ -38,17 +36,17 @@ sub getsetup () {
                },
                tag_autocreate => {
                        type => "boolean",
                },
                tag_autocreate => {
                        type => "boolean",

-                       example => 0,
-                       description => "Autocreate new tag pages",
+                       example => 1,
+                       description => "autocreate new tag pages?",

                        safe => 1,
                        safe => 1,

-                       rebuild => 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


@@ -57,28 +55,56 @@ sub tagpage ($) {
        return $tag;
 }
 
        return $tag;
 }
 

-sub taglink ($$$;@) {
+# Returns a tag name from a tag link
+sub tagname ($) {
+       my $tag=shift;
+       if (defined $config{tagbase}) {
+               $tag =~ s!^/\Q$config{tagbase}\E/!!;
+       } else {
+               $tag =~ s!^\.?/!!;
+       }
+       return pagetitle($tag, 1);
+}
+
+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 (defined $config{tag_autocreate} && $config{tag_autocreate}) {
-               my $tagfile = newpagefile(tagpage($tag), $config{default_pageext});
-               $tagfile=~s/^\///;
-
-               return if (! add_autofile($tagfile));

 
 

-               debug(sprintf(gettext("creating tag page %s"), $tag));
+       if ($config{tag_autocreate} ||
+           ($config{tagbase} && ! defined $config{tag_autocreate})) {
+               my $tagpage=taglink($tag);
+               if ($tagpage=~/^\.\/(.*)/) {
+                       $tagpage=$1;
+               }
+               else {
+                       $tagpage=~s/^\///;
+               }

 
 

-               my $template=template("autotag.tmpl");
-               $template->param(tag => $tag);
-               writefile($tagfile, $config{srcdir}, $template->output);
+               my $tagfile = newpagefile($tagpage, $config{default_pageext});
+
+               add_autofile($tagfile, "tag", sub {
+                       my $message=sprintf(gettext("creating tag page %s"), $tagpage);
+                       debug($message);
+
+                       my $template=template("autotag.tmpl");
+                       $template->param(tagname => tagname($tag));
+                       $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,13 +120,11 @@ sub preprocess_tag (@) {
 
        foreach my $tag (keys %params) {
                $tag=linkpage($tag);
 
        foreach my $tag (keys %params) {
                $tag=linkpage($tag);

-               $tags{$page}{$tag}=1;
-
-               # add tagpage if necessary
-               gentag($tag);
-
+               

                # hidden WikiLink
                # hidden WikiLink

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

        }
                
        return "";
        }
                
        return "";


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

-                       $tags{$params{page}}{$tag}=1;
-                       add_link($params{page}, tagpage($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($_);

-                       $tags{$params{page}}{$tag}=1;
-                       add_link($params{page}, tagpage($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 {


@@ -137,17 +161,20 @@ sub pagetemplate (@) {
        my $destpage=$params{destpage};
        my $template=$params{template};
 
        my $destpage=$params{destpage};
        my $template=$params{template};
 

+       my $tags = $typedlinks{$page}{tag};
+

        $template->param(tags => [
                map { 
        $template->param(tags => [
                map { 

-                       link => taglink($page, $destpage, $_, rel => "tag")
-               }, sort keys %{$tags{$page}}
-       ]) if exists $tags{$page} && %{$tags{$page}} && $template->query(name => "tags");
+                       link => htmllink_tag($page, $destpage, $_,
+                                       rel => "tag", linktext => tagname($_))
+               }, sort keys %$tags
+       ]) if defined $tags && %$tags && $template->query(name => "tags");

 
        if ($template->query(name => "categories")) {
                # It's an rss/atom template. Add any categories.
 
        if ($template->query(name => "categories")) {
                # It's an rss/atom template. Add any categories.

-               if (exists $tags{$page} && %{$tags{$page}}) {
-                       $template->param(categories => [map { category => $_ },
-                               sort keys %{$tags{$page}}]);
+               if (defined $tags && %$tags) {
+                       $template->param(categories => [map { category => tagname($_) },
+                               sort keys %$tags]);

                }
        }
 }
                }
        }
 }


@@ -155,9 +182,9 @@ sub pagetemplate (@) {
 package IkiWiki::PageSpec;
 
 sub match_tagged ($$;@) {
 package IkiWiki::PageSpec;
 
 sub match_tagged ($$;@) {

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

 }
 
 1
 }
 
 1