[tools] testbot/web: Highlight both the 32 and 64-bit checkboxes in case of error.

Francois Gouget fgouget at codeweavers.com
Tue Jul 12 08:22:22 CDT 2022


The comment was wrong since the code explicitly checks that both
ShowRun32 and ShowRun64 are true.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/web/Submit.pl | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/testbot/web/Submit.pl b/testbot/web/Submit.pl
index 4368aeeb30..2e6c3cc7fb 100644
--- a/testbot/web/Submit.pl
+++ b/testbot/web/Submit.pl
@@ -410,8 +410,7 @@ sub Validate($)
         !$self->{Run32} and $self->{ShowRun32} and
         !$self->{Run64} and $self->{ShowRun64})
     {
-      # Don't set ErrField since Run32 and Run64 may be hidden
-      $self->AddError("You must at least pick one of the 32 or 64 bit tests to run on the selected Windows VMs.");
+      $self->AddError("You must at least pick one of the 32 or 64 bit tests to run on the selected Windows VMs.", "Run32", "Run64");
       return undef;
     }
 
-- 
2.30.2



More information about the wine-devel mailing list