[tools] testbot/cgi: Use AddError() instead of direct field access in ItemPage.

Francois Gouget fgouget at codeweavers.com
Wed Jul 6 09:06:08 CDT 2022


Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/lib/ObjectModel/CGI/ItemPage.pm | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/testbot/lib/ObjectModel/CGI/ItemPage.pm b/testbot/lib/ObjectModel/CGI/ItemPage.pm
index cd40ba2e4b..3ec201d99c 100644
--- a/testbot/lib/ObjectModel/CGI/ItemPage.pm
+++ b/testbot/lib/ObjectModel/CGI/ItemPage.pm
@@ -176,8 +176,13 @@ sub Save($)
 
   return !1 if (!$self->SUPER::Save());
 
-  (my $_ErrKey, $self->{ErrField}, $self->{ErrMessage}) = $self->{Collection}->Save();
-  return !defined $self->{ErrMessage};
+  my ($_ErrKey, $ErrProperty, $ErrMessage) = $self->{Collection}->Save();
+  if (defined $ErrMessage)
+  {
+    $self->AddError($ErrMessage, $ErrProperty);
+    return undef;
+  }
+  return 1;
 }
 
 1;
-- 
2.30.2




More information about the wine-devel mailing list