]> sipb.mit.edu Git - ikiwiki.git/blobdiff - IkiWiki/Plugin/shortcut.pm
correct variable scoping error in nils's patch
[ikiwiki.git] / IkiWiki / Plugin / shortcut.pm
index 26385f3a6e83569ab0d89e8d1e968fc535d8776d..0e7769c67c7f22f36b9b1e857660cc9b8ba0ad9c 100644 (file)
@@ -3,7 +3,7 @@ package IkiWiki::Plugin::shortcut;
 
 use warnings;
 use strict;
 
 use warnings;
 use strict;
-use IkiWiki;
+use IkiWiki 2.00;
 
 sub import { #{{{
        hook(type => "checkconfig", id => "shortcut", call => \&checkconfig);
 
 sub import { #{{{
        hook(type => "checkconfig", id => "shortcut", call => \&checkconfig);
@@ -21,7 +21,7 @@ sub preprocess_shortcut (@) { #{{{
        my %params=@_;
 
        if (! defined $params{name} || ! defined $params{url}) {
        my %params=@_;
 
        if (! defined $params{name} || ! defined $params{url}) {
-               return "[[".gettext("shortcut missing name or url parameter")."]]";
+               return "[[shortcut ".gettext("missing name or url parameter")."]]";
        }
 
        hook(type => "preprocess", no_override => 1, id => $params{name},
        }
 
        hook(type => "preprocess", no_override => 1, id => $params{name},
@@ -30,7 +30,7 @@ sub preprocess_shortcut (@) { #{{{
        #translators: This is used to display what shortcuts are defined.
        #translators: First parameter is the name of the shortcut, the second
        #translators: is an URL.
        #translators: This is used to display what shortcuts are defined.
        #translators: First parameter is the name of the shortcut, the second
        #translators: is an URL.
-       return sprintf(gettext("shortcut %s points to %s"), $params{name}, $params{url});
+       return sprintf(gettext("shortcut %s points to <i>%s</i>"), $params{name}, $params{url});
 } # }}}
 
 sub shortcut_expand ($$@) { #{{{
 } # }}}
 
 sub shortcut_expand ($$@) { #{{{
@@ -54,6 +54,10 @@ sub shortcut_expand ($$@) { #{{{
        my $encoded_text=$text;
        $encoded_text=~s/([^A-Za-z0-9])/sprintf("%%%02X", ord($1))/seg;
        
        my $encoded_text=$text;
        $encoded_text=~s/([^A-Za-z0-9])/sprintf("%%%02X", ord($1))/seg;
        
+       $url=~s{\%([sS])}{
+               $1 eq 's' ? $encoded_text : $text
+       }eg;
+
        $text=~s/_/ /g;
        if (defined $desc) {
                $desc=~s/\%s/$text/g;
        $text=~s/_/ /g;
        if (defined $desc) {
                $desc=~s/\%s/$text/g;
@@ -62,7 +66,6 @@ sub shortcut_expand ($$@) { #{{{
                $desc=$text;
        }
 
                $desc=$text;
        }
 
-       $url=~s/\%s/$encoded_text/g;
        return "<a href=\"$url\">$desc</a>";
 } #}}}
 
        return "<a href=\"$url\">$desc</a>";
 } #}}}