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