[tools] testbot/web: Use AddError() to set the item list action error.

Francois Gouget fgouget at codeweavers.com
Wed Jul 6 09:05:39 CDT 2022


Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/web/FailuresList.pl    | 4 ++--
 testbot/web/admin/UsersList.pl | 4 ++--
 testbot/web/admin/VMsList.pl   | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/testbot/web/FailuresList.pl b/testbot/web/FailuresList.pl
index 2cb40c482e..d1f8de6d8e 100644
--- a/testbot/web/FailuresList.pl
+++ b/testbot/web/FailuresList.pl
@@ -138,8 +138,8 @@ sub OnItemAction($$$)
       my ($_ErrProperty, $ErrMessage) = $Failure->Save();
       if (defined $ErrMessage)
       {
-        # Setting ErrField is only useful on form pages
-        $self->{EnclosingPage}->SetError(undef, $ErrMessage);
+        # Setting the error field is only useful on form pages
+        $self->{EnclosingPage}->AddError($ErrMessage);
         return 0;
       }
       return 1;
diff --git a/testbot/web/admin/UsersList.pl b/testbot/web/admin/UsersList.pl
index 5f9f47b46c..6fdb06d0cf 100644
--- a/testbot/web/admin/UsersList.pl
+++ b/testbot/web/admin/UsersList.pl
@@ -109,8 +109,8 @@ sub OnItemAction($$$)
     my ($_ErrProperty, $ErrMessage) = $User->Save();
     if (defined $ErrMessage)
     {
-      # Setting ErrField is only useful on form pages
-      $self->{EnclosingPage}->SetError(undef, $ErrMessage);
+      # Setting the error field is only useful on form pages
+      $self->{EnclosingPage}->AddError($ErrMessage);
       return 0;
     }
 
diff --git a/testbot/web/admin/VMsList.pl b/testbot/web/admin/VMsList.pl
index 6c3ff2036f..3361c03fa3 100644
--- a/testbot/web/admin/VMsList.pl
+++ b/testbot/web/admin/VMsList.pl
@@ -52,8 +52,8 @@ sub OnItemAction($$$)
     my ($_ErrProperty, $ErrMessage) = $VM->Save();
     if (defined $ErrMessage)
     {
-      # Setting ErrField is only useful on form pages
-      $self->{EnclosingPage}->SetError(undef, $ErrMessage);
+      # Setting the error field is only useful on form pages
+      $self->{EnclosingPage}->AddError($ErrMessage);
       return 0;
     }
     return 1;
-- 
2.30.2




More information about the wine-devel mailing list