Merge branch 'feature/JAL-3187linkedFeatures' into feature/JAL-3251biotypedMappings

[jalview.git] / src / jalview / io / gff / GffHelperBase.java

diff --git a/src/jalview/io/gff/GffHelperBase.java b/src/jalview/io/gff/GffHelperBase.java
index 1d4d3ac..2206a03 100644 (file)
--- a/src/jalview/io/gff/GffHelperBase.java
+++ b/src/jalview/io/gff/GffHelperBase.java
@@ -88,15 +88,15 @@ public abstract class GffHelperBase implements GffHelperI
     int[] to = new int[] { toStart, toEnd };
 
     /*
-     * Jalview always models from dna to protein, so switch values if the
-     * GFF mapping is from protein to dna
+     * Jalview always models from dna to protein, so invert 
+     * mapping if the GFF mapping is from protein to dna
      */
-    if (mappingType == MappingType.PeptideToNucleotide)
+    if (MappingType.isPeptideToDna(mappingType))
     {
       int[] temp = from;
       from = to;
       to = temp;
-      mappingType = mappingType.getInverse();
+      mappingType = MappingType.CdsToPeptide;
     }
 
     int fromRatio = mappingType.getFromRatio();
@@ -286,7 +286,7 @@ public abstract class GffHelperBase implements GffHelperI
           String namesDelimiter, char nameValueSeparator,
           String valuesDelimiter)
   {
-    Map<String, List<String>> map = new HashMap<String, List<String>>();
+    Map<String, List<String>> map = new HashMap<>();
     if (text == null || text.trim().length() == 0)
     {
       return map;
@@ -314,7 +314,7 @@ public abstract class GffHelperBase implements GffHelperI
         List<String> vals = map.get(key);
         if (vals == null)
         {
-          vals = new ArrayList<String>();
+          vals = new ArrayList<>();
           map.put(key, vals);
         }
         for (String val : values.split(valuesDelimiter))