Attachment 74905 Details for Bug 197965 – fix

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] fix

fix_197965.txt (text/plain), 3.57 KB, created by Benno Baumgartner

on 2007-07-30 04:02:59 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Benno Baumgartner

Created: 2007-07-30 04:02:59 EDT

Size: 3.57 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jface
>Index: src/org/eclipse/jface/viewers/ViewerDropAdapter.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jface/src/org/eclipse/jface/viewers/ViewerDropAdapter.java,v
>retrieving revision 1.15
>diff -u -r1.15 ViewerDropAdapter.java
>--- src/org/eclipse/jface/viewers/ViewerDropAdapter.java	20 Jun 2007 19:53:46 -0000	1.15
>+++ src/org/eclipse/jface/viewers/ViewerDropAdapter.java	30 Jul 2007 08:03:29 -0000
>@@ -173,16 +173,7 @@
>      * that it is still enabled.
>      */
>     private void doDropValidation(DropTargetEvent event) {
>-        //update last valid operation
>-        if (event.detail != DND.DROP_NONE) {
>-            lastValidOperation = event.detail;
>-        }
>-        //valid drop and set event detail accordingly
>-        if (validateDrop(currentTarget, event.detail, event.currentDataType)) {
>-            currentOperation = lastValidOperation;
>-        } else {
>-            currentOperation = DND.DROP_NONE;
>-        }
>+        currentOperation= determineOperation(currentTarget, lastValidOperation, event.currentDataType);
>         event.detail = currentOperation;
>     }
> 
>@@ -193,6 +184,7 @@
>      */
>     public void dragEnter(DropTargetEvent event) {
>         currentTarget = determineTarget(event);
>+        lastValidOperation = event.detail;
>         doDropValidation(event);
>     }
> 
>@@ -203,6 +195,7 @@
>      */
>     public void dragOperationChanged(DropTargetEvent event) {
>         currentTarget = determineTarget(event);
>+        lastValidOperation = event.detail;
>         doDropValidation(event);
>     }
> 
>@@ -247,9 +240,7 @@
>      * Last chance for the action to disable itself
>      */
>     public void dropAccept(DropTargetEvent event) {
>-        if (!validateDrop(currentTarget, event.detail, event.currentDataType)) {
>-            event.detail = DND.DROP_NONE;
>-        }
>+        event.detail= determineOperation(currentTarget, event.detail, event.currentDataType);
>     }
> 
>     /**
>@@ -484,4 +475,37 @@
>      */
>     public abstract boolean validateDrop(Object target, int operation,
>             TransferData transferType);
>+    
>+    /**
>+     * Determine the operation which should be executed given the target and the operation
>+     * requested by the user. This method is called whenever some aspect of the drop operation 
>+     * changes. The operation is one of DND#DROP_DEFAULT, DND#DROP_COPY, DND#DROP_MOVE, DND#DROP_LINK.
>+     * <p>
>+     * The method returns the operation valid in the given context. The result is one of
>+     * DND#DROP_NONE, DND#DROP_COPY, DND#DROP_MOVE, DND#DROP_LINK, DND#DROP_DEFAULT.
>+     * </p>
>+     * <p>
>+     * Subclasses can overwrite this method to define which operation does make
>+     * sense in the given context.
>+     * </p>
>+     * 
>+     * @param target the object that the mouse is currently hovering over, or
>+     *   <code>null</code> if the mouse is hovering over empty space
>+     * @param operation the current drag operation (copy, move, etc.)
>+     * @param transferType the current transfer type
>+     * @return the operation which will be executed if no modifier key is pressed
>+     * 		by the user
>+     * 
>+     * @see DND#DROP_NONE
>+	 * @see DND#DROP_MOVE
>+	 * @see DND#DROP_COPY
>+	 * @see DND#DROP_LINK
>+     */
>+    protected int determineOperation(Object target, int operation, TransferData transferType) {
>+    	if (!validateDrop(target, operation, transferType)) {
>+    		return DND.DROP_NONE;
>+    	}
>+
>+    	return operation;
>+    }
> }

Actions: View | Diff

Attachments on bug 197965: 74689 | 74905 | 76108