From ab3938f0af7d44010231aba2adc736ed4ef3e329 Mon Sep 17 00:00:00 2001
From: Henri Sara <henri.sara@itmill.com>
Date: Thu, 28 Jan 2010 07:25:38 +0000
Subject: [PATCH] #4103 hide error indicator for an empty required
 Select/ComboBox to be consistent with other fields

svn changeset:11022/svn branch:6.2
---
 src/com/vaadin/ui/Select.java | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/com/vaadin/ui/Select.java b/src/com/vaadin/ui/Select.java
index e8e0dfebaa..4174001d87 100644
--- a/src/com/vaadin/ui/Select.java
+++ b/src/com/vaadin/ui/Select.java
@@ -234,6 +234,12 @@ public class Select extends AbstractSelect implements AbstractSelect.Filtering,
         currentPage = -1; // current page is always set by client
 
         optionRequest = true;
+
+        // Hide the error indicator if needed
+        if (isRequired() && isEmpty() && getComponentError() == null
+                && getErrorMessage() != null) {
+            target.addAttribute("hideErrors", true);
+        }
     }
 
     /**
-- 
2.39.5