Merge branch 'master' of ssh://git.ikiwiki.info

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

diff --git a/IkiWiki/Plugin/linkmap.pm b/IkiWiki/Plugin/linkmap.pm
index 0a8d0ca9a1bfc20e84251638ae8628036bd20ee3..b5ef1a137beb26e3e66aee89f468c33fd6e1865a 100644 (file)
--- a/IkiWiki/Plugin/linkmap.pm
+++ b/IkiWiki/Plugin/linkmap.pm
@@ -3,55 +3,52 @@ package IkiWiki::Plugin::linkmap;
 
 use warnings;
 use strict;
 
 use warnings;
 use strict;

-use IkiWiki;
+use IkiWiki 3.00;

 use IPC::Open2;
 use IPC::Open2;

+use HTML::Entities;

 
 

-sub import { #{{{
+sub import {
+       hook(type => "getsetup", id => "linkmap", call => \&getsetup);

        hook(type => "preprocess", id => "linkmap", call => \&preprocess);
        hook(type => "preprocess", id => "linkmap", call => \&preprocess);

-       hook(type => "format", id => "linkmap", call => \&format);
-} # }}}
+}
+
+sub getsetup () {
+       return
+               plugin => {
+                       safe => 1,
+                       rebuild => undef,
+                       section => "widget",
+               },
+}

 
 my $mapnum=0;
 
 my $mapnum=0;

-my %maps;

 
 

-sub preprocess (@) { #{{{
+sub pageescape {
+       my $item = shift;
+       # encoding explicitly in case ikiwiki is configured to accept <> or &
+       # in file names
+       my $title = pagetitle($item, 1);
+       # it would not be necessary to encode *all* the html entities (<> would
+       # be sufficient, &" probably a good idea), as dot accepts utf8, but it
+       # isn't bad either
+       $title = encode_entities($title);
+       return("<$title>");
+}
+
+sub preprocess (@) {

        my %params=@_;
 
        $params{pages}="*" unless defined $params{pages};
        
        my %params=@_;
 
        $params{pages}="*" unless defined $params{pages};
        

-       # Needs to update whenever a page is added or removed, so
-       # register a dependency.
-       add_depends($params{page}, $params{pages});
-       
-       # Can't just return the linkmap here, since the htmlscrubber
-       # scrubs out all <object> tags (with good reason!)
-       # Instead, insert a placeholder tag, which will be expanded during
-       # formatting.

        $mapnum++;
        $mapnum++;

-       $maps{$mapnum}=\%params;
-       return "<div class=\"linkmap$mapnum\"></div>";
-} # }}}
-
-sub format (@) { #{{{
-        my %params=@_;
-
-       $params{content}=~s/<div class=\"linkmap(\d+)"><\/div>/genmap($1)/eg;
-
-        return $params{content};
-} # }}}
-
-sub genmap ($) { #{{{
-       my $mapnum=shift;
-       return "" unless exists $maps{$mapnum};
-       my %params=%{$maps{$mapnum}};
+       my $connected=IkiWiki::yesno($params{connected});

 
        # Get all the items to map.
 
        # Get all the items to map.

-       my %mapitems = ();
-       foreach my $item (keys %links) {
-               if (pagespec_match($item, $params{pages}, $params{page})) {
-                       $mapitems{$item}=urlto($item, $params{destpage});
-               }
-       }
+       my %mapitems = map { $_ => urlto($_, $params{destpage}) }
+               pagespec_match_list($params{page}, $params{pages},
+                       # update when a page is added or removed, or its
+                       # links change
+                       deptype => deptype("presence", "links"));

 
        my $dest=$params{page}."/linkmap.png";
 
 
        my $dest=$params{page}."/linkmap.png";
 


@@ -62,7 +59,7 @@ sub genmap ($) { #{{{
 
        # Run dot to create the graphic and get the map data.
        my $pid;
 
        # Run dot to create the graphic and get the map data.
        my $pid;

-       my $sigpipe=0;;
+       my $sigpipe=0;

        $SIG{PIPE}=sub { $sigpipe=1 };
        $pid=open2(*IN, *OUT, "dot -Tpng -o '$config{destdir}/$dest' -Tcmapx");
        
        $SIG{PIPE}=sub { $sigpipe=1 };
        $pid=open2(*IN, *OUT, "dot -Tpng -o '$config{destdir}/$dest' -Tcmapx");
        


@@ -75,30 +72,42 @@ sub genmap ($) { #{{{
        print OUT "charset=\"utf-8\";\n";
        print OUT "ratio=compress;\nsize=\"".($params{width}+0).", ".($params{height}+0)."\";\n"
                if defined $params{width} and defined $params{height};
        print OUT "charset=\"utf-8\";\n";
        print OUT "ratio=compress;\nsize=\"".($params{width}+0).", ".($params{height}+0)."\";\n"
                if defined $params{width} and defined $params{height};

+       my %shown;
+       my $show=sub {
+               my $item=shift;
+               if (! $shown{$item}) {
+                       print OUT pageescape($item)." [shape=box,href=\"$mapitems{$item}\"];\n";
+                       $shown{$item}=1;
+               }
+       };

        foreach my $item (keys %mapitems) {
        foreach my $item (keys %mapitems) {

-               print OUT "\"$item\" [shape=box,href=\"$mapitems{$item}\"];\n";
+               $show->($item) unless $connected;

                foreach my $link (map { bestlink($item, $_) } @{$links{$item}}) {
                foreach my $link (map { bestlink($item, $_) } @{$links{$item}}) {

-                       print OUT "\"$item\" -> \"$link\";\n"
-                               if $mapitems{$link};
+                       next unless length $link and $mapitems{$link};
+                       foreach my $endpoint ($item, $link) {
+                               $show->($endpoint);
+                       }
+                       print OUT pageescape($item)." -> ".pageescape($link).";\n";

                }
        }
        print OUT "}\n";
                }
        }
        print OUT "}\n";

-       close OUT;
+       close OUT || error gettext("failed to run dot");

 
        local $/=undef;
 
        local $/=undef;

-       my $ret="<object data=\"".urlto($dest, $params{page}).
-              "\" type=\"image/png\" usemap=\"#linkmap$mapnum\">\n".
-               <IN>.
-               "</object>";
-       close IN;
+       my $ret="<img src=\"".urlto($dest, $params{destpage}).
+              "\" alt=\"".gettext("linkmap").
+              "\" usemap=\"#linkmap$mapnum\" />\n".
+               <IN>;
+       close IN || error gettext("failed to run dot");

        
        waitpid $pid, 0;
        
        waitpid $pid, 0;

-       $SIG{PIPE}="DEFAULT";
-       if ($sigpipe) {
-               return  "[[linkmap ".gettext("failed to run dot")."]]";
+       if ($?) {
+               error gettext("failed to run dot");

        }
        }

+       $SIG{PIPE}="DEFAULT";
+       error gettext("failed to run dot") if $sigpipe;

 
        return $ret;
 
        return $ret;

-} #}}}
+}

 
 1
 
 1