Francois Gouget : testbot/web: Use AddError() in the failure details page.
Alexandre Julliard
julliard at winehq.org
Wed Jul 6 12:02:25 CDT 2022
Module: tools
Branch: master
Commit: f130b873281ec2b96318e45b26cbc9f53d36e075
URL: https://source.winehq.org/git/tools.git/?a=commit;h=f130b873281ec2b96318e45b26cbc9f53d36e075
Author: Francois Gouget <fgouget at codeweavers.com>
Date: Wed Jul 6 16:05:48 2022 +0200
testbot/web: Use AddError() in the failure details page.
This is better than direct field access.
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
testbot/web/FailureDetails.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/testbot/web/FailureDetails.pl b/testbot/web/FailureDetails.pl
index 02803ed2..b4226f60 100644
--- a/testbot/web/FailureDetails.pl
+++ b/testbot/web/FailureDetails.pl
@@ -203,7 +203,7 @@ sub OnAction($$)
{
if (!$self->{RW})
{
- $self->{ErrMessage} = "Only admin users can create or modify failures";
+ $self->AddError("Only admin users can create or modify failures");
return 0;
}
if (($self->{Item}->BugId || "") ne $self->GetParam("BugId"))
More information about the wine-cvs
mailing list