From: burnus%gmx.de <>
Date: Tue, 29 Apr 2003 01:29:34 +0000 (+0000)
Subject: Bug 200198 - user-error.html.tmpl's use [% changedsince %] instead of $changedsince
X-Git-Tag: bugzilla-2.17.5~164
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=7e6f8dbfebfeda16a14f951370f31ee5fad9e0ee;p=thirdparty%2Fbugzilla.git

Bug 200198 - user-error.html.tmpl's use [% changedsince %] instead of $changedsince
r=gerv, a=justdave
---

diff --git a/duplicates.cgi b/duplicates.cgi
index 64a3f7ab3c..dc65ef5026 100755
--- a/duplicates.cgi
+++ b/duplicates.cgi
@@ -137,6 +137,15 @@ while (my ($key, $value) = each %count) {
     delete $count{$key} if $sortvisible && (lsearch(\@buglist, $key) == -1);
 }
 
+my $origmaxrows = $maxrows;
+detaint_natural($maxrows)
+  || ThrowUserError("invalid_maxrows", { maxrows => $origmaxrows});
+
+my $origchangedsince = $changedsince;
+detaint_natural($changedsince)
+  || ThrowUserError("invalid_changedsince", 
+                    { changedsince => $origchangedsince });
+
 # Try and open the database from "changedsince" days ago
 my $dobefore = 0;
 my %delta;
@@ -159,15 +168,6 @@ if (!tie(%before, 'AnyDBM_File', "data/duplicates/dupes$whenever",
     $dobefore = 1;
 }
 
-my $origmaxrows = $maxrows;
-detaint_natural($maxrows)
-  || ThrowUserError("invalid_maxrows", { maxrows => $origmaxrows});
-
-my $origchangedsince = $changedsince;
-detaint_natural($changedsince)
-  || ThrowUserError("invalid_changedsince", 
-                    { changedsince => $origchangedsince });
-
 my @bugs;
 my @bug_ids; 
 
diff --git a/template/en/default/global/user-error.html.tmpl b/template/en/default/global/user-error.html.tmpl
index 1aaa581b64..f626c640b3 100644
--- a/template/en/default/global/user-error.html.tmpl
+++ b/template/en/default/global/user-error.html.tmpl
@@ -446,7 +446,8 @@
 
   [% ELSIF error == "no_dupe_stats_error_whenever" %]
     [% title = "Error Reading Previous Dupes File" %]
-    An error occurred opening $changedsince days ago ($whenever)'s dupes file:
+    An error occurred opening [% changedsince FILTER html %] days ago
+    ([% whenever FILTER html %])'s dupes file:
     [% error_msg FILTER html %].
      
   [% ELSIF error == "no_dupe_stats_error_yesterday" %]