JAL-6386 update slivka client for j2s compatibility
[jalview.git] / src / jalview / ws / slivkaws / SlivkaWSDiscoverer.java
index 7720b11..7a41431 100644 (file)
@@ -1,14 +1,16 @@
 package jalview.ws.slivkaws;
 
+import jalview.bin.Cache;
 import jalview.gui.AlignFrame;
-import jalview.ws.WSMenuEntryProviderI;
+import jalview.ws.WSDiscovererI;
 import jalview.ws.api.ServiceWithParameters;
 import jalview.ws.jws2.PreferredServiceRegistry;
 
+import java.beans.PropertyChangeListener;
 import java.beans.PropertyChangeSupport;
-import java.io.IOError;
 import java.io.IOException;
-import java.net.URISyntaxException;
+import java.net.MalformedURLException;
+import java.net.URL;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -17,22 +19,18 @@ import javax.swing.JMenu;
 import uk.ac.dundee.compbio.slivkaclient.SlivkaClient;
 import uk.ac.dundee.compbio.slivkaclient.SlivkaService;
 
-public class SlivkaWSDiscoverer implements Runnable, WSMenuEntryProviderI
+public class SlivkaWSDiscoverer implements WSDiscovererI
 {
+  private static final String SLIVKA_HOST_URLS = "SLIVKAHOSTURLS";
+
+  private static final String COMPBIO_SLIVKA = "https://www.compbio.dundee.ac.uk/slivka/";
+
   private static SlivkaWSDiscoverer instance = null;
 
-  private SlivkaClient slivkaClient;
+  private List<ServiceWithParameters> services = List.of();
 
   private SlivkaWSDiscoverer()
   {
-    try
-    {
-      slivkaClient = new SlivkaClient(
-              "gjb-www-1.cluster.lifesci.dundee.ac.uk", 3203);
-    } catch (URISyntaxException e)
-    {
-      throw new RuntimeException(e);
-    }
   }
 
   public static SlivkaWSDiscoverer getInstance()
@@ -44,37 +42,8 @@ public class SlivkaWSDiscoverer implements Runnable, WSMenuEntryProviderI
     return instance;
   }
 
-  List<ServiceWithParameters> getServiceInstances()
-  {
-    List<ServiceWithParameters> instances = new ArrayList<>();
-    for (SlivkaService service : services)
-    {
-      for (String classifier : service.classifiers)
-      {
-        if (classifier.contains("Multiple sequence alignment"))
-        {
-          instances
-                  .add(new SlivkaMsaServiceInstance(slivkaClient, service));
-        }
-        else if (classifier.contains("Protein sequence analysis"))
-        {
-          instances.add(new SlivkaAnnotationServiceInstance(slivkaClient,
-                  service, false));
-        }
-        else if (classifier
-                .contains("Sequence alignment analysis (conservation)"))
-        {
-          instances.add(new SlivkaAnnotationServiceInstance(slivkaClient,
-                  service, true));
-        }
-
-      }
-    }
-    return instances;
-  }
-
   private PropertyChangeSupport changeSupport = new PropertyChangeSupport(
-          this);
+      this);
 
   @Override
   public void attachWSMenuEntry(JMenu wsmenu, final AlignFrame alignFrame)
@@ -88,49 +57,193 @@ public class SlivkaWSDiscoverer implements Runnable, WSMenuEntryProviderI
     slivkaMenu.add(disorderMenu);
     JMenu conservationMenu = new JMenu("Conservation");
     slivkaMenu.add(conservationMenu);
-    PreferredServiceRegistry.getRegistry().populateWSMenuEntry(
-            getServiceInstances(),
-              changeSupport, slivkaMenu, alignFrame, null);
-    
+    PreferredServiceRegistry.getRegistry().populateWSMenuEntry(services,
+        changeSupport, slivkaMenu, alignFrame, null);
+
   }
 
-  List<SlivkaService>services=null;
+  volatile boolean ready = false;
 
-  volatile boolean started = false, finished = false;
+  volatile Thread discovererThread = null;
 
-  Thread discoverer = null;
-  @Override
-  public void run()
+  private class DiscovererThread extends Thread
   {
-    discoverer = Thread.currentThread();
-    started = true;
-    try
+    private Thread oldThread;
+
+    DiscovererThread(Thread oldThread)
     {
-      services = slivkaClient.getServices();
-    } catch (IOException e)
+      super();
+      this.oldThread = oldThread;
+    }
+
+    @Override
+    public void run()
     {
-      throw new IOError(e);
+      if (oldThread != null)
+      {
+        oldThread.interrupt();
+        try
+        {
+          oldThread.join();
+        } catch (InterruptedException e)
+        {
+          return;
+        } finally
+        {
+          oldThread = null;
+        }
+      }
+      ready = false;
+      reloadServices();
+      ready = !isInterrupted();
     }
-    finished = true;
   }
 
-  public static List<ServiceWithParameters> getServices()
+  Thread discoverer = null;
+
+  @Override
+  public Thread startDiscoverer(PropertyChangeListener changeListener)
   {
-    SlivkaWSDiscoverer us = getInstance();
-    if (us.services == null)
+    changeSupport.addPropertyChangeListener(changeListener);
+    ready = false;
+    (discovererThread = new DiscovererThread(discovererThread)).start();
+    return discovererThread;
+  }
+
+  private void reloadServices()
+  {
+    Cache.log.info("Reloading Slivka services");
+    changeSupport.firePropertyChange("services", services, List.of());
+    ArrayList<ServiceWithParameters> instances = new ArrayList<>();
+
+    for (String url : getServiceUrls())
     {
-      us.run();
+      Cache.log.info(url);
+      SlivkaClient client;
+      client = new SlivkaClient(url);
+      try
+      {
+        for (SlivkaService service : client.getServices())
+        {
+          SlivkaWSInstance newinstance = null;
+          for (String classifier : service.classifiers)
+          {
+            if (classifier.contains("Multiple sequence alignment"))
+            {
+              newinstance = new SlivkaMsaServiceInstance(client, service);
+            }
+            if (classifier.contains("Protein sequence analysis")
+                && newinstance == null)
+            {
+              newinstance = new SlivkaAnnotationServiceInstance(client,
+                  service, false);
+            }
+            if (classifier
+                .contains("Sequence alignment analysis (conservation)"))
+            {
+              newinstance = new SlivkaAnnotationServiceInstance(client,
+                  service, true);
+            }
+          }
+          if (newinstance != null)
+          {
+            instances.add(newinstance);
+          }
+        }
+      } catch (IOException e)
+      {
+        continue;
+      }
     }
-    return us.getServiceInstances();
+
+    services = instances;
+    changeSupport.firePropertyChange("services", List.of(), services);
+
+    Cache.log.info("Slivka services reloading finished");
+  }
+
+  @Override
+  public List<ServiceWithParameters> getServices()
+  {
+    return services;
   }
 
+  @Override
   public boolean hasServices()
   {
-    return finished == true && services != null && services.size() > 0;
+    return ready == true && services.size() > 0;
   }
 
+  @Override
   public boolean isRunning()
   {
-    return discoverer != null && discoverer.isAlive();
+    return discovererThread == null || discovererThread.isAlive()
+        || discovererThread.getState() == Thread.State.NEW;
+  }
+
+  @Override
+  public void setServiceUrls(List<String> wsUrls)
+  {
+    if (wsUrls != null && !wsUrls.isEmpty())
+    {
+      Cache.setProperty(SLIVKA_HOST_URLS, String.join(",", wsUrls));
+    }
+    else
+    {
+      Cache.removeProperty(SLIVKA_HOST_URLS);
+    }
+  }
+
+  @Override
+  public List<String> getServiceUrls()
+  {
+    String surls = Cache.getDefault(SLIVKA_HOST_URLS, COMPBIO_SLIVKA);
+    String[] urls = surls.split(",");
+    ArrayList<String> valid = new ArrayList<>(urls.length);
+    for (String url : urls)
+    {
+      try
+      {
+        new URL(url);
+        valid.add(url);
+      } catch (MalformedURLException e)
+      {
+        Cache.log.warn("Problem whilst trying to make a URL from '"
+            + ((url != null) ? url : "<null>") + "'");
+        Cache.log.warn(
+            "This was probably due to a malformed comma separated list"
+                + " in the " + SLIVKA_HOST_URLS
+                + " entry of $(HOME)/.jalview_properties)");
+        Cache.log.debug("Exception was ", e);
+      }
+    }
+    return valid;
+  }
+
+  @Override
+  public boolean testServiceUrl(URL url)
+  {
+    return getServerStatusFor(url.toString()) == STATUS_OK;
+  }
+
+  @Override
+  public int getServerStatusFor(String url)
+  {
+    try
+    {
+      List<?> services = new SlivkaClient(url).getServices();
+      return services.isEmpty() ? STATUS_NO_SERVICES : STATUS_OK;
+    } catch (IOException e)
+    {
+      Cache.log.error("Slivka could not retrieve services list", e);
+      return STATUS_INVALID;
+    }
+  }
+
+  @Override
+  public String getErrorMessages()
+  {
+    // TODO Auto-generated method stub
+    return "";
   }
 }