Francois Gouget : testbot/web: Better document the default Days stats page value.
Alexandre Julliard
julliard at winehq.org
Tue Mar 22 16:37:57 CDT 2022
Module: tools
Branch: master
Commit: 259b97e6f58f4ddb23912765d3ee7ceb86e53454
URL: https://source.winehq.org/git/tools.git/?a=commit;h=259b97e6f58f4ddb23912765d3ee7ceb86e53454
Author: Francois Gouget <fgouget at codeweavers.com>
Date: Tue Mar 22 13:20:59 2022 +0100
testbot/web: Better document the default Days stats page value.
Store it in a variable at the start of the file so it's easy to spot.
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
testbot/web/Stats.pl | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/testbot/web/Stats.pl b/testbot/web/Stats.pl
index 5ec6979..8433c6d 100644
--- a/testbot/web/Stats.pl
+++ b/testbot/web/Stats.pl
@@ -32,6 +32,8 @@ use WineTestBot::Log; # For Elapsed()
use WineTestBot::Utils;
use WineTestBot::VMs;
+my $DAYS_DEFAULT = 7;
+
sub _initialize($$$)
{
@@ -46,7 +48,7 @@ sub _initialize($$$)
if (!$self->GetParam("Days") or !$self->Validate() or
!int($self->GetParam("Days"))) # 00 case!
{
- $self->SetParam("Days", 7) if (!defined $self->{ErrMessage});
+ $self->SetParam("Days", $DAYS_DEFAULT) if (!defined $self->{ErrMessage});
$self->{NoStats} = 1;
}
# The action is a no-op so unset it to not need to redefine OnAction()
More information about the wine-cvs
mailing list