X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;ds=sidebyside;f=src%2Fjalview%2Ffts%2Fservice%2Fpdb%2FPDBFTSRestClient.java;h=5293c3206d7ef8f683cabbb2dc3e2e76f96c3f14;hb=aace9d05c0870c703bfdfb28c1608213cee019bf;hp=219d6d6dd2de31176cc600c33910c9649a26bdbc;hpb=2051c117e19e8b7c72bf5a9f1db48f2b4687f71e;p=jalview.git diff --git a/src/jalview/fts/service/pdb/PDBFTSRestClient.java b/src/jalview/fts/service/pdb/PDBFTSRestClient.java index 219d6d6..5293c32 100644 --- a/src/jalview/fts/service/pdb/PDBFTSRestClient.java +++ b/src/jalview/fts/service/pdb/PDBFTSRestClient.java @@ -59,7 +59,7 @@ public class PDBFTSRestClient extends FTSRestClient private static FTSRestClientI instance = null; - public static final String PDB_SEARCH_ENDPOINT = "http://www.ebi.ac.uk/pdbe/search/pdb/select?"; + public static final String PDB_SEARCH_ENDPOINT = "https://www.ebi.ac.uk/pdbe/search/pdb/select?"; protected PDBFTSRestClient() { @@ -82,9 +82,10 @@ public class PDBFTSRestClient extends FTSRestClient ClientConfig clientConfig = new DefaultClientConfig(); Client client = Client.create(clientConfig); - String wantedFields = getDataColumnsFieldsAsCommaDelimitedString(pdbRestRequest - .getWantedFields()); - int responseSize = (pdbRestRequest.getResponseSize() == 0) ? getDefaultResponsePageSize() + String wantedFields = getDataColumnsFieldsAsCommaDelimitedString( + pdbRestRequest.getWantedFields()); + int responseSize = (pdbRestRequest.getResponseSize() == 0) + ? getDefaultResponsePageSize() : pdbRestRequest.getResponseSize(); int offSet = pdbRestRequest.getOffSet(); String sortParam = null; @@ -108,11 +109,11 @@ public class PDBFTSRestClient extends FTSRestClient } } - String facetPivot = (pdbRestRequest.getFacetPivot() == null || pdbRestRequest - .getFacetPivot().isEmpty()) ? "" : pdbRestRequest - .getFacetPivot(); - String facetPivotMinCount = String.valueOf(pdbRestRequest - .getFacetPivotMinCount()); + String facetPivot = (pdbRestRequest.getFacetPivot() == null + || pdbRestRequest.getFacetPivot().isEmpty()) ? "" + : pdbRestRequest.getFacetPivot(); + String facetPivotMinCount = String + .valueOf(pdbRestRequest.getFacetPivotMinCount()); String query = pdbRestRequest.getFieldToSearchBy() + pdbRestRequest.getSearchTerm() @@ -140,30 +141,30 @@ public class PDBFTSRestClient extends FTSRestClient .queryParam("wt", "json").queryParam("fl", wantedFields) .queryParam("rows", String.valueOf(responseSize)) .queryParam("start", String.valueOf(offSet)) - .queryParam("q", query) - .queryParam("sort", sortParam); + .queryParam("q", query).queryParam("sort", sortParam); } // Execute the REST request - ClientResponse clientResponse = webResource.accept( - MediaType.APPLICATION_JSON).get(ClientResponse.class); + ClientResponse clientResponse = webResource + .accept(MediaType.APPLICATION_JSON).get(ClientResponse.class); // Get the JSON string from the response object String responseString = clientResponse.getEntity(String.class); // System.out.println("query >>>>>>> " + pdbRestRequest.toString()); // Check the response status and report exception if one occurs - if (clientResponse.getStatus() != 200) + int responseStatus = clientResponse.getStatus(); + if (responseStatus != 200) { String errorMessage = ""; - if (clientResponse.getStatus() == 400) + if (responseStatus == 400) { errorMessage = parseJsonExceptionString(responseString); throw new Exception(errorMessage); } else { - errorMessage = getMessageByHTTPStatusCode(clientResponse -.getStatus(), "PDB"); + errorMessage = getMessageByHTTPStatusCode( + responseStatus, "PDB"); throw new Exception(errorMessage); } } @@ -181,9 +182,8 @@ public class PDBFTSRestClient extends FTSRestClient if (exceptionMsg.contains("SocketException")) { // No internet connection - throw new Exception( - MessageManager - .getString("exception.unable_to_detect_internet_connection")); + throw new Exception(MessageManager.getString( + "exception.unable_to_detect_internet_connection")); } else if (exceptionMsg.contains("UnknownHostException")) { @@ -198,7 +198,6 @@ public class PDBFTSRestClient extends FTSRestClient } } - /** * Process error response from PDB server if/when one occurs. * @@ -262,8 +261,8 @@ public class PDBFTSRestClient extends FTSRestClient .parse(pdbJsonResponseString); JSONObject pdbResponse = (JSONObject) jsonObj.get("response"); - String queryTime = ((JSONObject) jsonObj.get("responseHeader")).get( - "QTime").toString(); + String queryTime = ((JSONObject) jsonObj.get("responseHeader")) + .get("QTime").toString(); int numFound = Integer .valueOf(pdbResponse.get("numFound").toString()); if (numFound > 0) @@ -300,8 +299,9 @@ public class PDBFTSRestClient extends FTSRestClient Collection diplayFields = request.getWantedFields(); SequenceI associatedSeq = request.getAssociatedSequence(); int colCounter = 0; - summaryRowData = new Object[(associatedSeq != null) ? diplayFields - .size() + 1 : diplayFields.size()]; + summaryRowData = new Object[(associatedSeq != null) + ? diplayFields.size() + 1 + : diplayFields.size()]; if (associatedSeq != null) { associatedSequence = associatedSeq; @@ -327,16 +327,16 @@ public class PDBFTSRestClient extends FTSRestClient try { summaryRowData[colCounter++] = (field.getDataType() - .getDataTypeClass() == Integer.class) ? Integer - .valueOf(fieldData) - : (field.getDataType() - .getDataTypeClass() == Double.class) ? Double - .valueOf(fieldData) - : sanitiseData(fieldData); + .getDataTypeClass() == Integer.class) + ? Integer.valueOf(fieldData) + : (field.getDataType() + .getDataTypeClass() == Double.class) + ? Double.valueOf(fieldData) + : sanitiseData(fieldData); } catch (Exception e) { e.printStackTrace(); - System.out.println("offending value:" + fieldData); + System.out.println("offending value:" + fieldData); } } } @@ -405,7 +405,6 @@ public class PDBFTSRestClient extends FTSRestClient return "/fts/pdb_data_columns.txt"; } - public static FTSRestClientI getInstance() { if (instance == null) @@ -423,8 +422,8 @@ public class PDBFTSRestClient extends FTSRestClient || allDefaultDisplayedStructureDataColumns.isEmpty()) { allDefaultDisplayedStructureDataColumns = new ArrayList(); - allDefaultDisplayedStructureDataColumns.addAll(super - .getAllDefaultDisplayedFTSDataColumns()); + allDefaultDisplayedStructureDataColumns + .addAll(super.getAllDefaultDisplayedFTSDataColumns()); } return allDefaultDisplayedStructureDataColumns; }