X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;ds=inline;f=src%2Fjalview%2Fdatamodel%2FDBRefEntry.java;h=98868ce64edbae6634dd50e80337b580a6ed6613;hb=788e840cc150aa4777a370cebfdf5d17589bbbaf;hp=143a4d27982c5206f6b5ab0b0e0f7f6b1cc3689f;hpb=d240900df4c81bd7b054afc0f569c7a75d4e1751;p=jalview.git
diff --git a/src/jalview/datamodel/DBRefEntry.java b/src/jalview/datamodel/DBRefEntry.java
index 143a4d2..98868ce 100755
--- a/src/jalview/datamodel/DBRefEntry.java
+++ b/src/jalview/datamodel/DBRefEntry.java
@@ -27,7 +27,20 @@ import java.util.List;
public class DBRefEntry implements DBRefEntryI
{
- String source = "", version = "", accessionId = "";
+ /*
+ * the mapping to chromosome (genome) is held as an instance with
+ * source = speciesId
+ * version = assemblyId
+ * accessionId = "chromosome:" + chromosomeId
+ * map = mapping from sequence to reference assembly
+ */
+ public static final String CHROMOSOME = "chromosome";
+
+ String source = "";
+
+ String version = "";
+
+ String accessionId = "";
/**
* maps from associated sequence to the database sequence's coordinate system
@@ -68,10 +81,10 @@ public class DBRefEntry implements DBRefEntryI
public DBRefEntry(DBRefEntryI entry)
{
this((entry.getSource() == null ? "" : new String(entry.getSource())),
- (entry.getVersion() == null ? "" : new String(
- entry.getVersion())),
- (entry.getAccessionId() == null ? "" : new String(
- entry.getAccessionId())),
+ (entry.getVersion() == null ? ""
+ : new String(entry.getVersion())),
+ (entry.getAccessionId() == null ? ""
+ : new String(entry.getAccessionId())),
(entry.getMap() == null ? null : new Mapping(entry.getMap())));
}
@@ -89,9 +102,8 @@ public class DBRefEntry implements DBRefEntryI
{
return true;
}
- if (equalRef(entry)
- && ((map == null && entry.map == null) || (map != null
- && entry.map != null && map.equals(entry.map))))
+ if (equalRef(entry) && ((map == null && entry.map == null)
+ || (map != null && entry.map != null && map.equals(entry.map))))
{
return true;
}
@@ -104,7 +116,8 @@ public class DBRefEntry implements DBRefEntryI
*
* - source and accession are identical (ignoring case)
* - version is identical (ignoring case), or this version is of the format
- * "someSource:0", in which case the version for the other entry replaces it
+ * "someSource:0", in which case the version for the other entry replaces
+ * it
* - mappings are not compared but if this entry has no mapping, replace
* with that for the other entry
*
@@ -141,8 +154,8 @@ public class DBRefEntry implements DBRefEntryI
String otherAccession = other.getAccessionId();
if ((accessionId == null && otherAccession != null)
|| (accessionId != null && otherAccession == null)
- || (accessionId != null && !accessionId
- .equalsIgnoreCase(otherAccession)))
+ || (accessionId != null
+ && !accessionId.equalsIgnoreCase(otherAccession)))
{
return false;
}
@@ -160,9 +173,8 @@ public class DBRefEntry implements DBRefEntryI
}
else
{
- if (version != null
- && (otherVersion == null || !version
- .equalsIgnoreCase(otherVersion)))
+ if (version != null && (otherVersion == null
+ || !version.equalsIgnoreCase(otherVersion)))
{
return false;
}
@@ -197,12 +209,12 @@ public class DBRefEntry implements DBRefEntryI
return true;
}
if (entry != null
- && (source != null && entry.getSource() != null && source
- .equalsIgnoreCase(entry.getSource()))
- && (accessionId != null && entry.getAccessionId() != null && accessionId
- .equalsIgnoreCase(entry.getAccessionId()))
- && (version != null && entry.getVersion() != null && version
- .equalsIgnoreCase(entry.getVersion())))
+ && (source != null && entry.getSource() != null
+ && source.equalsIgnoreCase(entry.getSource()))
+ && (accessionId != null && entry.getAccessionId() != null
+ && accessionId.equalsIgnoreCase(entry.getAccessionId()))
+ && (version != null && entry.getVersion() != null
+ && version.equalsIgnoreCase(entry.getVersion())))
{
return true;
}
@@ -282,7 +294,7 @@ public class DBRefEntry implements DBRefEntryI
}
@Override
- public boolean isPrimary()
+ public boolean isPrimaryCandidate()
{
/*
* if a map is present, unless it is 1:1 and has no SequenceI mate, it cannot be a primary reference.
@@ -298,7 +310,7 @@ public class DBRefEntry implements DBRefEntryI
{
return false;
}
- // check map is really 1:1, between identical single ranges
+ // check map is between identical single contiguous ranges
List fromRanges = map.getMap().getFromRanges();
List toRanges = map.getMap().getToRanges();
if (fromRanges.size() != 1 || toRanges.size() != 1)
@@ -332,4 +344,14 @@ public class DBRefEntry implements DBRefEntryI
}
return true;
}
+
+ /**
+ * Mappings to chromosome are held with accessionId as "chromosome:id"
+ *
+ * @return
+ */
+ public boolean isChromosome()
+ {
+ return accessionId != null && accessionId.startsWith(CHROMOSOME + ":");
+ }
}