Fix infinite loop when moving terms with parent terms

diff --git a/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/editor/definedterm/e4/DefinedTermDropAdapterE4.java b/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/editor/definedterm/e4/DefinedTermDropAdapterE4.java
index fb77f256089cd34845f22fe1f737e40b40bc4a28..6e7d9167e87ffecaa6789ff9783acdd3c2696fcf 100644 (file)
--- a/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/editor/definedterm/e4/DefinedTermDropAdapterE4.java
+++ b/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/editor/definedterm/e4/DefinedTermDropAdapterE4.java
@@ -93,20 +93,16 @@ public class DefinedTermDropAdapterE4 extends EditViewerDropAdapter {
                 }
 
                 TermDto partOfDto = parentTerm.getPartOfDto();
-                while(partOfDto!=null){
-                    if(partOfDto.equals(droppedObject)){
-                        MessagingUtils.warningDialog(MOVE_FAILED, this.getClass(),
-                                MOVE_FAILED_MESSAGE);
-                        return false;
-                    }
+                if(partOfDto!=null && partOfDto.equals(droppedObject)){
+                    MessagingUtils.warningDialog(MOVE_FAILED, this.getClass(),
+                            MOVE_FAILED_MESSAGE);
+                    return false;
                 }
                 TermDto kindOfDto = parentTerm.getKindOfDto();
-                while(kindOfDto!=null){
-                    if(kindOfDto.equals(droppedObject)){
-                        MessagingUtils.warningDialog(MOVE_FAILED, this.getClass(),
-                                MOVE_FAILED_MESSAGE);
-                        return false;
-                    }
+                if(kindOfDto!=null && kindOfDto.equals(droppedObject)){
+                    MessagingUtils.warningDialog(MOVE_FAILED, this.getClass(),
+                            MOVE_FAILED_MESSAGE);
+                    return false;
                 }
             }
             else if(target instanceof TermVocabularyDto){