From 13961956f6f7deebf3bc4e285cec9ab162ef8d90 Mon Sep 17 00:00:00 2001
From: Aleksi Hietanen <aleksi@vaadin.com>
Date: Tue, 29 Aug 2017 12:01:47 +0300
Subject: [PATCH] Minor javadoc improvements (#9885)

---
 .../src/main/java/com/vaadin/data/Binder.java | 20 ++++++++++---------
 server/src/main/java/com/vaadin/ui/Grid.java  |  2 --
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/server/src/main/java/com/vaadin/data/Binder.java b/server/src/main/java/com/vaadin/data/Binder.java
index 43f893e33f..7632f2b09b 100644
--- a/server/src/main/java/com/vaadin/data/Binder.java
+++ b/server/src/main/java/com/vaadin/data/Binder.java
@@ -487,14 +487,13 @@ public class Binder<BEAN> implements Serializable {
         /**
          * Sets the field to be required. This means two things:
          * <ol>
-         * <li>the required indicator is visible</li>
-         * <li>the field value is validated for not being empty*</li>
+         * <li>the field value is validated for not being empty</li>
+         * <li>the required indicator will be displayed if the value of this
+         * field at validation time is equal to what
+         * {@link HasValue#getEmptyValue()} returns</li>
          * </ol>
          * For localizing the error message, use
          * {@link #asRequired(ErrorMessageProvider)}.
-         * <p>
-         * *Value not being the equal to what {@link HasValue#getEmptyValue()}
-         * returns.
          *
          * @see #asRequired(ErrorMessageProvider)
          * @see HasValue#setRequiredIndicatorVisible(boolean)
@@ -511,11 +510,11 @@ public class Binder<BEAN> implements Serializable {
         /**
          * Sets the field to be required. This means two things:
          * <ol>
-         * <li>the required indicator is visible</li>
-         * <li>the field value is validated for not being empty*</li>
+         * <li>the field value is validated for not being empty</li>
+         * <li>the required indicator will be displayed if the value of this
+         * field at validation time is equal to what
+         * {@link HasValue#getEmptyValue()} returns</li>
          * </ol>
-         * *Value not being the equal to what {@link HasValue#getEmptyValue()}
-         * returns.
          *
          * @see HasValue#setRequiredIndicatorVisible(boolean)
          * @see HasValue#isEmpty()
@@ -1609,6 +1608,9 @@ public class Binder<BEAN> implements Serializable {
      * level validators if a bean is currently set with
      * {@link #setBean(Object)}, and returns whether any of the validators
      * failed.
+     * <p>
+     * <b>Note:</b> Calling this method will not trigger status change events,
+     * unlike {@link #validate()}.
      *
      * @return whether this binder is in a valid state
      * @throws IllegalStateException
diff --git a/server/src/main/java/com/vaadin/ui/Grid.java b/server/src/main/java/com/vaadin/ui/Grid.java
index 663df20b88..ceeec79ac1 100644
--- a/server/src/main/java/com/vaadin/ui/Grid.java
+++ b/server/src/main/java/com/vaadin/ui/Grid.java
@@ -4370,8 +4370,6 @@ public class Grid<T> extends AbstractListing<T> implements HasComponents,
      * @param providers
      *            list of {@link DeclarativeValueProvider}s to store the data of
      *            each column to
-     *
-     * @since 8.1
      */
     protected void readData(Element body,
             List<DeclarativeValueProvider<T>> providers) {
-- 
2.39.5