Clean DropEvent and DragEndEvent properties (#8925)

* Add dropEffect parameter to DragEndEvent (#8895)

* Remove drop effect from drop event (#8895)

* Make sure that drop effect is not null

1 files changed, 1 insertions, 15 deletions

diff --git a/server/src/main/java/com/vaadin/event/dnd/DropEvent.java b/server/src/main/java/com/vaadin/event/dnd/DropEvent.java
index c2b4943d0f..932f5577a5 100644
--- a/server/src/main/java/com/vaadin/event/dnd/DropEvent.java
+++ b/server/src/main/java/com/vaadin/event/dnd/DropEvent.java
@@ -32,7 +32,6 @@ import com.vaadin.ui.Component;
  */
 public class DropEvent<T extends AbstractComponent> extends Component.Event {
     private final String dataTransferText;
-    private final DropEffect dropEffect;
     private final DragSourceExtension<? extends AbstractComponent> dragSourceExtension;
     private final AbstractComponent dragSource;
 
@@ -44,20 +43,16 @@ public class DropEvent<T extends AbstractComponent> extends Component.Event {
      * @param dataTransferText
      *         Data of type {@code "text"} from the {@code DataTransfer}
      *         object.
-     * @param dropEffect
-     *         Drop effect from {@code DataTransfer.dropEffect} object.
      * @param dragSourceExtension
      *         Drag source extension of the component that initiated the drop
      *         event.
      */
-    public DropEvent(T target, String dataTransferText, DropEffect dropEffect,
+    public DropEvent(T target, String dataTransferText,
             DragSourceExtension<? extends AbstractComponent> dragSourceExtension) {
         super(target);
 
         this.dataTransferText = dataTransferText;
 
-        this.dropEffect = dropEffect;
-
         this.dragSourceExtension = dragSourceExtension;
         this.dragSource = Optional.ofNullable(dragSourceExtension)
                 .map(DragSourceExtension::getParent).orElse(null);
@@ -75,15 +70,6 @@ public class DropEvent<T extends AbstractComponent> extends Component.Event {
     }
 
     /**
-     * Get drop effect set for the current drop target.
-     *
-     * @return {@code dropEffect} parameter set for the current drop target.
-     */
-    public DropEffect getDropEffect() {
-        return dropEffect;
-    }
-
-    /**
      * Returns the drag source component if the drag originated from a
      * component in the same UI as the drop target component, or an empty
      * optional.