X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fschemes%2FPIDColourScheme.java;h=9bd092e6be7f96f9a56370ee33bfd210e685e22d;hb=d587f1aa61946dc14f6f089cf1dc2a3116cfb773;hp=a79f0833da6dc4ffcc7d25e1b9b315cf6a10bc56;hpb=174230b4233d9ce80f94527768d2cd2f76da11ab;p=jalview.git

diff --git a/src/jalview/schemes/PIDColourScheme.java b/src/jalview/schemes/PIDColourScheme.java
index a79f083..9bd092e 100755
--- a/src/jalview/schemes/PIDColourScheme.java
+++ b/src/jalview/schemes/PIDColourScheme.java
@@ -21,6 +21,7 @@ package jalview.schemes;
 import java.awt.*;
 
 import jalview.datamodel.*;
+import jalview.analysis.AAFrequency;
 
 public class PIDColourScheme
     extends ResidueColourScheme
@@ -43,20 +44,26 @@ public class PIDColourScheme
       s = String.valueOf(res -= ('a' - 'A'));
     }
 
+    if (consensus == null
+        || j >= consensus.length
+        || consensus[j]==null)
+    {
+      return Color.white;
+    }
 
     if ( (threshold != 0) && !aboveThreshold(s, j))
     {
       return Color.white;
     }
 
-    currentColour = Color.white;
+    Color currentColour = Color.white;
 
     double sc = 0;
 
     if(consensus.length<=j)
       return Color.white;
 
-      if ( (Integer.parseInt(consensus[j].get("maxCount").toString()) != -1) &&
+      if ( (Integer.parseInt(consensus[j].get(AAFrequency.MAXCOUNT).toString()) != -1) &&
           consensus[j].contains(s))
       {
         sc = ( (Float) consensus[j].get(ignoreGaps)).floatValue();
@@ -76,7 +83,7 @@ public class PIDColourScheme
       }
 
     if(conservationColouring)
-         applyConservation(j);
+      currentColour = applyConservation(currentColour, j);
 
     return currentColour;
   }