Comments (maybe not so helpful - sorry)

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

diff --git a/IkiWiki/Plugin/progress.pm b/IkiWiki/Plugin/progress.pm
index 6cb21a916e477248b018d171ee419e935f91f1df..76d994acc7ad0f9d49c63c32d3d5d8dd54db5554 100644 (file)
--- a/IkiWiki/Plugin/progress.pm
+++ b/IkiWiki/Plugin/progress.pm
@@ -3,25 +3,25 @@ package IkiWiki::Plugin::progress;
 
 use warnings;
 use strict;
 
 use warnings;
 use strict;

-use IkiWiki 2.00;
+use IkiWiki 3.00;

 
 my $percentage_pattern = qr/[0-9]+\%?/; # pattern to validate percentages
 
 
 my $percentage_pattern = qr/[0-9]+\%?/; # pattern to validate percentages
 

-sub import { #{{{
+sub import {

        hook(type => "getsetup", id => "progress", call => \&getsetup);
        hook(type => "preprocess", id => "progress", call => \&preprocess);
        hook(type => "format",     id => "progress", call => \&format);
        hook(type => "getsetup", id => "progress", call => \&getsetup);
        hook(type => "preprocess", id => "progress", call => \&preprocess);
        hook(type => "format",     id => "progress", call => \&format);

-} # }}}
+}

 
 

-sub getsetup () { #{{{
+sub getsetup () {

        return 
                plugin => {
                        safe => 1,
                        rebuild => undef,
                },
        return 
                plugin => {
                        safe => 1,
                        rebuild => undef,
                },

-} #}}}
+}

 
 

-sub preprocess (@) { #{{{
+sub preprocess (@) {

        my %params=@_;
        
        my $fill;
        my %params=@_;
        
        my $fill;


@@ -29,12 +29,11 @@ sub preprocess (@) { #{{{
        if (defined $params{percent}) {
                $fill = $params{percent};
                ($fill) = $fill =~ m/($percentage_pattern)/; # fill is untainted now
        if (defined $params{percent}) {
                $fill = $params{percent};
                ($fill) = $fill =~ m/($percentage_pattern)/; # fill is untainted now

+               $fill=~s/%$//;

                if (! defined $fill || ! length $fill || $fill > 100 || $fill < 0) {
                if (! defined $fill || ! length $fill || $fill > 100 || $fill < 0) {

-                       error("illegal percent value $params{percent}");
-               }
-               elsif ($fill !~ /%$/) {
-                       $fill.="%";
+                       error(sprintf(gettext("illegal percent value %s"), $params{percent}));

                }
                }

+               $fill.="%";

        }
        elsif (defined $params{totalpages} and defined $params{donepages}) {
                add_depends($params{page}, $params{totalpages});
        }
        elsif (defined $params{totalpages} and defined $params{donepages}) {
                add_depends($params{page}, $params{totalpages});


@@ -57,7 +56,7 @@ sub preprocess (@) { #{{{
                }
        }
        else {
                }
        }
        else {

-               error("need either `percent` or `totalpages` and `donepages` parameters");
+               error(gettext("need either `percent` or `totalpages` and `donepages` parameters"));

        }
 
        return <<EODIV
        }
 
        return <<EODIV


@@ -65,9 +64,9 @@ sub preprocess (@) { #{{{
   <div class="progress-done" style="width: $fill">$fill</div>
 </div>
 EODIV
   <div class="progress-done" style="width: $fill">$fill</div>
 </div>
 EODIV

-} # }}}
+}

 
 

-sub format(@) { #{{{
+sub format(@) {

        my %params = @_;
 
        # If HTMLScrubber has removed the style attribute, then bring it back
        my %params = @_;
 
        # If HTMLScrubber has removed the style attribute, then bring it back


@@ -75,6 +74,6 @@ sub format(@) { #{{{
        $params{content} =~ s!<div class="progress-done">($percentage_pattern)</div>!<div class="progress-done" style="width: $1">$1</div>!g;
 
        return $params{content};    
        $params{content} =~ s!<div class="progress-done">($percentage_pattern)</div>!<div class="progress-done" style="width: $1">$1</div>!g;
 
        return $params{content};    

-} #}}}
+}

 
 1
 
 1