Merge remote-tracking branch 'origin/releases/Release_2_10_4_Branch' into develop

[jalview.git] / src / jalview / ext / ensembl / EnsemblLookup.java

diff --git a/src/jalview/ext/ensembl/EnsemblLookup.java b/src/jalview/ext/ensembl/EnsemblLookup.java
index ed1b4fa..102dffb 100644 (file)
--- a/src/jalview/ext/ensembl/EnsemblLookup.java
+++ b/src/jalview/ext/ensembl/EnsemblLookup.java
@@ -49,11 +49,6 @@ public class EnsemblLookup extends EnsemblRestClient
   private static final String SPECIES = "species";
 
   /**
-   * keep track of last identifier retrieved to break loops
-   */
-  private String lastId;
-
-  /**
    * Default constructor (to use rest.ensembl.org)
    */
   public EnsemblLookup()
@@ -265,18 +260,8 @@ public class EnsemblLookup extends EnsemblRestClient
     BufferedReader br = null;
     try
     {
-
       URL url = getUrl(identifier, objectType);
 
-      if (identifier.equals(lastId))
-      {
-        System.err.println("** Ensembl lookup " + url.toString()
-                + " looping on Parent!");
-        return null;
-      }
-
-      lastId = identifier;
-
       if (url != null)
       {
         br = getHttpResponse(url, ids);
@@ -303,46 +288,6 @@ public class EnsemblLookup extends EnsemblRestClient
   }
 
   /**
-   * Parses the JSON response and returns the gene identifier, or null if not
-   * found. If the returned object_type is Gene, returns the id, if Transcript
-   * returns the Parent. If it is Translation (peptide identifier), then the
-   * Parent is the transcript identifier, so we redo the search with this value,
-   * specifying that object_type should be Transcript.
-   * 
-   * @param jsonObject
-   * @return
-   */
-  protected String parseGeneId(JSONObject json)
-  {
-    if (json == null)
-    {
-      // e.g. lookup failed with 404 not found
-      return null;
-    }
-
-    String geneId = null;
-    String type = json.get(OBJECT_TYPE).toString();
-    if (OBJECT_TYPE_GENE.equalsIgnoreCase(type))
-    {
-      // got the gene - just returns its id
-      geneId = json.get(JSON_ID).toString();
-    }
-    else if (OBJECT_TYPE_TRANSCRIPT.equalsIgnoreCase(type))
-    {
-      // got the transcript - return its (Gene) Parent
-      geneId = json.get(PARENT).toString();
-    }
-    else if (OBJECT_TYPE_TRANSLATION.equalsIgnoreCase(type))
-    {
-      // got the protein - look up its Parent, restricted to type Transcript
-      String transcriptId = json.get(PARENT).toString();
-      geneId = parseGeneId(getResult(transcriptId, OBJECT_TYPE_TRANSCRIPT));
-    }
-
-    return geneId;
-  }
-
-  /**
    * Calls the /lookup/id rest service for the given id, and if successful,
    * parses and returns the gene's chromosomal coordinates
    *