X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fws%2Fjws2%2FJws2Discoverer.java;h=73d37009638232a079a1d74883d11ef02aa8fba1;hb=2facc0e2aee62c60c28cc59277805194b306f6f9;hp=dc7524669d81421055d697bf7ff6a6a81ac7da93;hpb=06931e7445db77675739e0f206d7b5fafc3f9c05;p=jalview.git diff --git a/src/jalview/ws/jws2/Jws2Discoverer.java b/src/jalview/ws/jws2/Jws2Discoverer.java index dc75246..73d3700 100644 --- a/src/jalview/ws/jws2/Jws2Discoverer.java +++ b/src/jalview/ws/jws2/Jws2Discoverer.java @@ -1,59 +1,53 @@ /* - * Jalview - A Sequence Alignment Editor and Viewer (Version 2.6) - * Copyright (C) 2010 J Procter, AM Waterhouse, G Barton, M Clamp, S Searle + * Jalview - A Sequence Alignment Editor and Viewer ($$Version-Rel$$) + * Copyright (C) $$Year-Rel$$ The Jalview Authors * * This file is part of Jalview. * * Jalview is free software: you can redistribute it and/or * modify it under the terms of the GNU General Public License - * as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version. - * + * as published by the Free Software Foundation, either version 3 + * of the License, or (at your option) any later version. + * * Jalview is distributed in the hope that it will be useful, but * WITHOUT ANY WARRANTY; without even the implied warranty * of MERCHANTABILITY or FITNESS FOR A PARTICULAR * PURPOSE. See the GNU General Public License for more details. * - * You should have received a copy of the GNU General Public License along with Jalview. If not, see . + * You should have received a copy of the GNU General Public License + * along with Jalview. If not, see . + * The Jalview Authors are detailed in the 'AUTHORS' file. */ package jalview.ws.jws2; -import java.awt.Color; -import java.awt.event.ActionEvent; -import java.awt.event.ActionListener; +import jalview.bin.Cache; +import jalview.gui.AlignFrame; +import jalview.util.MessageManager; +import jalview.ws.ServiceChangeListener; +import jalview.ws.WSDiscovererI; +import jalview.ws.api.ServiceWithParameters; +import jalview.ws.jws2.jabaws2.Jws2Instance; +import jalview.ws.params.ParamDatastoreI; + import java.beans.PropertyChangeEvent; import java.beans.PropertyChangeListener; -import java.io.Closeable; -import java.net.ConnectException; +import java.beans.PropertyChangeSupport; +import java.net.MalformedURLException; import java.net.URL; import java.util.ArrayList; +import java.util.Collections; import java.util.HashSet; -import java.util.Hashtable; import java.util.List; +import java.util.Set; import java.util.StringTokenizer; import java.util.Vector; +import java.util.concurrent.CopyOnWriteArraySet; +import java.util.concurrent.ExecutionException; +import java.util.concurrent.Future; +import java.util.concurrent.FutureTask; import javax.swing.JMenu; -import javax.swing.JMenuItem; -import javax.swing.event.MenuEvent; -import javax.swing.event.MenuListener; - -import org.apache.log4j.Level; -import jalview.bin.Cache; -import jalview.datamodel.AlignmentView; -import jalview.gui.AlignFrame; -import jalview.gui.Desktop; -import jalview.gui.JalviewChangeSupport; -import jalview.gui.JvSwingUtils; -import jalview.ws.WSMenuEntryProviderI; -import jalview.ws.jws2.Jws2Discoverer.Jws2Instance; -import jalview.ws.params.ParamDatastoreI; -import compbio.data.msa.MsaWS; -import compbio.metadata.Option; -import compbio.metadata.Preset; -import compbio.metadata.PresetManager; -import compbio.metadata.RunnerConfig; -import compbio.ws.client.Jws2Client; import compbio.ws.client.Services; /** @@ -63,36 +57,74 @@ import compbio.ws.client.Services; * @author JimP * */ -public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI +public class Jws2Discoverer implements WSDiscovererI, Runnable { - private java.beans.PropertyChangeSupport changeSupport = new java.beans.PropertyChangeSupport( - this); + public static final String COMPBIO_JABAWS = "http://www.compbio.dundee.ac.uk/jabaws"; + + /* + * the .jalview_properties entry for JWS2 URLS + */ + private final static String JWS2HOSTURLS = "JWS2HOSTURLS"; + + /* + * Singleton instance + */ + private static Jws2Discoverer discoverer; + + /* + * Override for testing only + */ + private static List testUrls = null; + + // preferred url has precedence over others + private String preferredUrl; + + private Set serviceListeners = new CopyOnWriteArraySet<>(); + + private Vector invalidServiceUrls = null; + + private Vector urlsWithoutServices = null; + + private Vector validServiceUrls = null; + + private volatile boolean running = false; + + private volatile boolean aborted = false; + + private volatile Thread oldthread = null; /** - * change listeners are notified of "services" property changes - * - * @param listener - * to be added that consumes new services Hashtable object. + * holds list of services. */ - public void addPropertyChangeListener( - java.beans.PropertyChangeListener listener) - { - changeSupport.addPropertyChangeListener(listener); - } + protected Vector services; /** - * - * - * @param listener - * to be removed + * Private constructor enforces use of singleton via getDiscoverer() */ - public void removePropertyChangeListener( - java.beans.PropertyChangeListener listener) + private Jws2Discoverer() { - changeSupport.removePropertyChangeListener(listener); } - boolean running = false, aborted = false; + + @Override + public void addServiceChangeListener(ServiceChangeListener listener) + { + serviceListeners.add(listener); + } + + @Override + public void removeServiceChangeListener(ServiceChangeListener listener) + { + serviceListeners.remove(listener); + } + + private void notifyServiceListeners(List services) + { + if (services == null) services = this.services; + for (var listener : serviceListeners) { + listener.servicesChanged(this, services); + } + } /** * @return the aborted @@ -111,10 +143,10 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI this.aborted = aborted; } - Thread oldthread = null; - + @Override public void run() { + if (running && oldthread != null && oldthread.isAlive()) { if (!aborted) @@ -125,26 +157,36 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI { try { - Cache.log - .debug("Waiting around for old discovery thread to finish."); + Cache.log.debug( + "Waiting around for old discovery thread to finish."); // wait around until old discoverer dies Thread.sleep(100); } catch (Exception e) { } } + aborted = false; Cache.log.debug("Old discovery thread has finished."); } running = true; + + // first set up exclusion list if needed + final Set ignoredServices = new HashSet<>(); + for (String ignored : Cache + .getDefault("IGNORED_JABAWS_SERVICETYPES", "").split("\\|")) + { + ignoredServices.add(ignored); + } + + notifyServiceListeners(Collections.emptyList()); oldthread = Thread.currentThread(); try { - Class foo = getClass().getClassLoader().loadClass( - "compbio.ws.client.Jws2Client"); + getClass().getClassLoader().loadClass("compbio.ws.client.Jws2Client"); } catch (ClassNotFoundException e) { - System.err - .println("Not enabling JABA Webservices : client jar is not available." + System.err.println( + "Not enabling JABA Webservices : client jar is not available." + "\nPlease check that your webstart JNLP file is up to date!"); running = false; return; @@ -162,44 +204,90 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI { invalidServiceUrls.removeAllElements(); } + if (validServiceUrls != null) + { + validServiceUrls.removeAllElements(); + } + ArrayList svctypes = new ArrayList<>(); - List qrys = new ArrayList(); - for (final String jwsservers : getServiceUrls()) + List qrys = new ArrayList<>(); + for (final String jwsserver : getServiceUrls()) { - JabaWsServerQuery squery = new JabaWsServerQuery(this, jwsservers); + JabaWsServerQuery squery = new JabaWsServerQuery(this, jwsserver); + if (svctypes.size() == 0) + { + // TODO: remove this ugly hack to get Canonical JABA service ordering + // for all possible services + for (Services sv : squery.JABAWS2SERVERS) + { + if (!ignoredServices.contains(sv.toString())) + { + svctypes.add(sv.toString()); + } + } + + } qrys.add(squery); new Thread(squery).start(); } - boolean finished = false; + boolean finished = true; do { + finished = true; try { Thread.sleep(100); } catch (Exception e) { } - ; for (JabaWsServerQuery squery : qrys) { - finished |= !squery.isRunning(); + if (squery.isRunning()) + { + finished = false; + } } if (aborted) { - Cache.log.debug("Aborting " + qrys.size() - + " JABAWS discovery threads."); + Cache.log.debug( + "Aborting " + qrys.size() + " JABAWS discovery threads."); for (JabaWsServerQuery squery : qrys) { squery.setQuit(true); } } } while (!aborted && !finished); - oldthread = null; - running = false; if (!aborted) { - changeSupport.firePropertyChange("services", new Vector(), services); + // resort services according to order found in jabaws service list + // also ensure services for each host are ordered in same way. + + if (services != null && services.size() > 0) + { + Jws2Instance[] svcs = new Jws2Instance[services.size()]; + int[] spos = new int[services.size()]; + int ipos = 0; + List svcUrls = getServiceUrls(); + for (Jws2Instance svc : services) + { + svcs[ipos] = svc; + spos[ipos++] = 1000 * svcUrls.indexOf(svc.getHostURL()) + 1 + + svctypes.indexOf(svc.getName()); + } + jalview.util.QuickSort.sort(spos, svcs); + services = new Vector<>(); + for (Jws2Instance svc : svcs) + { + if (!ignoredServices.contains(svc.getName())) + { + services.add(svc); + } + } + } } + oldthread = null; + running = false; + notifyServiceListeners(services); } /** @@ -209,342 +297,83 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI * @param srv * @param service2 */ - synchronized void addService(String jwsservers, Services srv, - MsaWS service2) + synchronized void addService(String jwsservers, Jws2Instance service) { if (services == null) { - services = new Vector(); + services = new Vector<>(); } - System.out.println("Discovered service: " + jwsservers + " " - + srv.toString()); - Jws2Instance service = new Jws2Instance(jwsservers, srv.toString(), - service2); + System.out.println( + "Discovered service: " + jwsservers + " " + service.toString()); + // Jws2Instance service = new Jws2Instance(jwsservers, srv.toString(), + // service2); services.add(service); // retrieve the presets and parameter set and cache now - service.getParamStore().getPresets(); - service.hasParameters(); - } - - public class Jws2Instance - { - public String hosturl; - - public String serviceType; - - public MsaWS service; - - public Jws2Instance(String hosturl, String serviceType, MsaWS service) + ParamDatastoreI pds = service.getParamStore(); + if (pds != null) { - super(); - this.hosturl = hosturl; - this.serviceType = serviceType; - this.service = service; + pds.getPresets(); } - - PresetManager presets = null; - - public JabaParamStore paramStore = null; - - /** - * non thread safe - gets the presets for this service (blocks whilst it - * calls the service to get the preset set) - * - * @return service presets or null if exceptions were raised. - */ - public PresetManager getPresets() - { - if (presets == null) - { - try - { - presets = service.getPresets(); - } catch (Exception ex) - { - System.err - .println("Exception when retrieving presets for service " - + serviceType + " at " + hosturl); - } - } - return presets; - } - - public String getHost() - { - return hosturl; - /* - * try { URL serviceurl = new URL(hosturl); if (serviceurl.getPort()!=80) - * { return serviceurl.getHost()+":"+serviceurl.getPort(); } return - * serviceurl.getHost(); } catch (Exception e) { - * System.err.println("Failed to parse service URL '" + hosturl + - * "' as a valid URL!"); } return null; - */ - } - - /** - * @return short description of what the service will do - */ - public String getActionText() - { - return "Align with " + serviceType; - } - - /** - * non-thread safe - blocks whilst accessing service to get complete set of - * available options and parameters - * - * @return - */ - public RunnerConfig getRunnerConfig() + service.hasParameters(); + if (validServiceUrls == null) { - return service.getRunnerOptions(); + validServiceUrls = new Vector<>(); } + validServiceUrls.add(jwsservers); + } - @Override - protected void finalize() throws Throwable + /** + * + * @param args + * @j2sIgnore + */ + public static void main(String[] args) + { + if (args.length > 0) { - if (service != null) + testUrls = new ArrayList<>(); + for (String url : args) { - try - { - Closeable svc = (Closeable) service; - service = null; - svc.close(); - } catch (Exception e) - { - } - ; + testUrls.add(url); } - super.finalize(); } - - public ParamDatastoreI getParamStore() - { - if (paramStore == null) + var discoverer = getDiscoverer(); + discoverer.addServiceChangeListener((_discoverer, _services) -> { + if (discoverer.services != null) { - try - { - paramStore = new JabaParamStore(this, - (Desktop.instance != null ? Desktop - .getUserParameterStore() : null)); - } catch (Exception ex) + System.out.println("Changesupport: There are now " + + discoverer.services.size() + " services"); + int i = 1; + for (ServiceWithParameters instance : discoverer.services) { + System.out.println( + "Service " + i++ + " " + instance.getClass() + + "@" + instance.getHostURL() + ": " + + instance.getActionText()); } } - return paramStore; - } - - public String getUri() - { - // this is only valid for Jaba 1.0 - this formula might have to change! - return hosturl - + (hosturl.lastIndexOf("/") == (hosturl.length() - 1) ? "" - : "/") + serviceType; - } - - private boolean hasParams = false, lookedForParams = false; - - public boolean hasParameters() - { - if (!lookedForParams) - { - lookedForParams = true; - try - { - hasParams = (getRunnerConfig().getArguments().size() > 0); - } catch (Exception e) - { - - } - } - return hasParams; - } - }; - - /** - * holds list of services. - */ - protected Vector services; - - public void attachWSMenuEntry(JMenu wsmenu, final AlignFrame alignFrame) - { - // dynamically regenerate service list. - final JMenu jws2al = wsmenu; // new JMenu("JABAWS Alignment"); - jws2al.addMenuListener(new MenuListener() - { - // TODO: future: add menu listener to parent menu - so submenus are - // populated *before* they are selected. - @Override - public void menuSelected(MenuEvent e) - { - populateWSMenuEntry(jws2al, alignFrame); - } - - @Override - public void menuDeselected(MenuEvent e) - { - // TODO Auto-generated method stub - - } - - @Override - public void menuCanceled(MenuEvent e) - { - // TODO Auto-generated method stub - - } - }); - wsmenu.add(jws2al); - - } - - private void populateWSMenuEntry(JMenu jws2al, final AlignFrame alignFrame) - { - if (running || services == null || services.size() == 0) + try { - return; - } - boolean byhost = Cache.getDefault("WSMENU_BYHOST", false), bytype = Cache - .getDefault("WSMENU_BYTYPE", false); - /** - * eventually, JWS2 services will appear under the same align/etc submenus. - * for moment we keep them separate. - */ - JMenu atpoint; - MsaWSClient msacl = new MsaWSClient(); - Vector hostLabels = new Vector(); - jws2al.removeAll(); - String lasthost = null; - Hashtable> hosts = new Hashtable>(); - String[] sorton; - for (Jws2Instance service : services) - { - ArrayList hostservices = hosts.get(service.getHost()); - if (hostservices == null) - { - hosts.put(service.getHost(), - hostservices = new ArrayList()); - } - hostservices.add(service); - } - sorton = hosts.keySet().toArray(new String[1]); - String hostlist[] = sorton.clone(); - jalview.util.QuickSort.sort(sorton, hostlist); - for (String host : hostlist) + discoverer.startDiscoverer().get(); + } catch (InterruptedException | ExecutionException e) { - Jws2Instance orderedsvcs[] = hosts.get(host).toArray( - new Jws2Instance[1]); - String sortbytype[] = new String[orderedsvcs.length]; - for (int i = 0; i < sortbytype.length; i++) - { - sortbytype[i] = orderedsvcs[i].serviceType; - } - jalview.util.QuickSort.sort(sortbytype, orderedsvcs); - for (final Jws2Instance service : orderedsvcs) - { - atpoint = jws2al; - String type = service.serviceType; - if (byhost) - { - atpoint = JvSwingUtils.findOrCreateMenu(atpoint, host); - if (atpoint.getToolTipText() == null) - { - atpoint.setToolTipText("Services at " + host); - } - } - if (bytype) - { - atpoint = JvSwingUtils.findOrCreateMenu(atpoint, type); - if (atpoint.getToolTipText() == null) - { - atpoint.setToolTipText(service.getActionText()); - } - } - if (!byhost - && !hostLabels.contains(host + service.serviceType - + service.getActionText())) - // !hostLabels.contains(host + (bytype ? - // service.serviceType+service.getActionText() : ""))) - { - // add a marker indicating where this service is hosted - // relies on services from the same host being listed in a - // contiguous - // group - JMenuItem hitm; - atpoint.addSeparator(); - if (lasthost == null || !lasthost.equals(host)) - { - atpoint.add(hitm = new JMenuItem(host)); - hitm.setForeground(Color.blue); - hitm.addActionListener(new ActionListener() - { - - @Override - public void actionPerformed(ActionEvent e) - { - Desktop.showUrl(service.getHost()); - } - }); - hitm.setToolTipText(JvSwingUtils - .wrapTooltip("Opens the JABAWS server's homepage in web browser")); - lasthost = host; - } - hostLabels.addElement(host + service.serviceType - + service.getActionText()); - // hostLabels.addElement(host + (bytype ? - // service.serviceType+service.getActionText() : "")); - } - msacl.attachWSMenuEntry(atpoint, service, alignFrame); - /* - * JMenuItem sitem = new JMenuItem(service.serviceType); - * sitem.setToolTipText("Hosted at " + service.hosturl); - * sitem.addActionListener(new ActionListener() { - * - * @Override public void actionPerformed(ActionEvent e) { AlignmentView - * msa = alignFrame.gatherSequencesForAlignment(); MsaWSClient client = - * new MsaWSClient(service, "JWS2 Alignment of " + - * alignFrame.getTitle(), msa, false, true, - * alignFrame.getViewport().getAlignment().getDataset(), alignFrame); } - * }); - */ - } } - - } - - public static void main(String[] args) - { - Thread runner = getDiscoverer().startDiscoverer( - new PropertyChangeListener() - { - - public void propertyChange(PropertyChangeEvent evt) - { - if (getDiscoverer().services != null) - { - System.out.println("Changesupport: There are now " - + getDiscoverer().services.size() + " services"); - - } - } - }); - while (runner.isAlive()) + try + { + Thread.sleep(50); + } catch (InterruptedException x) { - try - { - Thread.sleep(50); - } catch (InterruptedException e) - { - } - ; } } - private static Jws2Discoverer discoverer; - + /** + * Returns the singleton instance of this class. + * + * @return + */ public static Jws2Discoverer getDiscoverer() { if (discoverer == null) @@ -554,35 +383,32 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI return discoverer; } + @Override public boolean hasServices() { - // TODO Auto-generated method stub return !running && services != null && services.size() > 0; } + @Override public boolean isRunning() { return running; } - /** - * the jalview .properties entry for JWS2 URLS - */ - final static String JWS2HOSTURLS = "JWS2HOSTURLS"; - - public static void setServiceUrls(Vector urls) + @Override + public void setServiceUrls(List wsUrls) { - if (urls != null) + if (wsUrls != null && !wsUrls.isEmpty()) { - StringBuffer urlbuffer = new StringBuffer(); + StringBuilder urls = new StringBuilder(128); String sep = ""; - for (String url : urls) + for (String url : wsUrls) { - urlbuffer.append(sep); - urlbuffer.append(url); + urls.append(sep); + urls.append(url); sep = ","; } - Cache.setProperty(JWS2HOSTURLS, urlbuffer.toString()); + Cache.setProperty(JWS2HOSTURLS, urls.toString()); } else { @@ -590,11 +416,28 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI } } - public static Vector getServiceUrls() + /** + * Returns web service URLs, in the order in which they should be tried (or an + * empty list). + * + * @return + */ + @Override + public List getServiceUrls() { - String surls = Cache.getDefault(JWS2HOSTURLS, - "http://www.compbio.dundee.ac.uk/jabaws"); - Vector urls = new Vector(); + if (testUrls != null) + { + // return test urls, if there are any, instead of touching cache + return testUrls; + } + List urls = new ArrayList<>(); + + if (this.preferredUrl != null) + { + urls.add(preferredUrl); + } + + String surls = Cache.getDefault(JWS2HOSTURLS, COMPBIO_JABAWS); try { StringTokenizer st = new StringTokenizer(surls, ","); @@ -603,46 +446,40 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI String url = null; try { - java.net.URL u = new java.net.URL(url = st.nextToken()); + url = st.nextToken(); + new URL(url); if (!urls.contains(url)) { urls.add(url); } else { - jalview.bin.Cache.log.info("Ignoring duplicate url in " + Cache.log.warn("Ignoring duplicate url " + url + " in " + JWS2HOSTURLS + " list"); } - } catch (Exception ex) + } catch (MalformedURLException ex) { - jalview.bin.Cache.log - .warn("Problem whilst trying to make a URL from '" - + ((url != null) ? url : "") + "'"); - jalview.bin.Cache.log - .warn("This was probably due to a malformed comma separated list" - + " in the " - + JWS2HOSTURLS + Cache.log.warn("Problem whilst trying to make a URL from '" + + ((url != null) ? url : "") + "'"); + Cache.log.warn( + "This was probably due to a malformed comma separated list" + + " in the " + JWS2HOSTURLS + " entry of $(HOME)/.jalview_properties)"); - jalview.bin.Cache.log.debug("Exception was ", ex); + Cache.log.debug("Exception was ", ex); } } } catch (Exception ex) { - jalview.bin.Cache.log.warn( - "Error parsing comma separated list of urls in " - + JWS2HOSTURLS + " preference.", ex); - } - if (urls.size() >= 0) - { - return urls; + Cache.log.warn("Error parsing comma separated list of urls in " + + JWS2HOSTURLS + " preference.", ex); } - return null; + return urls; } - public Vector getServices() + @Override + public Vector getServices() { - return (services == null) ? new Vector() - : new Vector(services); + return (services == null) ? new Vector<>() : new Vector<>(services); } /** @@ -651,12 +488,14 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI * @param foo * @return */ - public static boolean testServiceUrl(URL foo) + @Override + public boolean testServiceUrl(URL foo) { try { - compbio.ws.client.WSTester.main(new String[] - { "-h=" + foo.toString() }); + compbio.ws.client.WSTester + .main(new String[] + { "-h=" + foo.toString() }); } catch (Exception e) { e.printStackTrace(); @@ -674,6 +513,22 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI return true; } + public boolean restart() + { + synchronized (this) + { + if (running) + { + aborted = true; + } + else + { + running = true; + } + return aborted; + } + } + /** * Start a fresh discovery thread and notify the given object when we're * finished. Any known existing threads will be killed before this one is @@ -682,20 +537,28 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI * @param changeSupport2 * @return new thread */ - public Thread startDiscoverer(PropertyChangeListener changeSupport2) + @Override + public FutureTask startDiscoverer() { + /* if (restart()) + { + return; + } + else + { + Thread thr = new Thread(this); + thr.start(); + } + */ if (isRunning()) { setAborted(true); } - addPropertyChangeListener(changeSupport2); - Thread thr = new Thread(this); - thr.start(); - return thr; + FutureTask task = new FutureTask<>(this, this); + new Thread(task).start(); + return task; } - Vector invalidServiceUrls = null, urlsWithoutServices = null; - /** * @return the invalidServiceUrls */ @@ -722,11 +585,11 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI { if (urlsWithoutServices == null) { - urlsWithoutServices = new Vector(); + urlsWithoutServices = new Vector<>(); } - if ((invalidServiceUrls == null || !invalidServiceUrls - .contains(jwsservers)) + if ((invalidServiceUrls == null + || !invalidServiceUrls.contains(jwsservers)) && !urlsWithoutServices.contains(jwsservers)) { urlsWithoutServices.add(jwsservers); @@ -742,7 +605,7 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI { if (invalidServiceUrls == null) { - invalidServiceUrls = new Vector(); + invalidServiceUrls = new Vector<>(); } if (!invalidServiceUrls.contains(jwsservers)) { @@ -755,6 +618,7 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI * @return a human readable report of any problems with the service URLs used * for discovery */ + @Override public String getErrorMessages() { if (!isRunning() && !isAborted()) @@ -764,7 +628,8 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI if (getInvalidServiceUrls() != null && getInvalidServiceUrls().size() > 0) { - ermsg.append("URLs that could not be contacted: \n"); + ermsg.append(MessageManager.getString("warn.urls_not_contacted") + + ": \n"); for (String svcurl : getInvalidServiceUrls()) { if (list) @@ -780,7 +645,8 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI if (getUrlsWithoutServices() != null && getUrlsWithoutServices().size() > 0) { - ermsg.append("URLs without any JABA Services : \n"); + ermsg.append( + MessageManager.getString("warn.urls_no_jaba") + ": \n"); for (String svcurl : getUrlsWithoutServices()) { if (list) @@ -801,4 +667,38 @@ public class Jws2Discoverer implements Runnable, WSMenuEntryProviderI return null; } + @Override + public int getServerStatusFor(String url) + { + if (validServiceUrls != null && validServiceUrls.contains(url)) + { + return STATUS_OK; + } + if (urlsWithoutServices != null && urlsWithoutServices.contains(url)) + { + return STATUS_NO_SERVICES; + } + if (invalidServiceUrls != null && invalidServiceUrls.contains(url)) + { + return STATUS_INVALID; + } + return STATUS_UNKNOWN; + } + + /** + * Set a URL to try before any others. For use with command-line parameter to + * configure a local Jabaws installation without the need to add to property + * files. + * + * @param value + * @throws MalformedURLException + */ + public void setPreferredUrl(String value) throws MalformedURLException + { + if (value != null && value.trim().length() > 0) + { + new URL(value); + preferredUrl = value; + } + } }