Fix typos (#10158)

client/src/main/java/com/vaadin/client/LocaleService.java

@@ -25,7 +25,7 @@ import java.util.logging.Logger;
 import com.vaadin.shared.ui.ui.UIState.LocaleData;
 
 /**
- * Date / time etc. localisation service for all widgets. Caches all loaded
+ * Date / time etc. localization service for all widgets. Caches all loaded
  * locales as JSONObjects.
  *
  * @author Vaadin Ltd.

client/src/main/java/com/vaadin/client/ui/VAbstractCalendarPanel.java

@@ -234,7 +234,7 @@ public abstract class VAbstractCalendarPanel<R extends Enum<R>>
      *            one of the days currently visible.
      */
     private void focusDay(Date date) {
-        // Only used when calender body is present
+        // Only used when calendar body is present
         if (acceptDayFocus()) {
             if (focusedDay != null) {
                 focusedDay.removeStyleDependentName(CN_FOCUSED);
@@ -1775,7 +1775,7 @@ public abstract class VAbstractCalendarPanel<R extends Enum<R>>
 
     /**
      * The submit listener is called when the user selects a value from the
-     * calender either by clicking the day or selects it by keyboard.
+     * calendar either by clicking the day or selects it by keyboard.
      *
      * @param submitListener
      *            The listener to trigger

client/src/main/java/com/vaadin/client/ui/VAbstractPopupCalendar.java

@@ -96,7 +96,7 @@ public abstract class VAbstractPopupCalendar<PANEL extends VAbstractCalendarPane
 
     private Label selectedDate;
 
-    private Element descriptionForAssisitveDevicesElement;
+    private Element descriptionForAssistiveDevicesElement;
 
     private final String CALENDAR_TOGGLE_ID = "popupButton";
 
@@ -129,15 +129,15 @@ public abstract class VAbstractPopupCalendar<PANEL extends VAbstractCalendarPane
 
         add(calendarToggle);
 
-        // Description of the usage of the widget for assisitve device users
-        descriptionForAssisitveDevicesElement = DOM.createDiv();
-        descriptionForAssisitveDevicesElement.setInnerText(
+        // Description of the usage of the widget for assistive device users
+        descriptionForAssistiveDevicesElement = DOM.createDiv();
+        descriptionForAssistiveDevicesElement.setInnerText(
                 TextualDateFieldState.DESCRIPTION_FOR_ASSISTIVE_DEVICES);
-        AriaHelper.ensureHasId(descriptionForAssisitveDevicesElement);
+        AriaHelper.ensureHasId(descriptionForAssistiveDevicesElement);
         Roles.getTextboxRole().setAriaDescribedbyProperty(text.getElement(),
-                Id.of(descriptionForAssisitveDevicesElement));
+                Id.of(descriptionForAssistiveDevicesElement));
         AriaHelper.setVisibleForAssistiveDevicesOnly(
-                descriptionForAssisitveDevicesElement, true);
+                descriptionForAssistiveDevicesElement, true);
 
         calendar = calendarPanel;
         calendar.setParentField(this);
@@ -207,13 +207,13 @@ public abstract class VAbstractPopupCalendar<PANEL extends VAbstractCalendarPane
     protected void onAttach() {
         super.onAttach();
         DOM.appendChild(RootPanel.get().getElement(),
-                descriptionForAssisitveDevicesElement);
+                descriptionForAssistiveDevicesElement);
     }
 
     @Override
     protected void onDetach() {
         super.onDetach();
-        descriptionForAssisitveDevicesElement.removeFromParent();
+        descriptionForAssistiveDevicesElement.removeFromParent();
         closeCalendarPanel();
     }
 
@@ -583,7 +583,7 @@ public abstract class VAbstractPopupCalendar<PANEL extends VAbstractCalendarPane
      */
     public void setDescriptionForAssistiveDevices(
             String descriptionForAssistiveDevices) {
-        descriptionForAssisitveDevicesElement
+        descriptionForAssistiveDevicesElement
                 .setInnerText(descriptionForAssistiveDevices);
     }
 
@@ -594,7 +594,7 @@ public abstract class VAbstractPopupCalendar<PANEL extends VAbstractCalendarPane
      * @return String with the description
      */
     public String getDescriptionForAssistiveDevices() {
-        return descriptionForAssisitveDevicesElement.getInnerText();
+        return descriptionForAssistiveDevicesElement.getInnerText();
     }
 
     /**

client/src/main/java/com/vaadin/client/ui/VDateTimeCalendarPanel.java

@@ -46,7 +46,7 @@ public class VDateTimeCalendarPanel
     private VTime time;
 
     /**
-     * TimeSelector is a widget consisting of list boxes that modifie the Date
+     * TimeSelector is a widget consisting of list boxes that modify the Date
      * object that is given for.
      *
      */

client/src/main/java/com/vaadin/client/widgets/Escalator.java

@@ -283,7 +283,7 @@ public class Escalator extends Widget
     // todo comments legend
     /*
      * [[optimize]]: There's an opportunity to rewrite the code in such a way
-     * that it _might_ perform better (rememeber to measure, implement,
+     * that it _might_ perform better (remember to measure, implement,
      * re-measure)
      */
     /*

client/src/main/java/com/vaadin/client/widgets/Grid.java

@@ -6976,7 +6976,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>,
     }
 
     /**
-     * Gets the {@link Escalator} used by this Grid instnace.
+     * Gets the {@link Escalator} used by this Grid instance.
      *
      * @return the escalator instance, never <code>null</code>
      */

compatibility-client/src/main/java/com/vaadin/v7/client/ui/VCalendarPanel.java

@@ -238,7 +238,7 @@ public class VCalendarPanel extends FocusableFlexTable implements
      *            one of the days currently visible.
      */
     private void focusDay(Date date) {
-        // Only used when calender body is present
+        // Only used when calendar body is present
         if (resolution.getCalendarField() > Resolution.MONTH
                 .getCalendarField()) {
             if (focusedDay != null) {
@@ -1632,7 +1632,7 @@ public class VCalendarPanel extends FocusableFlexTable implements
     }
 
     /**
-     * TimeSelector is a widget consisting of list boxes that modifie the Date
+     * TimeSelector is a widget consisting of list boxes that modify the Date
      * object that is given for.
      *
      */
@@ -1955,7 +1955,7 @@ public class VCalendarPanel extends FocusableFlexTable implements
 
     /**
      * The submit listener is called when the user selects a value from the
-     * calender either by clicking the day or selects it by keyboard.
+     * calendar either by clicking the day or selects it by keyboard.
      *
      * @param submitListener
      *            The listener to trigger

compatibility-client/src/main/java/com/vaadin/v7/client/ui/VPopupCalendar.java

@@ -98,7 +98,7 @@ public class VPopupCalendar extends VTextualDate
 
     private Label selectedDate;
 
-    private Element descriptionForAssisitveDevicesElement;
+    private Element descriptionForAssistiveDevicesElement;
 
     public VPopupCalendar() {
         super();
@@ -129,15 +129,15 @@ public class VPopupCalendar extends VTextualDate
 
         add(calendarToggle);
 
-        // Description of the usage of the widget for assisitve device users
-        descriptionForAssisitveDevicesElement = DOM.createDiv();
-        descriptionForAssisitveDevicesElement.setInnerText(
+        // Description of the usage of the widget for assistive device users
+        descriptionForAssistiveDevicesElement = DOM.createDiv();
+        descriptionForAssistiveDevicesElement.setInnerText(
                 PopupDateFieldState.DESCRIPTION_FOR_ASSISTIVE_DEVICES);
-        AriaHelper.ensureHasId(descriptionForAssisitveDevicesElement);
+        AriaHelper.ensureHasId(descriptionForAssistiveDevicesElement);
         Roles.getTextboxRole().setAriaDescribedbyProperty(text.getElement(),
-                Id.of(descriptionForAssisitveDevicesElement));
+                Id.of(descriptionForAssistiveDevicesElement));
         AriaHelper.setVisibleForAssistiveDevicesOnly(
-                descriptionForAssisitveDevicesElement, true);
+                descriptionForAssistiveDevicesElement, true);
 
         calendar = GWT.create(VCalendarPanel.class);
         calendar.setParentField(this);
@@ -207,13 +207,13 @@ public class VPopupCalendar extends VTextualDate
     protected void onAttach() {
         super.onAttach();
         DOM.appendChild(RootPanel.get().getElement(),
-                descriptionForAssisitveDevicesElement);
+                descriptionForAssistiveDevicesElement);
     }
 
     @Override
     protected void onDetach() {
         super.onDetach();
-        descriptionForAssisitveDevicesElement.removeFromParent();
+        descriptionForAssistiveDevicesElement.removeFromParent();
         closeCalendarPanel();
     }
 
@@ -571,7 +571,7 @@ public class VPopupCalendar extends VTextualDate
      */
     public void setDescriptionForAssistiveDevices(
             String descriptionForAssistiveDevices) {
-        descriptionForAssisitveDevicesElement
+        descriptionForAssistiveDevicesElement
                 .setInnerText(descriptionForAssistiveDevices);
     }
 
@@ -582,7 +582,7 @@ public class VPopupCalendar extends VTextualDate
      * @return String with the description
      */
     public String getDescriptionForAssistiveDevices() {
-        return descriptionForAssisitveDevicesElement.getInnerText();
+        return descriptionForAssistiveDevicesElement.getInnerText();
     }
 
     /**

compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Escalator.java

@@ -284,7 +284,7 @@ public class Escalator extends Widget
     // todo comments legend
     /*
      * [[optimize]]: There's an opportunity to rewrite the code in such a way
-     * that it _might_ perform better (rememeber to measure, implement,
+     * that it _might_ perform better (remember to measure, implement,
      * re-measure)
      */
     /*

compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Grid.java

@@ -6882,7 +6882,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>,
     }
 
     /**
-     * Gets the {@link Escalator} used by this Grid instnace.
+     * Gets the {@link Escalator} used by this Grid instance.
      *
      * @return the escalator instance, never <code>null</code>
      */

compatibility-server/src/main/java/com/vaadin/v7/data/fieldgroup/BeanFieldGroup.java

@@ -30,7 +30,8 @@ import com.vaadin.v7.ui.Field;
 
 /**
  *
- * @deprecated As of 8.0, replaced by {@link Binder} and {@link BeanValidationBinder}
+ * @deprecated As of 8.0, replaced by {@link Binder} and
+ *             {@link BeanValidationBinder}
  */
 @Deprecated
 public class BeanFieldGroup<T> extends FieldGroup {

compatibility-server/src/main/java/com/vaadin/v7/data/util/sqlcontainer/query/generator/StatementHelper.java

@@ -64,7 +64,7 @@ public class StatementHelper implements Serializable {
             dataTypes.put(parameters.size() - 1, parameter.getClass());
         } else {
             throw new IllegalArgumentException(
-                    "You cannot add null parameters using addParamaters(Object). "
+                    "You cannot add null parameters using addParameters(Object). "
                             + "Use addParameters(Object,Class) instead");
         }
     }

compatibility-server/src/main/java/com/vaadin/v7/ui/Calendar.java

@@ -1667,7 +1667,7 @@ public class Calendar extends AbstractLegacyComponent
     }
 
     /**
-     * Adds an action handler to the calender that handles event produced by the
+     * Adds an action handler to the calendar that handles event produced by the
      * context menu.
      *
      * <p>

compatibility-server/src/test/java/com/vaadin/v7/tests/server/component/calendar/ContainerDataSourceTest.java

@@ -23,7 +23,6 @@ import java.util.List;
 import java.util.Locale;
 import java.util.TimeZone;
 
-import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
 

server/src/main/java/com/vaadin/data/BinderValidationStatus.java

@@ -137,10 +137,8 @@ public class BinderValidationStatus<BEAN> implements Serializable {
      * @return a list of all validation errors
      */
     public List<ValidationResult> getValidationErrors() {
-        List<ValidationResult> errors =
-                getFieldValidationErrors().stream()
-                        .map(s -> s.getResult().get())
-                        .collect(Collectors.toList());
+        List<ValidationResult> errors = getFieldValidationErrors().stream()
+                .map(s -> s.getResult().get()).collect(Collectors.toList());
         errors.addAll(getBeanValidationErrors());
         return errors;
     }

server/src/main/java/com/vaadin/event/EventRouter.java

@@ -164,7 +164,7 @@ public class EventRouter implements MethodEventSource {
     }
 
     /*
-     * Removes the event listener methods matching the given given paramaters.
+     * Removes the event listener methods matching the given given parameters.
      * Don't add a JavaDoc comment here, we use the default documentation from
      * implemented interface.
      */

server/src/main/java/com/vaadin/server/VariableOwner.java

@@ -33,8 +33,8 @@ import com.vaadin.ui.LegacyComponent;
  * @author Vaadin Ltd.
  * @since 3.0
  * @deprecated As of 7.0. Only provided to ease porting of Vaadin 6 components.
- *             Do not implement this directly, implement {@link LegacyComponent}
- *             .
+ *             Do not implement this directly, implement
+ *             {@link LegacyComponent}.
  */
 @Deprecated
 public interface VariableOwner extends Serializable {

server/src/main/java/com/vaadin/ui/Button.java

@@ -519,7 +519,7 @@ public class Button extends AbstractFocusable
      * Sets the component's icon and alt text.
      * <p>
      * An alt text is shown when an image could not be loaded, and read by
-     * assisitve devices.
+     * assistive devices.
      *
      * @param icon
      *            the icon to be shown with the component's caption.

shared/src/main/java/com/vaadin/shared/annotations/NoLayout.java

@@ -21,7 +21,7 @@ import java.lang.annotation.Target;
 
 /**
  * Annotation used to mark client RPC methods, state fields, or state setter
- * methods that should not trigger an layout phase after changes have been
+ * methods that should not trigger a layout phase after changes have been
  * processed. Whenever there's at least one change that is not marked with this
  * annotation, the framework will assume some sizes might have changed an will
  * therefore start a layout phase after applying the changes.

testbench-api/src/main/java/com/vaadin/testbench/elements/AbstractComponentElement.java

@@ -53,7 +53,7 @@ public class AbstractComponentElement extends AbstractElement {
     public boolean isReadOnly() {
         final String READONLY_CSS_CLASS = "v-readonly";
         String readonlyClass = getAttribute("class");
-        // lookin for READONLY_CSS_CLASS string
+        // looking for READONLY_CSS_CLASS string
         String[] cssSelectors = readonlyClass.split("\\s");
         for (String selector : cssSelectors) {
             if (selector.equals(READONLY_CSS_CLASS)) {

uitest/src/main/java/com/vaadin/tests/LayoutDemo.java

@@ -138,7 +138,7 @@ public class LayoutDemo extends com.vaadin.server.LegacyApplication {
         layout.addComponent(new Label(
                 "Panel is a container for other components, by default it draws a frame around it's "
                         + "extremities and may have a caption to clarify the nature of the contained components' purpose."
-                        + " Panel contains an layout where the actual contained components are added, "
+                        + " Panel contains a layout where the actual contained components are added, "
                         + "this layout may be switched on the fly.",
                 ContentMode.HTML));
         panel.setWidth("222px");

uitest/src/main/java/com/vaadin/tests/RandomLayoutStress.java

@@ -178,21 +178,24 @@ public class RandomLayoutStress extends com.vaadin.server.LegacyApplication {
             panelLayout.addComponent(new Label(
                     "Panel is a container for other components, by default it draws a frame around it's "
                             + "extremities and may have a caption to clarify the nature of the contained components' purpose."
-                            + " Panel contains an layout where the actual contained components are added, "
+                            + " Panel contains a layout where the actual contained components are added, "
                             + "this layout may be switched on the fly."));
             ((Panel) result).setWidth("250px");
             break;
         case 6:
             // Datefield
             result = new TestDateField();
-            ((AbstractDateField<LocalDate, DateResolution>) result).setStyleName("calendar");
-            ((AbstractDateField<LocalDate, DateResolution>) result).setValue(LocalDate.now());
+            ((AbstractDateField<LocalDate, DateResolution>) result)
+                    .setStyleName("calendar");
+            ((AbstractDateField<LocalDate, DateResolution>) result)
+                    .setValue(LocalDate.now());
             result.setCaption("Calendar component " + caption);
             break;
         case 7:
             // Datefield
             result = new TestDateField();
-            ((AbstractDateField<LocalDate, DateResolution>) result).setValue(LocalDate.now());
+            ((AbstractDateField<LocalDate, DateResolution>) result)
+                    .setValue(LocalDate.now());
             result.setCaption("Calendar component " + caption);
             break;
         }

uitest/src/main/java/com/vaadin/tests/widgetset/client/grid/PureGWTTestApplication.java

@@ -29,7 +29,7 @@ import com.vaadin.client.ui.SubPartAware;
 
 /**
  * Pure GWT Test Application base for testing features of a single widget;
- * provides a menu system and convenience method for adding items to it.
+ * provides a menu system and a convenience method for adding items to it.
  *
  * @since
  * @author Vaadin Ltd