- added feedback if drag&drop in DerivateVie does not work

diff --git a/eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/derivate/DerivateDropListener.java b/eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/derivate/DerivateDropListener.java
index c25c266f0a9690f5ab4c281ea332b2bf6fffde4d..5b0d134cbe67dc4f0421f940b2939859a3220256 100644 (file)
--- a/eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/derivate/DerivateDropListener.java
+++ b/eu.etaxonomy.taxeditor.editor/src/main/java/eu/etaxonomy/taxeditor/editor/view/derivate/DerivateDropListener.java
@@ -74,6 +74,9 @@ public class DerivateDropListener extends ViewerDropAdapter {
                     LocalSelectionTransfer.getTransfer().setSelection(null);
                     return true;
                 }
+                else{
+                    AbstractUtility.warningDialog("Moving derivate not possible!", derivateView, "Could not move "+draggednode.getValue()+" to "+targetNode.getValue());
+                }
             }
         }
         return false;
@@ -105,8 +108,7 @@ public class DerivateDropListener extends ViewerDropAdapter {
             //check if type is a sub derivate type
             if(!fromParentNode.equals(targetNode)//don't drag on direct parent node
                     && fromParentSpecimen instanceof SpecimenOrObservationBase<?>){
-                occurrenceService.moveDerivate((SpecimenOrObservationBase<?>)fromParentSpecimen, targetSpecimen, draggedSpecimen);
-                return true;
+                return occurrenceService.moveDerivate((SpecimenOrObservationBase<?>)fromParentSpecimen, targetSpecimen, draggedSpecimen);
             }
         }
 
@@ -114,17 +116,15 @@ public class DerivateDropListener extends ViewerDropAdapter {
         else if(draggedNodeValue instanceof SingleRead && targetNodeValue instanceof Sequence){
             SingleRead singleRead = (SingleRead) draggedNodeValue;
             if(fromParentSpecimen instanceof Sequence){
-                CdmStore.getService(ISequenceService.class).moveSingleRead((Sequence)fromParentSpecimen, (Sequence)targetNodeValue, singleRead);
+                return CdmStore.getService(ISequenceService.class).moveSingleRead((Sequence)fromParentSpecimen, (Sequence)targetNodeValue, singleRead);
             }
-            return true;
         }
 
         // drag'n'drop for Sequence
         else if(draggedNodeValue instanceof Sequence && targetNodeValue instanceof DnaSample ){
             Sequence sequence = (Sequence)draggedNodeValue;
             if(fromParentSpecimen instanceof DnaSample){
-                occurrenceService.moveSequence((DnaSample)fromParentSpecimen, (DnaSample)targetNodeValue, sequence);
-                return true;
+                return occurrenceService.moveSequence((DnaSample)fromParentSpecimen, (DnaSample)targetNodeValue, sequence);
             }
         }
         return false;