]> git.deb.at Git - deb/packages.git/blobdiff - lib/Packages/HTML.pm
Fix some minor issues
[deb/packages.git] / lib / Packages / HTML.pm
index 28dbb76e64a0a7c4759ae80432dce98fd83b076b..fb16d2fdc66837214586de6ceda1648e747bfc3f 100644 (file)
@@ -8,7 +8,7 @@ use URI::Escape;
 use HTML::Entities;
 use Locale::gettext;
 
-use Packages::CGI ();
+use Packages::CGI qw(make_url make_search_url);
 use Packages::Search qw( read_entry_simple );
 use Packages::Config qw( :all );
 
@@ -82,12 +82,14 @@ sub pkg_list {
     my $suite = $opts->{suite}[0];
 
     my $str = "";
-    foreach my $p ( @$pkgs ) {
+    foreach my $p ( sort @$pkgs ) {
 
+       # we don't deal with virtual packages here because for the
+       # current uses of this function this isn't needed
        my $short_desc = (read_entry_simple( $packages, $p, $opts->{h_archives}, $suite))->[-1];
 
        if ( $short_desc ) {
-           $str .= "<dt><a href=\"$ROOT/$suite/$p\">$p</a></dt>\n".
+           $str .= "<dt><a href=\"".make_url($p,'',{source=>undef})."\">$p</a></dt>\n".
                    "\t<dd>$short_desc</dd>\n";
        } else {
            $str .= "<dt>$p</dt>\n\t<dd>"._g("Not available")."</dd>\n";
@@ -125,11 +127,8 @@ sub pmoreinfo {
     my $src_dir = $page->get_src('directory');
     if ($info{sourcedownload}) {
        my $files = $page->get_src( 'files' );
-       my $path = (@{$opts->{archive}} >1) ?
-           $suite :
-           "$suite/$opts->{archive}[0]";
        $str .= _g( "Source Package:" );
-       $str .= " <a href=\"$ROOT/$path/source/$source\">$source</a>, ".
+       $str .= " <a href=\"".make_url($source,'',{source=>'source'})."\">$source</a>, ".
            _g( "Download" ).":\n";
 
        unless (defined($files) and @$files) {
@@ -137,7 +136,7 @@ sub pmoreinfo {
        } else {
            foreach( @$files ) {
                my ($src_file_md5, $src_file_size, $src_file_name) = split /\s/o, $_;
-               for ($page->get_newest('archive')) {
+               for ("$suite/".$page->get_newest('archive')) {
                    /security/o && do {
                        $str .= "<a href=\"$env->{security}/$src_dir/$src_file_name\">["; last };
                    /volatile/o && do {
@@ -145,7 +144,7 @@ sub pmoreinfo {
                    /backports/o && do {
                        $str .= "<a href=\"$env->{backports}/$src_dir/$src_file_name\">["; last };
                    /non-us/io && do {
-                       $str .= "<a href=\"$env->{nonus_site}/$src_dir/$src_file_name\">["; last };
+                       $str .= "<a href=\"$env->{'non-US'}/$src_dir/$src_file_name\">["; last };
                    $str .= "<a href=\"$env->{us}/$src_dir/$src_file_name\">[";
                }
                if ($src_file_name =~ /dsc$/) {
@@ -216,13 +215,11 @@ sub pmoreinfo {
 }
 
 sub dep_item {
-    my ( $link, $name, $info, $desc ) = @_;
-    my $post_link = '';
-    if ($link) {
-       $link = "<a href=\"$link\">";
+    my ( $suite, $name, $info, $desc ) = @_;
+    my ($link, $post_link) = ('', '');
+    if ($suite) {
+       $link = "<a href=\"".make_url($name,'',{suite=>$suite})."\">";
        $post_link = '</a>';
-    } else {
-       $link = '';
     }
     if ($info) {
        $info = " $info";
@@ -238,6 +235,20 @@ sub dep_item {
     return "$link$name$post_link$info$desc";
 } # end dep_item
 
+sub provides_string {
+    my ($suite, $entry, $also) = @_;
+    my %tmp = map { $_ => 1 } split /\s/, $entry;
+    my @provided_by = keys %tmp; # weed out duplicates
+    my $short_desc = $also ? _g("also a virtual package provided by ")
+       : _g("virtual package provided by ");
+    if (@provided_by < 10) {
+       $short_desc .= join( ', ',map { "<a href=\"".make_url($_,'',{suite=>$suite})."\">$_</a>" } @provided_by);
+    } else {
+       $short_desc .= sprintf( _g("%s packages"), scalar(@provided_by));
+    }
+    return $short_desc;
+}
+
 sub print_deps {
     my ( $packages, $opts, $pkg, $relations, $type) = @_;
     my %dep_type = ('depends' => 'dep', 'recommends' => 'rec', 
@@ -246,7 +257,6 @@ sub print_deps {
     my $res = "<ul class=\"ul$dep_type{$type}\">\n";
     my $first = 1;
     my $suite = $opts->{suite}[0];
-    my $one_archive = @{$opts->{archive}} > 1 ? '': $opts->{archive}[0];
 
 #    use Data::Dumper;
 #    debug( "print_deps called:\n".Dumper( $pkg, $relations, \$type ), 3 ) if DEBUG;
@@ -286,28 +296,26 @@ sub print_deps {
            my $entry = $entries{$p_name} ||
                read_entry_simple( $packages, $p_name, $opts->{h_archives}, $suite);
            my $short_desc = $entry->[-1];
-           my $arch = $entry->[2];
-           my $archive = $entry->[0];
+           my $arch = $entry->[3];
+           my $archive = $entry->[1];
+           my $p_suite = $entry->[2];
            if ( $short_desc ) {
-               my $path = $one_archive eq $archive ? "$suite/$archive" :
-                   $suite;
                if ( $is_old_pkgs ) {
-                   push @res_pkgs, dep_item( "$ROOT/$path/$p_name",
+                   push @res_pkgs, dep_item( $p_suite,
                                              $p_name, "$pkg_version$arch_str" );
-               } elsif ($arch eq 'virtual') {
-                   my @provided_by = split /\s/, $short_desc;
-                   $short_desc = "virtual package provided by ";
-                   if (@provided_by < 10) {
-                       $short_desc .= join( ', ',map { "<a href=\"$ROOT/$path/$_\">$_</a>" } @provided_by);
-                   } else {
-                       $short_desc .= scalar(@provided_by)." packages";
-                   }
-                   push @res_pkgs, dep_item( "$ROOT/$path/$p_name",
-                                             $p_name, "$pkg_version$arch_str", $short_desc );
-               } else {
+               } elsif (defined $entry->[1]) {
                    $entries{$p_name} ||= $entry;
                    $short_desc = encode_entities( $short_desc, "<>&\"" );
-                   push @res_pkgs, dep_item( "$ROOT/$path/$p_name",
+                   $short_desc .= "<br>".provides_string( $p_suite,
+                                                          $entry->[0],
+                                                          1 )
+                       if defined $entry->[0];
+                   push @res_pkgs, dep_item( $p_suite,
+                                             $p_name, "$pkg_version$arch_str", $short_desc );
+               } elsif (defined $entry->[0]) {
+                   $short_desc = provides_string( $p_suite,
+                                                  $entry->[0] );
+                   push @res_pkgs, dep_item( $p_suite,
                                              $p_name, "$pkg_version$arch_str", $short_desc );
                }
            } elsif ( $is_old_pkgs ) {
@@ -382,12 +390,7 @@ sub header {
        $search_in_header = <<MENU;
 <form method="GET" action="$SEARCH_URL">
 <div id="hpacketsearch">
-<input type="hidden" name="debug" value="$values{debug}">
-<input type="hidden" name="suite" value="$values{suite}">
-<input type="hidden" name="exact" value="$values{exact}">
-<input type="hidden" name="arch" value="$values{arch}">
-<input type="hidden" name="section" value="$values{section}">
-<input type="text" size="30" name="keywords" value="$values{keywords}" id="kw">
+<input type="text" size="30" name="keywords" value="" id="kw">
 <input type="submit" value="%s">
 <span style="font-size: 60%%"><a href="$SEARCH_PAGE#search_packages">%s</a></span>
 <br>