Francois Gouget : testbot/web: Use GetErrMessage() instead of direct field access.
Alexandre Julliard
julliard at winehq.org
Tue Jun 28 11:49:22 CDT 2022
Module: tools
Branch: master
Commit: 518ec6e3c0f190ea5a775bda69213bc5eb1460e2
URL: https://source.winehq.org/git/tools.git/?a=commit;h=518ec6e3c0f190ea5a775bda69213bc5eb1460e2
Author: Francois Gouget <fgouget at codeweavers.com>
Date: Mon Jun 27 19:59:56 2022 +0200
testbot/web: Use GetErrMessage() instead of direct field access.
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
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 773fe3bb..ec2eaba7 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 9c18c375..74711df5 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);
}
}
More information about the wine-cvs
mailing list