From a9c83ba07e4dec90eb7d6e332b91adde9613ebb4 Mon Sep 17 00:00:00 2001 From: Ahmed Ashour Date: Fri, 20 Oct 2017 10:10:02 +0200 Subject: Use .toLowerCase and .toUpperCase(Locale.ROOT) (#10182) * Use .toLowerCase and .toUpperCase(Locale.ROOT) * Spaces * add spaces after if( --- .../com/vaadin/ui/components/grid/GridDropTarget.java | 3 ++- .../vaadin/ui/components/grid/TreeGridDropTarget.java | 19 ++++++++++--------- 2 files changed, 12 insertions(+), 10 deletions(-) (limited to 'server/src/main/java/com/vaadin/ui/components') diff --git a/server/src/main/java/com/vaadin/ui/components/grid/GridDropTarget.java b/server/src/main/java/com/vaadin/ui/components/grid/GridDropTarget.java index 342946d5c9..edb1534a05 100644 --- a/server/src/main/java/com/vaadin/ui/components/grid/GridDropTarget.java +++ b/server/src/main/java/com/vaadin/ui/components/grid/GridDropTarget.java @@ -16,6 +16,7 @@ package com.vaadin.ui.components.grid; import java.util.LinkedHashMap; +import java.util.Locale; import java.util.Map; import com.vaadin.shared.Registration; @@ -156,7 +157,7 @@ public class GridDropTarget extends DropTargetExtension> { GridDropEvent event = new GridDropEvent<>(getParent(), dataPreserveOrder, - DropEffect.valueOf(dropEffect.toUpperCase()), + DropEffect.valueOf(dropEffect.toUpperCase(Locale.ROOT)), getUI().getActiveDragSource(), dropTargetRow, dropLocation, mouseEventDetails); diff --git a/server/src/main/java/com/vaadin/ui/components/grid/TreeGridDropTarget.java b/server/src/main/java/com/vaadin/ui/components/grid/TreeGridDropTarget.java index 1d440b5fec..e471f350bc 100644 --- a/server/src/main/java/com/vaadin/ui/components/grid/TreeGridDropTarget.java +++ b/server/src/main/java/com/vaadin/ui/components/grid/TreeGridDropTarget.java @@ -16,6 +16,7 @@ package com.vaadin.ui.components.grid; import java.util.LinkedHashMap; +import java.util.Locale; import java.util.Map; import com.vaadin.shared.Registration; @@ -30,7 +31,7 @@ import com.vaadin.ui.TreeGrid; * counterpart of GridDropTargetExtensionConnector. * * @param - * Type of the TreeGrid bean. + * Type of the TreeGrid bean. * @author Vaadin Ltd * @since 8.1 */ @@ -41,22 +42,22 @@ public class TreeGridDropTarget extends GridDropTarget { * drop. * * @param target - * TreeGrid to be extended. + * TreeGrid to be extended. * @param dropMode - * Drop mode that describes the allowed drop locations within the - * TreeGrid's row. + * Drop mode that describes the allowed drop locations within the + * TreeGrid's row. */ public TreeGridDropTarget(TreeGrid target, DropMode dropMode) { super(target, dropMode); } /** - * Attaches drop listener for the current drop target. {@link - * TreeGridDropListener#drop(TreeGridDropEvent)} is called when drop event - * happens on the client side. + * Attaches drop listener for the current drop target. + * {@link TreeGridDropListener#drop(TreeGridDropEvent)} is called when drop + * event happens on the client side. * * @param listener - * Listener to handle drop event. + * Listener to handle drop event. * @return Handle to be used to remove this listener. */ public Registration addTreeGridDropListener( @@ -79,7 +80,7 @@ public class TreeGridDropTarget extends GridDropTarget { TreeGridDropEvent event = new TreeGridDropEvent<>(getParent(), dataPreserveOrder, - DropEffect.valueOf(dropEffect.toUpperCase()), + DropEffect.valueOf(dropEffect.toUpperCase(Locale.ROOT)), getUI().getActiveDragSource(), dropTargetRow, dropLocation, mouseEventDetails, depth, collapsed); -- cgit v1.2.3