Fix some minor issues

[deb/packages.git] / lib / Packages / DoShow.pm

diff --git a/lib/Packages/DoShow.pm b/lib/Packages/DoShow.pm
index 076e11f7c3826a909ed459b412eb7a2e898c6b3f..e3e1945b6b7cc7123e4705a47f681e79f16106f1 100644 (file)
--- a/lib/Packages/DoShow.pm
+++ b/lib/Packages/DoShow.pm
@@ -1,6 +1,7 @@
 package Packages::DoShow;
 
 use strict;
+use warnings;
 
 use POSIX;
 use URI::Escape;
@@ -81,7 +82,7 @@ sub do_show {
                if ($suite eq $_) {
                    $$menu .= "[ <strong>$_</strong> ] ";
                } else {
-                   $$menu .= "[ <a href=\"$ROOT/$_/$encodedpkg\">$_</a> ] ";
+                   $$menu .= "[ <a href=\"".make_url($encodedpkg,'',{suite=>$suite})."\">$_</a> ] ";
                }
            }
            $$menu .= '<br>';
@@ -143,20 +144,18 @@ sub do_show {
                        $long_desc =~ s/\n /\n/sgo;
                        $long_desc =~ s/\n.\n/\n<p>\n/go;
                        $long_desc =~ s/(((\n|\A) [^\n]*)+)/\n<pre>$1\n<\/pre>/sgo;
-#          $long_desc = conv_desc( $lang, $long_desc );
-#          $short_desc = conv_desc( $lang, $short_desc );
                        my @menu = ( [ _g( "Distribution:" ),
                                       _g( "Overview over this suite" ),
-                                      "$ROOT/$suite/",
+                                      make_url("/",''),
                                       $suite ],
                                     [ _g( "Section:" ),
                                       _g( "All packages in this section" ),
-                                      "$ROOT/$suite/$subsection/",
+                                      make_url("$subsection/",''),
                                       $subsection ], );
                        my $source = $page->get_src('package');
                        push @menu, [ _g( "Source:" ),
                                      _g( "Source package building this package" ),
-                                     "$ROOT/$suite/source/$source",
+                                     make_url($source,'',{source=>'source'}),
                                      $source ] if $source;
                        $$menu .= simple_menu( @menu );
 
@@ -171,7 +170,7 @@ sub do_show {
                        $package_page .= title( $title );
                        
                        if (my $provided_by = $page->{provided_by}) {
-                           note( _g( "This is also a virtual package provided by ").join( ', ', map { "<a href=\"$ROOT/$suite/$_\">$_</a>"  } @$provided_by) );
+                           note( _g( "This is also a virtual package provided by ").join( ', ', map { "<a href=\"".make_url($_,'')."\">$_</a>"  } @$provided_by) );
                        }
                        
                        if ($suite eq "experimental") {
@@ -228,18 +227,22 @@ sub do_show {
                        $package_page .= "<th>"._g( "Package Size")."</th><th>"._g("Installed Size")."</th><th>"._g("Files")."</th></tr>\n";
                        foreach my $a ( @archs ) {
                            $package_page .= "<tr>\n";
-                           $package_page .=  "<th><a href=\"$ROOT/$suite/$encodedpkg/$a/download";
+                           $package_page .=  "<th><a href=\"".make_url("$encodedpkg/$a/download",'');
                            $package_page .=  "\">$a</a></th>\n";
                            $package_page .= "<td>".$versions->{$a}."</td>"
                                if $multiple_versions;
                            $package_page .= '<td class="size">';
-                           $package_page .=  floor(($sizes_deb->{$a}/102.4)+0.5)/10 . "&nbsp;kB";
+                           # package size
+                           $package_page .=  sprintf(_g('%.1f&nbsp;kB'),
+                                                     floor(($sizes_deb->{$a}/102.4)+0.5)/10);
                            $package_page .= '</td><td class="size">';
-                           $package_page .=  $sizes_inst->{$a} . "&nbsp;kB";
+                           # installed size
+                           $package_page .=  sprintf(_g('%d&nbsp;kB'),
+                                                     $sizes_inst->{$a});
                            $package_page .= "</td>\n<td>";
                            if ( $suite ne "experimental" ) {
                                $package_page .= sprintf( "[<a href=\"%s\">"._g( "list of files" )."</a>]\n",
-                                                         "$ROOT/$suite/$encodedpkg/$a/filelist", $pkg );
+                                                         make_url("$encodedpkg/$a/filelist",''), $pkg );
                            } else {
                                $package_page .= _g( "no current information" );
                            }
@@ -262,11 +265,11 @@ sub do_show {
 
                        $$menu .= simple_menu( [ _g( "Distribution:" ),
                                                 _g( "Overview over this distribution" ),
-                                                "$ROOT/",
+                                                make_url('/',''),
                                                 $suite ],
                                               [ _g( "Section:" ),
                                                 _g( "All packages in this section" ),
-                                                "$ROOT/$suite/virtual/",
+                                                make_url("virtual/",''),
                                                 
                                                 'virtual' ], );
 
@@ -304,11 +307,11 @@ sub do_show {
 
                    $$menu .= simple_menu( [ _g( "Distribution:" ),
                                             _g( "Overview over this suite" ),
-                                            "$ROOT/$suite/",
+                                            make_url('/',''),
                                             $suite ],
                                           [ _g( "Section:" ),
                                             _g( "All packages in this section" ),
-                                            "$ROOT/$suite/$subsection/",
+                                            make_url("$subsection/",''),
                                             $subsection ],
                                           );