]> source.dussan.org Git - vaadin-framework.git/commitdiff
Clarify set/addSortComparator parameter name (#8562)
authorLeif Åstrand <legioth@gmail.com>
Tue, 14 Feb 2017 13:00:06 +0000 (15:00 +0200)
committerHenri Sara <henri.sara@gmail.com>
Thu, 16 Feb 2017 08:12:49 +0000 (10:12 +0200)
server/src/main/java/com/vaadin/data/provider/ListDataProvider.java

index 039366418a81ee6e8b9eee2d9413e47f511c3b11..2f1e17d67326a2d3b4f584fe34bbd27f78fe08d8 100644 (file)
@@ -74,7 +74,7 @@ public class ListDataProvider<T>
 
     /**
      * Returns the underlying data items.
-     * 
+     *
      * @return the underlying data items
      */
     public Collection<T> getItems() {
@@ -134,12 +134,12 @@ public class ListDataProvider<T>
      * @see #setSortOrder(ValueProvider, SortDirection)
      * @see #addSortComparator(SerializableComparator)
      *
-     * @param sortOrder
+     * @param comparator
      *            a comparator to use, or <code>null</code> to clear any
      *            previously set sort order
      */
-    public void setSortComparator(SerializableComparator<T> sortOrder) {
-        this.sortOrder = sortOrder;
+    public void setSortComparator(SerializableComparator<T> comparator) {
+        this.sortOrder = comparator;
         refreshAll();
     }
 
@@ -180,20 +180,20 @@ public class ListDataProvider<T>
      * @see #setSortComparator(SerializableComparator)
      * @see #addSortOrder(ValueProvider, SortDirection)
      *
-     * @param sortOrder
+     * @param comparator
      *            a comparator to add, not <code>null</code>
      */
-    public void addSortComparator(SerializableComparator<T> sortOrder) {
-        Objects.requireNonNull(sortOrder, "Sort order to add cannot be null");
+    public void addSortComparator(SerializableComparator<T> comparator) {
+        Objects.requireNonNull(comparator, "Sort order to add cannot be null");
 
         SerializableComparator<T> originalComparator = this.sortOrder;
         if (originalComparator == null) {
-            setSortComparator(sortOrder);
+            setSortComparator(comparator);
         } else {
             setSortComparator((a, b) -> {
                 int result = originalComparator.compare(a, b);
                 if (result == 0) {
-                    result = sortOrder.compare(a, b);
+                    result = comparator.compare(a, b);
                 }
                 return result;
             });