repobrowse: DRY-up query class and support id2

We'll be adding diff support against arbitrary objects, soon
(which could cause us to be OOM-ed easily if we're not careful
 with big objects).

1 files changed, 4 insertions, 2 deletions

diff --git a/lib/PublicInbox/RepoBrowseQuery.pm b/lib/PublicInbox/RepoBrowseQuery.pm
index 979a2b68..861e587b 100644
--- a/lib/PublicInbox/RepoBrowseQuery.pm
+++ b/lib/PublicInbox/RepoBrowseQuery.pm
@@ -1,16 +1,18 @@
 # Copyright (C) 2015 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 
+# query parameter management for repobrowse
 package PublicInbox::RepoBrowseQuery;
 use strict;
 use warnings;
 use PublicInbox::Hval;
+my @KNOWN_PARAMS = qw(id id2 h showmsg ofs);
 
 sub new {
         my ($class, $cgi) = @_;
         my $self = bless {}, $class;
 
-        foreach my $k (qw(id h showmsg ofs)) {
+        foreach my $k (@KNOWN_PARAMS) {
                 my $v = $cgi->param($k);
                 $self->{$k} = defined $v ? $v : '';
         }
@@ -27,7 +29,7 @@ sub qs {
         }
 
         my @qs;
-        foreach my $k (qw(id h showmsg ofs)) {
+        foreach my $k (@KNOWN_PARAMS) {
                 my $v = $self->{$k};
 
                 next if ($v eq '');