From: jprocter <Jim Procter>
Date: Tue, 4 Nov 2008 13:31:19 +0000 (+0000)
Subject: bugfix db ref retrieve routine so it removes dead query strings properly
X-Git-Tag: Release_2_5~428
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=7d71a7e377e57d614a772ce6a80e8795dd2a3e24;hp=77089d22860b84145b18f084efe95f6efa484674;p=jalview.git

bugfix db ref retrieve routine so it removes dead query strings properly
---

diff --git a/src/jalview/ws/DBRefFetcher.java b/src/jalview/ws/DBRefFetcher.java
index 76e73e4..96ac7e8 100644
--- a/src/jalview/ws/DBRefFetcher.java
+++ b/src/jalview/ws/DBRefFetcher.java
@@ -213,27 +213,30 @@ public class DBRefFetcher implements Runnable
         {
           // Still queries to make for current seqIndex
           StringBuffer queryString = new StringBuffer("");
-          int nqSize = (maxqlen > queries.size()) ? queries.size()
+          int numq=0,nqSize = (maxqlen > queries.size()) ? queries.size()
                   : maxqlen;
-          for (int nq = 0, numq = 0; nq < nqSize; nq++)
+          
+          while (queries.size()>0 && numq < nqSize)
           {
-            String query = (String) queries.elementAt(nq);
+            String query = (String) queries.elementAt(0);
             if (dbsource.isValidReference(query))
             {
-              queryString.append((nq == 0) ? "" : dbsource
+              queryString.append((numq == 0) ? "" : dbsource
                       .getAccessionSeparator());
               queryString.append(query);
               numq++;
             }
-          }
-          for (int nq = 0; nq < nqSize; nq++)
-          {
+            // remove the extracted query string
             queries.removeElementAt(0);
           }
           // make the queries and process the response
           AlignmentI retrieved = null;
           try
           {
+            if (jalview.bin.Cache.log.isDebugEnabled())
+            {
+              jalview.bin.Cache.log.debug("Querying "+dbsource.getDbName()+" with : '"+queryString.toString()+"'");
+            }
             retrieved = dbsource.getSequenceRecords(queryString.toString());
           } catch (Exception ex)
           {
@@ -259,7 +262,7 @@ public class DBRefFetcher implements Runnable
                     { dbSources[db] }); // jalview.datamodel.DBRefSource.UNIPROT
             // });
             // check for existing dbrefs to use
-            if (uprefs != null)
+            if (uprefs != null && uprefs.length>0)
             {
               for (int j = 0; j < uprefs.length; j++)
               {