JAL-4047 JAL-4948 show count of each dbref authority and 'first' reference

[jalview.git] / src / jalview / viewmodel / seqfeatures / IdColumns.java

diff --git a/src/jalview/viewmodel/seqfeatures/IdColumns.java b/src/jalview/viewmodel/seqfeatures/IdColumns.java
index 5b16b1a..a938d63 100644 (file)
--- a/src/jalview/viewmodel/seqfeatures/IdColumns.java
+++ b/src/jalview/viewmodel/seqfeatures/IdColumns.java
@@ -8,6 +8,7 @@ import java.util.Vector;
 
 import jalview.api.DBRefEntryI;
 import jalview.datamodel.AlignmentI;
+import jalview.datamodel.DBRefEntry;
 import jalview.datamodel.SequenceFeature;
 import jalview.datamodel.SequenceI;
 import jalview.datamodel.features.SequenceFeaturesI;
@@ -90,9 +91,12 @@ public class IdColumns
           updateListForFeature(sf);
         }
       }
-      for (DBRefEntryI dbr : seq.getDBRefs())
+      if (seq.getDBRefs() != null)
       {
-        updateListForDbxref(dbr);
+        for (DBRefEntryI dbr : seq.getDBRefs())
+        {
+          updateListForDbxref(dbr);
+        }
       }
     }
   }
@@ -184,14 +188,26 @@ public class IdColumns
       }
       if (col.DbRefName != null)
       {
+        List<DBRefEntryI> refs = new ArrayList<DBRefEntryI>();
         for (DBRefEntryI dbr : dseq.getDBRefs())
         {
           if (dbr.getSource().equals(col.DbRefName))
           {
-            return new ColumnCell(dbr.getAccessionId(), Color.black,
-                    Color.white);
+            refs.add(dbr);
           }
         }
+        if (refs.size()==0)
+        {
+          return null;
+        }
+        StringBuilder dbr = new StringBuilder();
+        dbr.append(refs.size());
+        for (DBRefEntryI ref : refs)
+        {
+          dbr.append(" "+ref.getAccessionId());
+        }
+        return new ColumnCell(dbr.toString(), Color.black, Color.white);
+
       }
     }
     // no value for this sequence in given column