[tools] testbot/web: Use GetErrMessage() instead of direct field access.

Francois Gouget fgouget at codeweavers.com
Mon Jun 27 12:59:56 CDT 2022


Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/web/Stats.pl     | 2 +-
 testbot/web/admin/Log.pl | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/testbot/web/Stats.pl b/testbot/web/Stats.pl
index 773fe3bb99..ec2eaba7e2 100644
--- a/testbot/web/Stats.pl
+++ b/testbot/web/Stats.pl
@@ -49,7 +49,7 @@ sub _initialize($$$)
   if (!$self->GetParam("Days") or !$self->Validate() or
       !int($self->GetParam("Days"))) # 00 case!
   {
-    $self->SetParam("Days", $DAYS_DEFAULT) if (!defined $self->{ErrMessage});
+    $self->SetParam("Days", $DAYS_DEFAULT) if (!defined $self->GetErrMessage());
     $self->{NoStats} = 1;
   }
   # The action is a no-op so unset it to not need to redefine OnAction()
diff --git a/testbot/web/admin/Log.pl b/testbot/web/admin/Log.pl
index 9c18c375e2..74711df5f3 100644
--- a/testbot/web/admin/Log.pl
+++ b/testbot/web/admin/Log.pl
@@ -43,7 +43,7 @@ sub _initialize($$$)
   if (!$self->GetParam("Hours") or !$self->Validate() or
       !int($self->GetParam("Hours"))) # 00 case!
   {
-    $self->SetParam("Hours", $HOURS_DEFAULT) if (!defined $self->{ErrMessage});
+    $self->SetParam("Hours", $HOURS_DEFAULT) if (!defined $self->GetErrMessage());
     $self->SetParam("Action", undef);
   }
 }
-- 
2.30.2



More information about the wine-devel mailing list