X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fanalysis%2FNJTree.java;h=78c4e600bc8d580e6a8df4a5854ebe8227217376;hb=d587f1aa61946dc14f6f089cf1dc2a3116cfb773;hp=4cd123f237841c08b161da3a4b8c2a42c2f8407e;hpb=aad4f693fc478138fd225741acf1f8d44defd162;p=jalview.git

diff --git a/src/jalview/analysis/NJTree.java b/src/jalview/analysis/NJTree.java
index 4cd123f..78c4e60 100755
--- a/src/jalview/analysis/NJTree.java
+++ b/src/jalview/analysis/NJTree.java
@@ -150,8 +150,8 @@ public class NJTree
                 j.setElement(nam);
                 if (one2many.contains(nam)) {
                   countOne2Many++;
-                  if (jalview.bin.Cache.log.isDebugEnabled())
-                    jalview.bin.Cache.log.debug("One 2 many relationship for "+nam.getName());
+                //  if (jalview.bin.Cache.log.isDebugEnabled())
+                //    jalview.bin.Cache.log.debug("One 2 many relationship for "+nam.getName());
                 } else {
                   one2many.addElement(nam);
                   namesleft--;
@@ -163,10 +163,10 @@ public class NJTree
                 j.setPlaceholder(true);
             }
         }
-        if (jalview.bin.Cache.log.isDebugEnabled() && countOne2Many>0) {
-          jalview.bin.Cache.log.debug("There were "+countOne2Many+" alignment sequence ids (out of "+one2many.size()+" unique ids) linked to two or more leaves.");
-        }
-        one2many.clear();
+      //  if (jalview.bin.Cache.log.isDebugEnabled() && countOne2Many>0) {
+      //    jalview.bin.Cache.log.debug("There were "+countOne2Many+" alignment sequence ids (out of "+one2many.size()+" unique ids) linked to two or more leaves.");
+      //  }
+      //  one2many.clear();
     }
 
     /**
@@ -284,12 +284,21 @@ public class NJTree
 
                 if (nam != null)
                 {
+                    if (!leaf.isPlaceholder()) {
+                      // remapping the node to a new sequenceI - should remove any refs to old one.
+                      // TODO - make many sequenceI to one leaf mappings possible! (JBPNote)
+                    }
                     leaf.setPlaceholder(false);
                     leaf.setElement(nam);
                 }
                 else
                 {
+                    if (!leaf.isPlaceholder()) {
+                        // Construct a new placeholder sequence object for this leaf
+                        leaf.setElement(new Sequence(leaf.getName(), "THISISAPLACEHLDER"));
+                    }
                     leaf.setPlaceholder(true);
+                    
                 }
             }
         }