Francois Gouget : testbot/web: Better document the default Hours log page value.

Alexandre Julliard julliard at winehq.org
Tue Mar 22 16:37:57 CDT 2022


Module: tools
Branch: master
Commit: 22e64592a8af717408d240329a0d5123580db029
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=22e64592a8af717408d240329a0d5123580db029

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Mar 22 13:20:01 2022 +0100

testbot/web: Better document the default Hours log 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/admin/Log.pl | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/testbot/web/admin/Log.pl b/testbot/web/admin/Log.pl
index ddc655f..8d3a8ce 100644
--- a/testbot/web/admin/Log.pl
+++ b/testbot/web/admin/Log.pl
@@ -28,6 +28,8 @@ use ObjectModel::BasicPropertyDescriptor;
 use WineTestBot::Config;
 use WineTestBot::Log;
 
+my $HOURS_DEFAULT = 1;
+
 
 sub _initialize($$$)
 {
@@ -41,7 +43,7 @@ sub _initialize($$$)
   if (!$self->GetParam("Hours") or !$self->Validate() or
       !int($self->GetParam("Hours"))) # 00 case!
   {
-    $self->SetParam("Hours", 1) if (!defined $self->{ErrMessage});
+    $self->SetParam("Hours", $HOURS_DEFAULT) if (!defined $self->{ErrMessage});
     $self->SetParam("Action", undef);
   }
 }




More information about the wine-cvs mailing list