]> git.donarmstrong.com Git - debbugs.git/blobdiff - cgi/pkgreport.cgi
Merge branch 'mouseify'
[debbugs.git] / cgi / pkgreport.cgi
index 062aef8113577a6aee988317763cb4a6aeb42dd4..3855928e042a941d388bf74bbd9b4045c912ca18 100755 (executable)
@@ -1,4 +1,4 @@
-#!/usr/bin/perl -wT
+#!/usr/bin/perl -T
 # This script is part of debbugs, and is released
 # under the terms of the GPL version 2, or any later
 # version at your option.
 # Copyright 2007 by Don Armstrong <don@donarmstrong.com>.
 
 
-package debbugs;
-
 use warnings;
 use strict;
+
+# Sanitize environent for taint
+BEGIN{
+    delete @ENV{qw(IFS CDPATH ENV BASH_ENV)};
+}
+
+# if we're running out of git, we want to use the git base directory as the
+# first INC directory. If you're not running out of git, don't do that.
+use File::Basename qw(dirname);
+use Cwd qw(abs_path);
+our $debbugs_dir;
+BEGIN {
+    $debbugs_dir =
+       abs_path(dirname(abs_path(__FILE__)) . '/../');
+    # clear the taint; we'll assume that the absolute path to __FILE__ is the
+    # right path if there's a .git directory there
+    ($debbugs_dir) = $debbugs_dir =~ /([[:print:]]+)/;
+    if (defined $debbugs_dir and
+       -d $debbugs_dir . '/.git/') {
+    } else {
+       undef $debbugs_dir;
+    }
+    # if the first directory in @INC is not an absolute directory, assume that
+    # someone has overridden us via -I.
+    if ($INC[0] !~ /^\//) {
+       undef $debbugs_dir;
+    }
+}
+use if defined $debbugs_dir, lib => $debbugs_dir;
+
+binmode(STDOUT,':encoding(UTF-8)');
 use POSIX qw(strftime nice);
+use List::AllUtils qw(uniq);
 
 use Debbugs::Config qw(:globals :text :config);
+
 use Debbugs::User;
-use Debbugs::CGI qw(version_url maint_decode);
-use Debbugs::Common qw(getparsedaddrs :date make_list getmaintainers getpseudodesc);
+
+use Debbugs::Common qw(getparsedaddrs make_list getmaintainers getpseudodesc);
+
 use Debbugs::Bugs qw(get_bugs bug_filter newest_bug);
-use Debbugs::Packages qw(getsrcpkgs getpkgsrc get_versions);
-use Debbugs::Status qw(:status);
+use Debbugs::Packages qw(source_to_binary binary_to_source get_versions);
+use Debbugs::Collection::Bug;
+
+use Debbugs::Status qw(splitpackages);
+
 use Debbugs::CGI qw(:all);
 
-use vars qw($gPackagePages $gWebDomain %gSeverityDisplay @gSeverityList);
+use Debbugs::CGI::Pkgreport qw(:all);
 
-if (defined $ENV{REQUEST_METHOD} and $ENV{REQUEST_METHOD} eq 'HEAD') {
-    print "Content-Type: text/html; charset=utf-8\n\n";
-    exit 0;
-}
+use Debbugs::Text qw(:templates);
+
+use Debbugs::DB;
 
-nice(5);
+my $s;
+my @schema_arg = ();
+if (defined $config{database}) {
+    $s = Debbugs::DB->connect($config{database}) or
+        die "Unable to connect to DB";
+    @schema_arg = ('schema',$s);
+}
 
 use CGI::Simple;
 my $q = new CGI::Simple;
 
+if ($q->request_method() eq 'HEAD') {
+     print $q->header(-type => "text/html",
+                     -charset => 'utf-8',
+                    );
+     exit 0;
+}
+
+my $default_params = {ordering => 'normal',
+                     archive  => 0,
+                     repeatmerged => 0,
+                     include      => [],
+                     exclude      => [],
+                    };
+
 our %param = cgi_parameters(query => $q,
                            single => [qw(ordering archive repeatmerged),
                                       qw(bug-rev pend-rev sev-rev),
                                       qw(maxdays mindays version),
-                                      qw(data which dist newest),
+                                      qw(data which dist),
+                                      qw(noaffects),
                                      ],
-                           default => {ordering => 'normal',
-                                       archive  => 0,
-                                       repeatmerged => 1,
-                                      },
+                           default => $default_params,
                           );
 
+my ($form_options,$param) = ({},undef);
+($form_options,$param)= form_options_and_normal_param(\%param)
+     if $param{form_options};
+
+%param = %{$param} if defined $param;
+
+if (exists $param{form_options} and defined $param{form_options}) {
+     delete $param{form_options};
+     delete $param{submit} if exists $param{submit};
+     for my $default (keys %{$default_params}) {
+         if (exists $param{$default} and
+             not ref($default_params->{$default}) and
+             $default_params->{$default} eq $param{$default}
+            ) {
+              delete $param{$default};
+         }
+     }
+     for my $incexc (qw(include exclude)) {
+         next unless exists $param{$incexc};
+         # normalize tag to tags
+         $param{$incexc} = [map {s/^tag:/tags:/; $_} grep /\S\:\S/, make_list($param{$incexc})];
+     }
+     for my $key (keys %package_search_keys) {
+         next unless exists $param{key};
+         $param{$key} = [map {split /\s*,\s*/} make_list($param{$key})];
+     }
+     # kill off keys for which empty values are meaningless
+     for my $key (qw(package src submitter affects severity status dist)) {
+         next unless exists $param{$key};
+         $param{$key} = [grep {defined $_ and length $_}
+                         make_list($param{$key})];
+     }
+     print $q->redirect(munge_url('pkgreport.cgi?',%param));
+     exit 0;
+}
+
+# normalize innclude/exclude keys; currently this is in two locations,
+# which is suboptimal. Closes: #567407
+for my $incexc (qw(include exclude)) {
+    next unless exists $param{$incexc};
+    # normalize tag to tags
+    $param{$incexc} = [map {s/^tag:/tags:/; $_} make_list($param{$incexc})];
+}
+
+
+
 # map from yes|no to 1|0
 for my $key (qw(repeatmerged bug-rev pend-rev sev-rev)) {
      if (exists $param{$key}){
@@ -68,8 +166,11 @@ elsif (lc($param{archive}) eq 'yes') {
      $param{archive} = 1;
 }
 
+# fixup dist
+if (exists $param{dist} and $param{dist} eq '') {
+     delete $param{dist};
+}
 
-my $archive = ($param{'archive'} || "no") eq "yes";
 my $include = $param{'&include'} || $param{'include'} || "";
 my $exclude = $param{'&exclude'} || $param{'exclude'} || "";
 
@@ -85,6 +186,8 @@ unless (defined $ordering) {
    $ordering = "raw" if $raw_sort;
    $ordering = 'age' if $age_sort;
 }
+$param{ordering} = $ordering;
+
 our ($bug_order) = $ordering =~ /(age(?:rev)?)/;
 $bug_order = '' if not defined $bug_order;
 
@@ -124,11 +227,11 @@ for my $incexcmap (@inc_exc_mapping) {
      delete $param{$incexcmap->{key}};
 }
 
+
 my $maxdays = ($param{'maxdays'} || -1);
 my $mindays = ($param{'mindays'} || 0);
 my $version = $param{'version'} || undef;
-# XXX Once the options/selection is rewritten, this should go away
-my $dist = $param{dist} || undef;
+
 
 our %hidden = map { $_, 1 } qw(status severity classification);
 our %cats = (
@@ -161,13 +264,10 @@ our %cats = (
         "ord" => [2,3,4,1,0,5],
     } ],
     "oldview" => [ qw(status severity) ],
-    "normal" => [ qw(status severity classification) ],
+            "normal" => [ qw(status severity classification) ],
+            raw => [{nam => 'Raw',def => 'Raw'}],
 );
 
-my @select_key = (qw(submitter maint pkg package src usertag),
-                 qw(status tag maintenc owner severity newest)
-                );
-
 if (exists $param{which} and exists $param{data}) {
      $param{$param{which}} = [exists $param{$param{which}}?(make_list($param{$param{which}})):(),
                              make_list($param{data}),
@@ -181,170 +281,172 @@ if (defined $param{maintenc}) {
      delete $param{maintenc}
 }
 
-
-if (not grep {exists $param{$_}} @select_key and exists $param{users}) {
-     $param{usertag} = [make_list($param{users})];
-}
-
-quitcgi("You have to choose something to select by") unless grep {exists $param{$_}} @select_key;
-
 if (exists $param{pkg}) {
      $param{package} = $param{pkg};
      delete $param{pkg};
 }
 
-our %bugusertags;
-our %ut;
+if (not grep {exists $param{$_}} keys %package_search_keys and exists $param{users}) {
+     $param{usertag} = [make_list($param{users})];
+}
+
+my %bugusertags;
+my %ut;
+my %seen_users;
+
+my @users;
 for my $user (map {split /[\s*,\s*]+/} make_list($param{users}||[])) {
     next unless length($user);
-    add_user($user);
+    push @users, $user;
+    add_user($user,\%ut,\%bugusertags,\%seen_users,\%cats,\%hidden);
 }
 
 if (defined $param{usertag}) {
-    my %select_ut = ();
-    my ($u, $t) = split /:/, $param{usertag}, 2;
-    Debbugs::User::read_usertags(\%select_ut, $u);
-    unless (defined $t && $t ne "") {
-        $t = join(",", keys(%select_ut));
-    }
-
-    add_user($u);
-    push @{$param{tag}}, split /,/, $t;
+     for my $usertag (make_list($param{usertag})) {
+         my %select_ut = ();
+         my ($u, $t) = split /:/, $usertag, 2;
+         Debbugs::User::read_usertags(\%select_ut, $u);
+         unless (defined $t && $t ne "") {
+              $t = join(",", keys(%select_ut));
+          }
+         push @users,$u;
+         add_user($u,\%ut,\%bugusertags,\%seen_users,\%cats,\%hidden);
+         push @{$param{tag}}, split /,/, $t;
+     }
 }
 
-my $Archived = $archive ? " Archived" : "";
+quitcgi("You have to choose something to select by", '400 Bad Request')
+  unless grep {exists $param{$_}} keys %package_search_keys;
+
 
-our $this = munge_url('pkgreport.cgi?',
+my $Archived = $param{archive} ? " Archived" : "";
+
+my $this = munge_url('pkgreport.cgi?',
                      %param,
                     );
 
 my %indexentry;
 my %strings = ();
 
-my $dtime = strftime "%a, %e %b %Y %T UTC", gmtime;
-my $tail_html = $debbugs::gHTMLTail;
-$tail_html = $debbugs::gHTMLTail;
-$tail_html =~ s/SUBSTITUTE_DTIME/$dtime/;
-
-our %seen_users;
-sub add_user {
-    my $ut = \%ut;
-    my $u = shift;
-
-    return if $seen_users{$u};
-    $seen_users{$u} = 1;
-
-    my $user = Debbugs::User::get_user($u);
-
-    my %vis = map { $_, 1 } @{$user->{"visible_cats"}};
-    for my $c (keys %{$user->{"categories"}}) {
-        $cats{$c} = $user->{"categories"}->{$c};
-       $hidden{$c} = 1 unless defined $vis{$c};
-    }
-
-    for my $t (keys %{$user->{"tags"}}) {
-        $ut->{$t} = [] unless defined $ut->{$t};
-        push @{$ut->{$t}}, @{$user->{"tags"}->{$t}};
-    }
-
-    %bugusertags = ();
-    for my $t (keys %{$ut}) {
-        for my $b (@{$ut->{$t}}) {
-            $bugusertags{$b} = [] unless defined $bugusertags{$b};
-            push @{$bugusertags{$b}}, $t;
-        }
-    }
-#    set_option("bugusertags", \%bugusertags);
-}
-
 my @bugs;
 
 # addusers for source and binary packages being searched for
-my $pkgsrc = getpkgsrc();
-my $srcpkg = getsrcpkgs();
-for my $package (# For binary packages, add the binary package
-                # and corresponding source package
-                make_list($param{package}||[]),
-                (map {defined $pkgsrc->{$_}?($pkgsrc->{$_}):()}
-                 make_list($param{package}||[]),
-                ),
-                # For source packages, add the source package
-                # and corresponding binary packages
-                make_list($param{src}||[]),
-                (map {defined $srcpkg->{$_}?($srcpkg->{$_}):()}
-                 make_list($param{src}||[]),
-                ),
-               ) {
-     next unless defined $package;
-     add_user($package.'@'.$config{usertag_package_domain})
-         if defined $config{usertag_package_domain};
+if (defined $config{usertag_package_domain}) {
+    my @possible_packages;
+    if (exists $param{package} and
+       defined $param{package}
+       ) {
+       # For binary packages, add the binary package and corresponding source package
+       push @possible_packages,
+           make_list($param{package});
+       push @possible_packages,
+           binary_to_source(source_only => 1,
+                            binary=>$param{package},
+                            @schema_arg,
+                           );
+    }
+    if (exists $param{src} and
+       defined $param{src}
+       ) {
+       # For source packages, add the source package and corresponding binary packages
+       push @possible_packages,
+           make_list($param{src});
+       push @possible_packages,
+           source_to_binary(binary_only => 1,
+                            source => $param{src},
+                            dist => [@{$config{distributions}}],
+                            @schema_arg,
+                           );
+    }
+    for my $package (@possible_packages) {
+       next unless defined $package and length $package;
+       push @users,
+           $package.'@'.$config{usertag_package_domain};
+       add_user($package.'@'.$config{usertag_package_domain},
+                \%ut,\%bugusertags,\%seen_users,\%cats,\%hidden);
+    }
 }
 
-
 # walk through the keys and make the right get_bugs query.
 
-my @search_key_order = (package   => 'in package',
-                       tag       => 'tagged',
-                       severity  => 'with severity',
-                       src       => 'in source package',
-                       maint     => 'in packages maintained by',
-                       submitter => 'submitted by',
-                       owner     => 'owned by',
-                       status    => 'with status',
-                      );
-my %search_keys = @search_key_order;
+my $form_option_variables = {};
+$form_option_variables->{search_key_order} = [@package_search_key_order];
 
 # Set the title sanely and clean up parameters
 my @title;
-while (my ($key,$value) = splice @search_key_order, 0, 2) {
+my @temp = @package_search_key_order;
+while (my ($key,$value) = splice @temp, 0, 2) {
      next unless exists $param{$key};
      my @entries = ();
-     $param{$key} = [map {split /\s*,\s*/} make_list($param{$key})];
      for my $entry (make_list($param{$key})) {
+         # we'll handle newest below
          my $extra = '';
          if (exists $param{dist} and ($key eq 'package' or $key eq 'src')) {
-              my @versions = get_versions(package => $entry,
+              my %versions = get_versions(package => $entry,
                                           (exists $param{dist}?(dist => $param{dist}):()),
-                                          (exists $param{arch}?(arch => $param{arch}):()),
+                                          (exists $param{arch}?(arch => $param{arch}):(arch => $config{default_architectures})),
                                           ($key eq 'src'?(arch => q(source)):()),
+                                          no_source_arch => 1,
+                                          return_archs => 1,
+                                          @schema_arg,
                                          );
-              my $verdesc = join(', ',@versions);
-              $verdesc = 'version'.(@versions>1?'s ':' ').$verdesc;
-              $extra= " ($verdesc)" if @versions;
+              my $verdesc;
+              if (keys %versions > 1) {
+                   $verdesc = 'versions '. join(', ',
+                                   map { $_ .' ['.join(', ',
+                                                   sort @{$versions{$_}}
+                                                  ).']';
+                                  } keys %versions);
+              }
+              else {
+                   $verdesc = 'version '.join(', ',
+                                              keys %versions
+                                             );
+              }
+              $extra= " ($verdesc)" if keys %versions;
+         }
+         if ($key eq 'maint' and $entry eq '') {
+              push @entries, "no one (packages without maintainers)"
          }
-         push @entries, $entry.$extra;
+         else {
+              push @entries, $entry.$extra;
+         }
+     }
+     if ($key eq 'newest') {
+        push @title, 'in '.join(' or ',@entries).' newest reports';
+     } else {
+        push @title,$value.' '.join(' or ', @entries) if @entries;
      }
-     push @title,$value.' '.join(' or ', @entries);
 }
+
 my $title = $gBugs.' '.join(' and ', map {/ or /?"($_)":$_} @title);
 @title = ();
 
-# we have to special case the maint="" search, unfortunatly.
-if (defined $param{maint} and $param{maint} eq "" or ref($param{maint}) and not @{$param{maint}}) {
-     my %maintainers = %{getmaintainers()};
-     @bugs = get_bugs(function =>
-                     sub {my %d=@_;
-                          foreach my $try (splitpackages($d{"pkg"})) {
-                               return 1 if !getparsedaddrs($maintainers{$try});
-                          }
-                          return 0;
-                     }
-                    );
-     $title = $gBugs.' in packages with no maintainer';
-}
-elsif (defined $param{newest}) {
-     my $newest_bug = newest_bug();
-     @bugs = ($newest_bug - $param{newest} + 1) .. $newest_bug;
-     $title = @bugs.' newest '.$gBugs;
-}
-else {
-     #yeah for magick!
-     @bugs = get_bugs((map {exists $param{$_}?($_,$param{$_}):()}
-                      keys %search_keys, 'archive'),
-                     usertags => \%ut,
-                    );
+#yeah for magick!
+@bugs = get_bugs((map {exists $param{$_}?($_,$param{$_}):()}
+                 keys %package_search_keys, 'archive'),
+                usertags => \%ut,
+                @schema_arg,
+               );
+
+# shove in bugs which affect this package if there is a package or a
+# source given (by default), but no affects options given
+if (not exists $param{affects} and not exists $param{noaffects} and
+    (exists $param{src} or
+     exists $param{package})) {
+    push @bugs, get_bugs((map {my $key = $_;
+                              exists $param{$key}?($key =~ /^(?:package|src)$/?'affects':$key,
+                                                 ($key eq 'src'?[map {"src:$_"}make_list($param{$key})]:$param{$_})):()}
+                         grep {$_ ne 'newest'}
+                         keys %package_search_keys, 'archive'),
+                        usertags => \%ut,
+                        @schema_arg,
+                       );
 }
 
+# filter out included or excluded bugs
+
+
 if (defined $param{version}) {
      $title .= " at version $param{version}";
 }
@@ -355,15 +457,49 @@ elsif (defined $param{dist}) {
 $title = html_escape($title);
 
 my @names; my @prior; my @order;
-determine_ordering();
+determine_ordering(cats => \%cats,
+                  param => \%param,
+                  ordering => \$ordering,
+                  names => \@names,
+                  prior => \@prior,
+                  title => \@title,
+                  order => \@order,
+                 );
 
 # strip out duplicate bugs
 my %bugs;
 @bugs{@bugs} = @bugs;
 @bugs = keys %bugs;
 
-my $result = pkg_htmlizebugs(\@bugs);
-
+my $bugs = Debbugs::Collection::Bug->
+    new(bugs => \@bugs,
+       @schema_arg,
+       users => [map {my $u = Debbugs::User->new($_);
+                      $u->has_bug_tags()?($u):()
+                  } @users],
+       );
+
+$bugs->load_related_packages_and_versions();
+
+my $result = pkg_htmlizebugs(bugs => $bugs,
+                            names => \@names,
+                            title => \@title,
+                            order => \@order,
+                            prior => \@prior,
+                            ordering => $ordering,
+                            bugusertags => \%bugusertags,
+                            bug_rev => $bug_rev,
+                            bug_order => $bug_order,
+                            repeatmerged => $param{repeatmerged},
+                            include => $include,
+                            exclude => $exclude,
+                            this => $this,
+                            options => \%param,
+                             @schema_arg,
+                            (exists $param{dist})?(dist    => $param{dist}):(),
+                           );
+
+print "Cache-Control: public, max-age=300\n";
 print "Content-Type: text/html; charset=utf-8\n\n";
 
 print "<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\">\n";
@@ -373,6 +509,7 @@ print "<HTML><HEAD>\n" .
     "</HEAD>\n" .
     '<BODY onload="pagemain();">' .
     "\n";
+print qq(<DIV id="status_mask"></DIV>\n);
 print "<H1>" . "$gProject$Archived $gBug report logs: $title" .
       "</H1>\n";
 
@@ -386,83 +523,23 @@ if (defined $pseudodesc and defined $pkg and exists $pseudodesc->{$pkg}) {
      delete $param{dist};
 }
 
-# output infomration about the packages
+# output information about the packages
 
 for my $package (make_list($param{package}||[])) {
-     output_package_info('binary',$package);
+     print generate_package_info(binary => 1,
+                                package => $package,
+                                options => \%param,
+                                bugs    => \@bugs,
+                                @schema_arg,
+                               );
 }
 for my $package (make_list($param{src}||[])) {
-     output_package_info('source',$package);
-}
-
-sub output_package_info{
-    my ($srcorbin,$package) = @_;
-    my $showpkg = html_escape($package);
-    my $maintainers = getmaintainers();
-    my $maint = $maintainers->{$package};
-    if (defined $maint) {
-        print '<p>';
-        print htmlize_maintlinks(sub { $_[0] == 1 ? "Maintainer for $showpkg is "
-                                        : "Maintainers for $showpkg are "
-                                   },
-                             $maint);
-        print ".</p>\n";
-    } else {
-        print "<p>No maintainer for $showpkg. Please do not report new bugs against this package.</p>\n";
-    }
-    my %pkgsrc = %{getpkgsrc()};
-    my $srcforpkg = $package;
-    if ($srcorbin eq 'binary') {
-        $srcforpkg = $pkgsrc{$package};
-        defined $srcforpkg or $srcforpkg = $package;
-    }
-    my @pkgs = getsrcpkgs($srcforpkg);
-    @pkgs = grep( !/^\Q$package\E$/, @pkgs );
-    if ( @pkgs ) {
-        @pkgs = sort @pkgs;
-        if ($srcorbin eq 'binary') {
-             print "<p>You may want to refer to the following packages that are part of the same source:\n";
-        } else {
-             print "<p>You may want to refer to the following individual bug pages:\n";
-        }
-        #push @pkgs, $src if ( $src && !grep(/^\Q$src\E$/, @pkgs) );
-        print join( ", ", map( "<A href=\"" . html_escape(munge_url($this,package=>$_,src=>[],newest=>[])) . "\">$_</A>", @pkgs ) );
-        print ".\n";
-    }
-    my @references;
-    my $pseudodesc = getpseudodesc();
-    if ($package and defined($pseudodesc) and exists($pseudodesc->{$package})) {
-        push @references, "to the <a href=\"http://${debbugs::gWebDomain}/pseudo-packages${debbugs::gHTMLSuffix}\">".
-             "list of other pseudo-packages</a>";
-    } else {
-        if ($package and defined $gPackagePages) {
-             push @references, sprintf "to the <a href=\"%s\">%s package page</a>",
-                  html_escape("http://${debbugs::gPackagePages}/$package"), html_escape("$package");
-        }
-        if (defined $gSubscriptionDomain) {
-             my $ptslink = $package ? $srcforpkg : $src;
-             push @references, "to the <a href=\"http://$gSubscriptionDomain/$ptslink\">Package Tracking System</a>";
-        }
-        # Only output this if the source listing is non-trivial.
-        if ($srcorbin eq 'binary' and $srcforpkg) {
-             push @references, sprintf "to the source package <a href=\"%s\">%s</a>'s bug page", html_escape(munge_url($this,src=>$srcforpkg,package=>[],newest=>[])), html_escape($srcforpkg);
-        }
-    }
-    if (@references) {
-        $references[$#references] = "or $references[$#references]" if @references > 1;
-        print "<p>You might like to refer ", join(", ", @references), ".</p>\n";
-    }
-    if (defined $param{maint} || defined $param{maintenc}) {
-        print "<p>If you find a bug not listed here, please\n";
-        printf "<a href=\"%s\">report it</a>.</p>\n",
-             html_escape("http://${debbugs::gWebDomain}/Reporting${debbugs::gHTMLSuffix}");
-    }
-    if (not $maint and not @bugs) {
-        print "<p>There is no record of the " . html_escape($package) .
-             ($srcorbin eq 'binary' ? " package" : " source package") .
-                   ", and no bugs have been filed against it.</p>";
-        $showresult = 0;
-    }
+     print generate_package_info(binary => 0,
+                                package => $package,
+                                options => \%param,
+                                bugs    => \@bugs,
+                                @schema_arg,
+                               );
 }
 
 if (exists $param{maint} or exists $param{maintenc}) {
@@ -476,677 +553,25 @@ if (exists $param{submitter}) {
     print "different addresses.\n";
 }
 
-my $archive_links;
-my @archive_links;
-my %archive_values = (both => 'archived and unarchived',
-                     0    => 'not archived',
-                     1    => 'archived',
-                    );
-while (my ($key,$value) = each %archive_values) {
-     next if $key eq lc($param{archive});
-     push @archive_links, qq(<a href=").
-         html_escape(pkg_url((
-                      map {
-                           $_ eq 'archive'?():($_,$param{$_})
-                      } keys %param),
-                           archive => $key
-                          )).qq(">$value reports </a>);
-}
-print '<p>See the '.join (' or ',@archive_links)."</p>\n";
-
-print $result if $showresult;
-
-print pkg_javascript() . "\n";
-print "<h2 class=\"outstanding\"><a class=\"options\" href=\"javascript:toggle(1)\">Options</a></h2>\n";
-print "<div id=\"a_1\">\n";
-printf "<form action=\"%s\" method=POST>\n", myurl();
-
-print "<table class=\"forms\">\n";
-
-my ($checked_any, $checked_sui, $checked_ver) = ("", "", "");
-if (defined $dist) {
-  $checked_sui = "CHECKED";
-} elsif (defined $version) {
-  $checked_ver = "CHECKED";
-} else {
-  $checked_any = "CHECKED";
-}
-
-print "<tr><td>Show bugs applicable to</td>\n";
-print "    <td><input id=\"b_1_1\" name=vt value=none type=radio onchange=\"enable(1);\" $checked_any>anything</td></tr>\n";
-print "<tr><td></td>";
-print "    <td><input id=\"b_1_2\" name=vt value=bysuite type=radio onchange=\"enable(1);\" $checked_sui>" . pkg_htmlselectsuite(1,2,1) . " for " . pkg_htmlselectarch(1,2,2) . "</td></tr>\n";
-
-if (defined $pkg) {
-    my $v = html_escape($version) || "";
-    my $pkgsane = html_escape($pkg);
-    print "<tr><td></td>";
-    print "    <td><input id=\"b_1_3\" name=vt value=bypkg type=radio onchange=\"enable(1);\" $checked_ver>$pkgsane version <input id=\"b_1_3_1\" name=version value=\"$v\"></td></tr>\n";
-} elsif (defined $src) {
-    my $v = html_escape($version) || "";
-    my $srcsane = html_escape($src);
-    print "<tr><td></td>";
-    print "    <td><input name=vt value=bysrc type=radio onchange=\"enable(1);\" $checked_ver>$srcsane version <input id=\"b_1_3_1\" name=version value=\"$v\"></td></tr>\n";
-}
-print "<tr><td>&nbsp;</td></tr>\n";
-
-my $includetags = html_escape(join(" ", grep { !m/^subj:/i } map {split /[\s,]+/} ref($include)?@{$include}:$include));
-my $excludetags = html_escape(join(" ", grep { !m/^subj:/i } map {split /[\s,]+/} ref($exclude)?@{$exclude}:$exclude));
-my $includesubj = html_escape(join(" ", map { s/^subj://i; $_ } grep { m/^subj:/i } map {split /[\s,]+/} ref($include)?@{$include}:$include));
-my $excludesubj = html_escape(join(" ", map { s/^subj://i; $_ } grep { m/^subj:/i } map {split /[\s,]+/} ref($exclude)?@{$exclude}:$exclude));
-my $vismindays = ($mindays == 0 ? "" : $mindays);
-my $vismaxdays = ($maxdays == -1 ? "" : $maxdays);
-
-my $sel_rmy = ($param{repeatmerged} ? " selected" : "");
-my $sel_rmn = ($param{repeatmerged} ? "" : " selected");
-my $sel_ordraw = ($ordering eq "raw" ? " selected" : "");
-my $sel_ordold = ($ordering eq "oldview" ? " selected" : "");
-my $sel_ordnor = ($ordering eq "normal" ? " selected" : "");
-my $sel_ordage = ($ordering eq "age" ? " selected" : "");
-
-my $chk_bugrev = ($bug_rev ? " checked" : "");
-my $chk_pendrev = ($pend_rev ? " checked" : "");
-my $chk_sevrev = ($sev_rev ? " checked" : "");
-
-print <<EOF;
-<tr><td>Only include bugs tagged with </td><td><input name=include value="$includetags"> or that have <input name=includesubj value="$includesubj"> in their subject</td></tr>
-<tr><td>Exclude bugs tagged with </td><td><input name=exclude value="$excludetags"> or that have <input name=excludesubj value="$excludesubj"> in their subject</td></tr>
-<tr><td>Only show bugs older than</td><td><input name=mindays value="$vismindays" size=5> days, and younger than <input name=maxdays value="$vismaxdays" size=5> days</td></tr>
-
-<tr><td>&nbsp;</td></tr>
-
-</td></tr>
-<tr><td>Merged bugs should be</td><td>
-<select name=repeatmerged>
-<option value=yes$sel_rmy>displayed separately</option>
-<option value=no$sel_rmn>combined</option>
-</select>
-<tr><td>Categorise bugs by</td><td>
-<select name=ordering>
-<option value=raw$sel_ordraw>bug number only</option>
-<option value=old$sel_ordold>status and severity</option>
-<option value=normal$sel_ordnor>status, severity and classification</option>
-<option value=age$sel_ordage>status, severity, classification, and age</option>
-EOF
-
-{
-my $any = 0;
-my $o = $param{"ordering"} || "";
-for my $n (keys %cats) {
-    next if ($n eq "normal" || $n eq "oldview");
-    next if defined $hidden{$n};
-    unless ($any) {
-        $any = 1;
-       print "<option disabled>------</option>\n";
-    }
-    my @names = map { ref($_) eq "HASH" ? $_->{"nam"} : $_ } @{$cats{$n}};
-    my $name;
-    if (@names == 1) { $name = $names[0]; }
-    else { $name = " and " . pop(@names); $name = join(", ", @names) . $name; }
-
-    printf "<option value=\"%s\"%s>%s</option>\n",
-        $n, ($o eq $n ? " selected" : ""), $name;
-}
-}
-
-print "</select></td></tr>\n";
+print $result;
 
-printf "<tr><td>Order bugs by</td><td>%s</td></tr>\n",
-    pkg_htmlselectyesno("pend-rev", "outstanding bugs first", "done bugs first", $pend_rev);
-printf "<tr><td></td><td>%s</td></tr>\n",
-    pkg_htmlselectyesno("sev-rev", "highest severity first", "lowest severity first", $sev_rev);
-printf "<tr><td></td><td>%s</td></tr>\n",
-    pkg_htmlselectyesno("bug-rev", "oldest bugs first", "newest bugs first", $bug_rev);
+print fill_in_template(template=>'cgi/pkgreport_javascript');
 
-print <<EOF;
-<tr><td>&nbsp;</td></tr>
-<tr><td colspan=2><input value="Reload page" type="submit"> with new settings</td></tr>
-EOF
+print qq(<h2 class="outstanding"><!--<a class="options" href="javascript:toggle(1)">-->Options<!--</a>--></h2>\n);
 
-print "</table></form></div>\n";
+$param{orderings} =
+    [uniq((grep {!$hidden{$_}} keys %cats),
+         $param{ordering})];
+print option_form(template => 'cgi/pkgreport_options',
+                 param    => \%param,
+                 form_options => $form_options,
+                 variables => $form_option_variables,
+                );
 
 print "<hr>\n";
-print "<p>$tail_html";
-
+print fill_in_template(template=>'html/html_tail',
+                      hole_var => {'&strftime' => \&POSIX::strftime,
+                                  },
+                     );
 print "</body></html>\n";
 
-sub pkg_htmlindexentrystatus {
-    my $s = shift;
-    my %status = %{$s};
-
-    my $result = "";
-
-    my $showseverity;
-    if  ($status{severity} eq 'normal') {
-        $showseverity = '';
-    } elsif (isstrongseverity($status{severity})) {
-        $showseverity = "Severity: <em class=\"severity\">$status{severity}</em>;\n";
-    } else {
-        $showseverity = "Severity: <em>$status{severity}</em>;\n";
-    }
-
-    $result .= pkg_htmlpackagelinks($status{"package"}, 1);
-
-    my $showversions = '';
-    if (@{$status{found_versions}}) {
-        my @found = @{$status{found_versions}};
-        $showversions .= join ', ', map {s{/}{ }; html_escape($_)} @found;
-    }
-    if (@{$status{fixed_versions}}) {
-        $showversions .= '; ' if length $showversions;
-        $showversions .= '<strong>fixed</strong>: ';
-        my @fixed = @{$status{fixed_versions}};
-        $showversions .= join ', ', map {s{/}{ }; html_escape($_)} @fixed;
-    }
-    $result .= ' (<a href="'.
-        version_url($status{package},
-                    $status{found_versions},
-                    $status{fixed_versions},
-                   ).qq{">$showversions</a>)} if length $showversions;
-    $result .= ";\n";
-
-    $result .= $showseverity;
-    $result .= pkg_htmladdresslinks("Reported by: ", \&submitterurl,
-                                $status{originator});
-    $result .= ";\nOwned by: " . html_escape($status{owner})
-               if length $status{owner};
-    $result .= ";\nTags: <strong>" 
-                 . html_escape(join(", ", sort(split(/\s+/, $status{tags}))))
-                 . "</strong>"
-                       if (length($status{tags}));
-
-    $result .= buglinklist(";\nMerged with ", ", ",
-        split(/ /,$status{mergedwith}));
-    $result .= buglinklist(";\nBlocked by ", ", ",
-        split(/ /,$status{blockedby}));
-    $result .= buglinklist(";\nBlocks ", ", ",
-        split(/ /,$status{blocks}));
-
-    if (length($status{done})) {
-        $result .= "<br><strong>Done:</strong> " . html_escape($status{done});
-        my $days = bug_archiveable(bug => $status{id},
-                                  status => \%status,
-                                  days_until => 1,
-                                 );
-        if ($days >= 0 and defined $status{location} and $status{location} ne 'archive') {
-            $result .= ";\n<strong>Can be archived" . ( $days == 0 ? " today" : $days == 1 ? " in $days day" : " in $days days" ) . "</strong>";
-        }
-       elsif (defined $status{location} and $status{location} eq 'archived') {
-            $result .= ";\n<strong>Archived.</strong>";
-       }
-    }
-
-    unless (length($status{done})) {
-        if (length($status{forwarded})) {
-            $result .= ";\n<strong>Forwarded</strong> to "
-                       . join(', ',
-                             map {maybelink($_)}
-                             split /[,\s]+/,$status{forwarded}
-                            );
-        }
-       # Check the age of the logfile
-       my ($days_last,$eng_last) = secs_to_english(time - $status{log_modified});
-        my ($days,$eng) = secs_to_english(time - $status{date});
-       
-        if ($days >= 7) {
-            my $font = "";
-            my $efont = "";
-            $font = "em" if ($days > 30);
-            $font = "strong" if ($days > 60);
-            $efont = "</$font>" if ($font);
-            $font = "<$font>" if ($font);
-
-            $result .= ";\n ${font}$eng old$efont";
-        }
-       if ($days_last > 7) {
-           my $font = "";
-            my $efont = "";
-            $font = "em" if ($days_last > 30);
-            $font = "strong" if ($days_last > 60);
-            $efont = "</$font>" if ($font);
-            $font = "<$font>" if ($font);
-
-            $result .= ";\n ${font}Modified $eng_last ago$efont";
-       }
-    }
-
-    $result .= ".";
-
-    return $result;
-}
-
-
-sub pkg_htmlizebugs {
-    $b = $_[0];
-    my @bugs = @$b;
-
-    my @status = ();
-    my %count;
-    my $header = '';
-    my $footer = "<h2 class=\"outstanding\">Summary</h2>\n";
-
-    my @dummy = ($gRemoveAge); #, @gSeverityList, @gSeverityDisplay);  #, $gHTMLExpireNote);
-
-    if (@bugs == 0) {
-        return "<HR><H2>No reports found!</H2></HR>\n";
-    }
-
-    if ( $bug_rev ) {
-        @bugs = sort {$b<=>$a} @bugs;
-    } else {
-        @bugs = sort {$a<=>$b} @bugs;
-    }
-    my %seenmerged;
-
-    my %common = (
-        'show_list_header' => 1,
-        'show_list_footer' => 1,
-    );
-
-    my %section = ();
-    # Make the include/exclude map
-    my %include;
-    my %exclude;
-    for my $include (make_list($param{include})) {
-        next unless defined $include;
-        my ($key,$value) = split /\s*:\s*/,$include,2;
-        next unless defined $value;
-        push @{$include{$key}}, split /\s*,\s*/, $value;
-    }
-    for my $exclude (make_list($param{exclude})) {
-        next unless defined $exclude;
-        my ($key,$value) = split /\s*:\s*/,$exclude,2;
-        next unless defined $value;
-        push @{$exclude{$key}}, split /\s*,\s*/, $value;
-    }
-
-    foreach my $bug (@bugs) {
-        my %status = %{get_bug_status(bug=>$bug,
-                                     (exists $param{dist}?(dist => $param{dist}):()),
-                                     bugusertags => \%bugusertags,
-                                     (exists $param{version}?(version => $param{version}):()),
-                                     (exists $param{arch}?(arch => $param{arch}):()),
-                                    )};
-        next unless %status;
-        next if bug_filter(bug => $bug,
-                          status => \%status,
-                          (exists $param{repeatmerged}?(repeat_merged => $param{repeatmerged}):()),
-                          seen_merged => \%seenmerged,
-                          (keys %include ? (include => \%include):()),
-                          (keys %exclude ? (exclude => \%exclude):()),
-                         );
-
-       my $html = sprintf "<li><a href=\"%s\">#%d: %s</a>\n<br>",
-            bug_url($bug), $bug, html_escape($status{subject});
-        $html .= pkg_htmlindexentrystatus(\%status) . "\n";
-       push @status, [ $bug, \%status, $html ];
-    }
-    if ($bug_order eq 'age') {
-        # MWHAHAHAHA
-        @status = sort {$a->[1]{log_modified} <=> $b->[1]{log_modified}} @status;
-    }
-    elsif ($bug_order eq 'agerev') {
-        @status = sort {$b->[1]{log_modified} <=> $a->[1]{log_modified}} @status;
-    }
-    for my $entry (@status) {
-        my $key = "";
-       for my $i (0..$#prior) {
-           my $v = get_bug_order_index($prior[$i], $entry->[1]);
-            $count{"g_${i}_${v}"}++;
-           $key .= "_$v";
-       }
-        $section{$key} .= $entry->[2];
-        $count{"_$key"}++;
-    }
-
-    my $result = "";
-    if ($ordering eq "raw") {
-        $result .= "<UL class=\"bugs\">\n" . join("", map( { $_->[ 2 ] } @status ) ) . "</UL>\n";
-    } else {
-        $header .= "<ul>\n<div class=\"msgreceived\">\n";
-       my @keys_in_order = ("");
-       for my $o (@order) {
-           push @keys_in_order, "X";
-           while ((my $k = shift @keys_in_order) ne "X") {
-               for my $k2 (@{$o}) {
-                   $k2+=0;
-                   push @keys_in_order, "${k}_${k2}";
-               }
-           }
-       }
-        for my $order (@keys_in_order) {
-            next unless defined $section{$order};
-           my @ttl = split /_/, $order; shift @ttl;
-           my $title = $title[0]->[$ttl[0]] . " bugs";
-           if ($#ttl > 0) {
-               $title .= " -- ";
-               $title .= join("; ", grep {($_ || "") ne ""}
-                       map { $title[$_]->[$ttl[$_]] } 1..$#ttl);
-           }
-           $title = html_escape($title);
-
-            my $count = $count{"_$order"};
-            my $bugs = $count == 1 ? "bug" : "bugs";
-
-            $header .= "<li><a href=\"#$order\">$title</a> ($count $bugs)</li>\n";
-            if ($common{show_list_header}) {
-                my $count = $count{"_$order"};
-                my $bugs = $count == 1 ? "bug" : "bugs";
-                $result .= "<H2 CLASS=\"outstanding\"><a name=\"$order\"></a>$title ($count $bugs)</H2>\n";
-            } else {
-                $result .= "<H2 CLASS=\"outstanding\">$title</H2>\n";
-            }
-            $result .= "<div class=\"msgreceived\">\n<UL class=\"bugs\">\n";
-           $result .= "\n\n\n\n";
-            $result .= $section{$order};
-           $result .= "\n\n\n\n";
-            $result .= "</UL>\n</div>\n";
-        } 
-        $header .= "</ul></div>\n";
-
-        $footer .= "<ul>\n<div class=\"msgreceived\">";
-        for my $i (0..$#prior) {
-            my $local_result = '';
-            foreach my $key ( @{$order[$i]} ) {
-                my $count = $count{"g_${i}_$key"};
-                next if !$count or !$title[$i]->[$key];
-                $local_result .= "<li>$count $title[$i]->[$key]</li>\n";
-            }
-            if ( $local_result ) {
-                $footer .= "<li>$names[$i]<ul>\n$local_result</ul></li>\n";
-            }
-        }
-        $footer .= "</div></ul>\n";
-    }
-
-    $result = $header . $result if ( $common{show_list_header} );
-    $result .= $footer if ( $common{show_list_footer} );
-    return $result;
-}
-
-sub pkg_htmlpackagelinks {
-    my $pkgs = shift;
-    return unless defined $pkgs and $pkgs ne '';
-    my $strong = shift;
-    my @pkglist = splitpackages($pkgs);
-
-    $strong = 0;
-    my $openstrong  = $strong ? '<strong>' : '';
-    my $closestrong = $strong ? '</strong>' : '';
-
-    return 'Package' . (@pkglist > 1 ? 's' : '') . ': ' .
-           join(', ',
-                map {
-                    '<a class="submitter" href="' . munge_url($this,src=>[],package=>$_,newest=>[]) . '">' .
-                    $openstrong . html_escape($_) . $closestrong . '</a>'
-                } @pkglist
-           );
-}
-
-sub pkg_htmladdresslinks {
-     htmlize_addresslinks(@_,'submitter');
-}
-
-sub pkg_javascript {
-    return <<EOF ;
-<script type="text/javascript">
-<!--
-function pagemain() {
-       toggle(1);
-//     toggle(2);
-       enable(1);
-}
-
-function setCookie(name, value, expires, path, domain, secure) {
-  var curCookie = name + "=" + escape(value) +
-      ((expires) ? "; expires=" + expires.toGMTString() : "") +
-      ((path) ? "; path=" + path : "") +
-      ((domain) ? "; domain=" + domain : "") +
-      ((secure) ? "; secure" : "");
-  document.cookie = curCookie;
-}
-
-function save_cat_cookies() {
-  var cat = document.categories.categorisation.value;
-  var exp = new Date();
-  exp.setTime(exp.getTime() + 10 * 365 * 24 * 60 * 60 * 1000);
-  var oldexp = new Date();
-  oldexp.setTime(oldexp.getTime() - 1 * 365 * 24 * 60 * 60 * 1000);
-  var lev;
-  var done = 0;
-
-  var u = document.getElementById("users");
-  if (u != null) { u = u.value; }
-  if (u == "") { u = null; }
-  if (u != null) {
-      setCookie("cat" + cat + "_users", u, exp, "/");
-  } else {
-      setCookie("cat" + cat + "_users", "", oldexp, "/");
-  }
-
-  var bits = new Array("nam", "pri", "ttl", "ord");
-  for (var i = 0; i < 4; i++) {
-      for (var j = 0; j < bits.length; j++) {
-          var e = document.getElementById(bits[j] + i);
-         if (e) e = e.value;
-         if (e == null) { e = ""; }
-         if (j == 0 && e == "") { done = 1; }
-         if (done || e == "") {
-              setCookie("cat" + cat + "_" + bits[j] + i, "", oldexp, "/");
-         } else {
-              setCookie("cat" + cat + "_" + bits[j] + i, e, exp, "/");
-         }
-      }
-  }
-}
-
-function toggle(i) {
-        var a = document.getElementById("a_" + i);
-        if (a) {
-             if (a.style.display == "none") {
-                     a.style.display = "";
-             } else {
-                     a.style.display = "none";
-             }
-        }
-}
-
-function enable(x) {
-    for (var i = 1; ; i++) {
-        var a = document.getElementById("b_" + x + "_" + i);
-        if (a == null) break;
-        var ischecked = a.checked;
-        for (var j = 1; ; j++) {
-            var b = document.getElementById("b_" + x + "_"+ i + "_" + j);
-            if (b == null) break;
-            if (ischecked) {
-                b.disabled = false;
-            } else {
-                b.disabled = true;
-            }
-        }
-    }
-}
--->
-</script>
-EOF
-}
-
-sub pkg_htmlselectyesno {
-    my ($name, $n, $y, $default) = @_;
-    return sprintf('<select name="%s"><option value=no%s>%s</option><option value=yes%s>%s</option></select>', $name, ($default ? "" : " selected"), $n, ($default ? " selected" : ""), $y);
-}
-
-sub pkg_htmlselectsuite {
-    my $id = sprintf "b_%d_%d_%d", $_[0], $_[1], $_[2];
-    my @suites = ("stable", "testing", "unstable", "experimental");
-    my %suiteaka = ("stable", "etch", "testing", "lenny", "unstable", "sid");
-    my $defaultsuite = "unstable";
-
-    my $result = sprintf '<select name=dist id="%s">', $id;
-    for my $s (@suites) {
-        $result .= sprintf '<option value="%s"%s>%s%s</option>',
-                $s, ($defaultsuite eq $s ? " selected" : ""),
-                $s, (defined $suiteaka{$s} ? " (" . $suiteaka{$s} . ")" : "");
-    }
-    $result .= '</select>';
-    return $result;
-}
-
-sub pkg_htmlselectarch {
-    my $id = sprintf "b_%d_%d_%d", $_[0], $_[1], $_[2];
-    my @arches = qw(alpha amd64 arm hppa i386 ia64 m68k mips mipsel powerpc s390 sparc);
-
-    my $result = sprintf '<select name=arch id="%s">', $id;
-    $result .= '<option value="any">any architecture</option>';
-    for my $a (@arches) {
-        $result .= sprintf '<option value="%s">%s</option>', $a, $a;
-    }
-    $result .= '</select>';
-    return $result;
-}
-
-sub myurl {
-     return html_escape(pkg_url(map {exists $param{$_}?($_,$param{$_}):()}
-                            qw(archive repeatmerged mindays maxdays),
-                            qw(version dist arch pkg src tag maint submitter)
-                           )
-                   );
-}
-
-sub make_order_list {
-    my $vfull = shift;
-    my @x = ();
-
-    if ($vfull =~ m/^([^:]+):(.*)$/) {
-        my $v = $1;
-        for my $vv (split /,/, $2) {
-           push @x, "$v=$vv";
-       }
-    } else {
-       for my $v (split /,/, $vfull) {
-            next unless $v =~ m/.=./;
-            push @x, $v;
-        }
-    }
-    push @x, "";  # catch all
-    return @x;
-}
-
-sub get_bug_order_index {
-    my $order = shift;
-    my $status = shift;
-    my $pos = -1;
-
-    my %tags = ();
-    %tags = map { $_, 1 } split / /, $status->{"tags"}
-        if defined $status->{"tags"};
-
-    for my $el (@${order}) {
-       $pos++;
-        my $match = 1;
-        for my $item (split /[+]/, $el) {
-           my ($f, $v) = split /=/, $item, 2;
-           next unless (defined $f and defined $v);
-           my $isokay = 0;
-           $isokay = 1 if (defined $status->{$f} and $v eq $status->{$f});
-           $isokay = 1 if ($f eq "tag" && defined $tags{$v});
-           unless ($isokay) {
-               $match = 0;
-               last;
-           }
-        }
-        if ($match) {
-            return $pos;
-            last;
-        }
-    }
-    return $pos + 1;
-}
-
-sub buglinklist {
-    my ($prefix, $infix, @els) = @_;
-    return '' if not @els;
-    return $prefix . bug_linklist($infix,'submitter',@els);
-}
-
-
-# sets: my @names; my @prior; my @title; my @order;
-
-sub determine_ordering {
-    $cats{status}[0]{ord} = [ reverse @{$cats{status}[0]{ord}} ]
-        if ($pend_rev);
-    $cats{severity}[0]{ord} = [ reverse @{$cats{severity}[0]{ord}} ]
-        if ($sev_rev);
-
-    my $i;
-    if (defined $param{"pri0"}) {
-        my @c = ();
-        $i = 0;
-        while (defined $param{"pri$i"}) {
-            my $h = {};
-
-            my ($pri) = make_list($param{"pri$i"});
-            if ($pri =~ m/^([^:]*):(.*)$/) {
-              $h->{"nam"} = $1;  # overridden later if necesary
-              $h->{"pri"} = [ map { "$1=$_" } (split /,/, $2) ];
-            } else {
-              $h->{"pri"} = [ split /,/, $pri ];
-            }
-
-           ($h->{"nam"}) = make_list($param{"nam$i"})
-                if (defined $param{"nam$i"});
-            $h->{"ord"} = [ split /\s*,\s*/, make_list($param{"ord$i"}) ]
-                if (defined $param{"ord$i"});
-            $h->{"ttl"} = [ split /\s*,\s*/, make_list($param{"ttl$i"}) ]
-                if (defined $param{"ttl$i"});
-
-            push @c, $h;
-           $i++;
-        }
-        $cats{"_"} = [@c];
-        $ordering = "_";
-    }
-
-    $ordering = "normal" unless defined $cats{$ordering};
-
-    sub get_ordering {
-        my @res;
-       my $cats = shift;
-        my $o = shift;
-        for my $c (@{$cats->{$o}}) {
-            if (ref($c) eq "HASH") {
-                push @res, $c;
-            } else {
-                push @res, get_ordering($cats, $c);
-            }
-        }
-        return @res;
-    }
-    my @cats = get_ordering(\%cats, $ordering);
-
-    sub toenglish {
-        my $expr = shift;
-        $expr =~ s/[+]/ and /g;
-        $expr =~ s/[a-z]+=//g;
-        return $expr;
-    }
-    $i = 0;
-    for my $c (@cats) {
-       $i++;
-        push @prior, $c->{"pri"};
-       push @names, ($c->{"nam"} || "Bug attribute #" . $i);
-        if (defined $c->{"ord"}) {
-            push @order, $c->{"ord"};
-        } else {
-            push @order, [ 0..$#{$prior[-1]} ];
-        }
-        my @t = @{ $c->{"ttl"} } if defined $c->{ttl};
-       if (@t < $#{$prior[-1]}) {
-            push @t, map { toenglish($prior[-1][$_]) } @t..($#{$prior[-1]});
-       }
-       push @t, $c->{"def"} || "";
-        push @title, [@t];
-    }
-}