Do not allow the about: URI scheme

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

diff --git a/IkiWiki/Plugin/map.pm b/IkiWiki/Plugin/map.pm
index 97450eb6dd988fbe2d04cf684528658406ca4ddb..581ae5011f8a002190f133f6ce999797bae43b1b 100644 (file)
--- a/IkiWiki/Plugin/map.pm
+++ b/IkiWiki/Plugin/map.pm
@@ -37,6 +37,9 @@ sub preprocess (@) { #{{{
                                my @b=split(/\//, $common_prefix);
                                $common_prefix="";
                                while (@a && @b && $a[0] eq $b[0]) {
+                                       if (length $common_prefix) {
+                                               $common_prefix.="/";
+                                       }
                                        $common_prefix.=shift(@a);
                                        shift @b;
                                }
@@ -45,7 +48,8 @@ sub preprocess (@) { #{{{
        }
        
        # Common prefix should not be a page in the map.
-       while (length $common_prefix && exists $mapitems{$common_prefix}) {
+       while (defined $common_prefix && length $common_prefix &&
+              exists $mapitems{$common_prefix}) {
                $common_prefix=IkiWiki::dirname($common_prefix);
        }
 
@@ -60,19 +64,23 @@ sub preprocess (@) { #{{{
        my $parent="";
        my $indent=0;
        my $openli=0;
+       my $dummy=0;
        my $map = "<div class='map'>\n<ul>\n";
        foreach my $item (sort keys %mapitems) {
-               $item=~s/^\Q$common_prefix\E\/// if length $common_prefix;
+               $item=~s/^\Q$common_prefix\E\///
+                       if defined $common_prefix && length $common_prefix;
                my $depth = ($item =~ tr/\//\//) + 1;
                my $baseitem=IkiWiki::dirname($item);
                while (length $parent && length $baseitem && $baseitem !~ /^\Q$parent\E(\/|$)/) {
                        $parent=IkiWiki::dirname($parent);
+                       last if !$dummy && length $parent && $baseitem =~ /^\Q$parent\E(\/|$)/;
                        $indent--;
                        $map .= "</li>\n";
                        if ($indent > 0) {
                                $map .= "</ul>\n";
                        }
                }
+               $dummy=0;
                while ($depth < $indent) {
                        $indent--;
                        $map .= "</li>\n";
@@ -89,10 +97,13 @@ sub preprocess (@) { #{{{
                                $map .= "<ul>\n";
                        }
                        if ($depth > $indent) {
+                               $dummy=1;
                                $p.="/".shift(@bits);
                                $map .= "<li>"
-                                       .htmllink($params{page}, $params{destpage}, $p, class => "mapparent")
-                                       ."</span>\n";
+                                       .htmllink($params{page}, $params{destpage},
+                                                $p, class => "mapparent",
+                                                noimageinline => 1)
+                                       ."\n";
                                $openli=1;
                        }
                        else {
@@ -102,8 +113,9 @@ sub preprocess (@) { #{{{
                $map .= "</li>\n" if $openli;
                $map .= "<li>"
                        .htmllink($params{page}, $params{destpage}, 
-                               "/".$common_prefix."/".$item, class => "mapitem")
-                       ."</span>\n";
+                               "/".$common_prefix."/".$item,
+                               class => "mapitem", noimageinline => 1)
+                       ."\n";
                $openli=1;
                $parent=$item;
        }