Javadocs for new 7.6 API (#19135) and some fixes

Change-Id: Ic93545c9c3ed48d6328ee29af744cbfbff0b5c0b

3 files changed, 19 insertions, 12 deletions

diff --git a/server/src/com/vaadin/ui/Grid.java b/server/src/com/vaadin/ui/Grid.java
index d7efdf5814..65d7d8e2d2 100644
--- a/server/src/com/vaadin/ui/Grid.java
+++ b/server/src/com/vaadin/ui/Grid.java
@@ -2050,7 +2050,7 @@ public class Grid extends AbstractFocusable implements SelectionNotifier,
      * the cells in the row for which a {@link CellDescriptionGenerator cell
      * description} is not generated.
      * 
-     * @see Grid#setRowDescriptionGenerator(CellDescriptionGenerator)
+     * @see Grid#setRowDescriptionGenerator
      * 
      * @since 7.6
      */
@@ -5828,8 +5828,7 @@ public class Grid extends AbstractFocusable implements SelectionNotifier,
 
     /**
      * Gets the
-     * {@link com.vaadin.data.RpcDataProviderExtension.DataProviderKeyMapper
-     * DataProviderKeyMapper} being used by the data source.
+     * {@link KeyMapper } being used by the data source.
      * 
      * @return the key mapper being used by the data source
      */
diff --git a/server/src/com/vaadin/ui/Table.java b/server/src/com/vaadin/ui/Table.java
index 66fa257ac2..a6b40ff06b 100644
--- a/server/src/com/vaadin/ui/Table.java
+++ b/server/src/com/vaadin/ui/Table.java
@@ -1033,7 +1033,7 @@ public class Table extends AbstractSelect implements Action.Container,
      * </p>
      * 
      * <p>
-     * If Table has height set ({@link #setHeight(float, int)} ) the client side
+     * If Table has height set ({@link #setHeight(float, Unit)} ) the client side
      * may update the page length automatically the correct value.
      * </p>
      * 
@@ -2527,8 +2527,8 @@ public class Table extends AbstractSelect implements Action.Container,
      * 
      * @see #getPropertyValue(Object, Object, Property)
      * 
-     * @param oldVisibleComponents
-     *            a set of components that should be unregistered.
+     * @param component
+     *            component that should be unregistered.
      */
     protected void unregisterComponent(Component component) {
         getLogger().log(
@@ -2598,7 +2598,7 @@ public class Table extends AbstractSelect implements Action.Container,
      * Gets the row header mode.
      * 
      * @return the Row header mode.
-     * @see #setRowHeaderMode(int)
+     * @see #setRowHeaderMode
      */
     public RowHeaderMode getRowHeaderMode() {
         return rowHeaderMode;
@@ -2830,10 +2830,10 @@ public class Table extends AbstractSelect implements Action.Container,
     /**
      * Gets items ids from a range of key values
      * 
-     * @param startRowKey
+     * @param itemId
      *            The start key
-     * @param endRowKey
-     *            The end key
+     * @param length
+     *            amount of items to be retrieved
      * @return
      */
     private LinkedHashSet<Object> getItemIdsInRange(Object itemId,
@@ -6399,8 +6399,11 @@ public class Table extends AbstractSelect implements Action.Container,
         return result;
     }
 
-    /* Context Click handling */
-
+    /**
+     * ContextClickEvent for the Table Component.
+     *
+     * @since 7.6
+     */
     public static class TableContextClickEvent extends ContextClickEvent {
 
         private final Object itemId;
diff --git a/server/src/com/vaadin/ui/Tree.java b/server/src/com/vaadin/ui/Tree.java
index b0f41b382e..de8e74bb03 100644
--- a/server/src/com/vaadin/ui/Tree.java
+++ b/server/src/com/vaadin/ui/Tree.java
@@ -77,6 +77,11 @@ import com.vaadin.util.ReflectTools;
 public class Tree extends AbstractSelect implements Container.Hierarchical,
         Action.Container, ItemClickNotifier, DragSource, DropTarget {
 
+    /**
+     * ContextClickEvent for the Tree Component.
+     *
+     * @since 7.6
+     */
     public static class TreeContextClickEvent extends ContextClickEvent {
 
         private final Object itemId;