X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;ds=sidebyside;f=src%2Fjalview%2Fext%2Fensembl%2FEnsemblFeatures.java;h=4d6c1a9fa43ca0761f5d19f119434966ea1c34fd;hb=c794c5033adeee182b03a5ea92c0a7495a29661f;hp=99ebc69a0349fa2d2ba1bfe8cc2423ca6fccc69b;hpb=3609d4b908fa64cab35f2348401baab3347188fc;p=jalview.git

diff --git a/src/jalview/ext/ensembl/EnsemblFeatures.java b/src/jalview/ext/ensembl/EnsemblFeatures.java
index 99ebc69..4d6c1a9 100644
--- a/src/jalview/ext/ensembl/EnsemblFeatures.java
+++ b/src/jalview/ext/ensembl/EnsemblFeatures.java
@@ -28,17 +28,14 @@ import jalview.datamodel.SequenceI;
 import jalview.io.gff.SequenceOntologyI;
 import jalview.util.JSONUtils;
 
-import java.io.BufferedReader;
 import java.io.IOException;
 import java.net.MalformedURLException;
 import java.net.URL;
 import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
+import java.util.Map;
 
-import org.json.simple.JSONArray;
-import org.json.simple.JSONObject;
-import org.json.simple.parser.JSONParser;
 import org.json.simple.parser.ParseException;
 
 /**
@@ -92,16 +89,14 @@ class EnsemblFeatures extends EnsemblRestClient
   public AlignmentI getSequenceRecords(String query) throws IOException
   {
     // TODO: use a vararg String... for getSequenceRecords instead?
+	  
     List<String> queries = new ArrayList<>();
     queries.add(query);
-    BufferedReader fp = getSequenceReader(queries);
-    if (fp == null)
-    {
-      return null;
-    }
-
-    SequenceI seq = parseFeaturesJson(fp);
+    SequenceI seq = parseFeaturesJson(queries);
+    if (seq == null)
+    	return null;
     return new Alignment(new SequenceI[] { seq });
+
   }
 
   /**
@@ -111,20 +106,22 @@ class EnsemblFeatures extends EnsemblRestClient
    * @param br
    * @return
    */
-  private SequenceI parseFeaturesJson(BufferedReader br)
+  @SuppressWarnings("unchecked")
+  private SequenceI parseFeaturesJson(List<String> queries)
   {
     SequenceI seq = new Sequence("Dummy", "");
-
-    JSONParser jp = new JSONParser();
     try
     {
-      JSONArray responses = (JSONArray) jp.parse(br);
-      Iterator rvals = responses.iterator();
-      while (rvals.hasNext())
+      Iterator<Object> rvals = (Iterator<Object>) getJSON(null, queries, -1, MODE_ITERATOR, null);
+      if (rvals == null)
       {
+    	  return null;
+      }
+      while (rvals.hasNext())
+      {    	  
         try
         {
-          JSONObject obj = (JSONObject) rvals.next();
+          Map<String, Object> obj = (Map<String, Object>) rvals.next();
           String type = obj.get("feature_type").toString();
           int start = Integer.parseInt(obj.get("start").toString());
           int end = Integer.parseInt(obj.get("end").toString());
@@ -132,10 +129,10 @@ class EnsemblFeatures extends EnsemblRestClient
           String strand = obj.get("strand").toString();
           Object phase = obj.get("phase");
           String alleles = JSONUtils
-                  .arrayToList((JSONArray) obj.get("alleles"));
+                  .arrayToStringList((List<Object>) obj.get("alleles"));
           String clinSig = JSONUtils
-                  .arrayToList(
-                          (JSONArray) obj.get("clinical_significance"));
+                  .arrayToStringList(
+                          (List<Object>) obj.get("clinical_significance"));
 
           /*
            * convert 'variation' to 'sequence_variant', and 'cds' to 'CDS'
@@ -150,7 +147,7 @@ class EnsemblFeatures extends EnsemblRestClient
           {
             type = SequenceOntologyI.CDS;
           }
-          
+
           String desc = getFirstNotNull(obj, "alleles", "external_name",
                   JSON_ID);
           SequenceFeature sf = new SequenceFeature(type, desc, start, end,
@@ -167,6 +164,7 @@ class EnsemblFeatures extends EnsemblRestClient
           sf.setValue("clinical_significance", clinSig);
 
           seq.addSequenceFeature(sf);
+          
         } catch (Throwable t)
         {
           // ignore - keep trying other features
@@ -174,6 +172,7 @@ class EnsemblFeatures extends EnsemblRestClient
       }
     } catch (ParseException | IOException e)
     {
+    	e.printStackTrace();
       // ignore
     }
 
@@ -189,17 +188,16 @@ class EnsemblFeatures extends EnsemblRestClient
    * @param keys
    * @return
    */
-  protected String getFirstNotNull(JSONObject obj, String... keys)
+  @SuppressWarnings("unchecked")
+  protected String getFirstNotNull(Map<String, Object> obj, String... keys)
   {
-    String desc = null;
-
     for (String key : keys)
     {
       Object val = obj.get(key);
       if (val != null)
       {
-        String s = val instanceof JSONArray
-                ? JSONUtils.arrayToList((JSONArray) val)
+        String s = val instanceof List<?>
+                ? JSONUtils.arrayToStringList((List<Object>) val)
                 : val.toString();
         if (!s.isEmpty())
         {
@@ -207,7 +205,7 @@ class EnsemblFeatures extends EnsemblRestClient
         }
       }
     }
-    return desc;
+    return null;
   }
 
   /**
@@ -218,7 +216,7 @@ class EnsemblFeatures extends EnsemblRestClient
    * @param obj
    * @param key
    */
-  protected void setFeatureAttribute(SequenceFeature sf, JSONObject obj,
+  protected void setFeatureAttribute(SequenceFeature sf, Map<String, Object> obj,
           String key)
   {
     Object object = obj.get(key);