todo for refactoring webservice GUI creation mechanism

diff --git a/src/jalview/gui/AlignFrame.java b/src/jalview/gui/AlignFrame.java
index 72691cc..6e7b15d 100755 (executable)
--- a/src/jalview/gui/AlignFrame.java
+++ b/src/jalview/gui/AlignFrame.java
@@ -3445,15 +3445,16 @@ public class AlignFrame
   public void BuildWebServiceMenu()
   {
     // TODO: add support for context dependent disabling of services based on alignment and current selection
-    // TODO: refactor to allow list of AbstractName/Handler bindings to be stored or retrieved from elsewhere
     // TODO: add additional serviceHandle parameter to specify abstract handler class independently of AbstractName
     // TODO: add in rediscovery GUI function to restart discoverer
     // TODO: group services by location as well as function and/or introduce object broker mechanism.
     if ( (Discoverer.services != null)
         && (Discoverer.services.size() > 0))
     {
+      // TODO: refactor to allow list of AbstractName/Handler bindings to be stored or retrieved from elsewhere
       Vector msaws = (Vector) Discoverer.services.get("MsaWS");
       Vector secstrpr = (Vector) Discoverer.services.get("SecStrPred");
+      // TODO: move GUI generation code onto service implementation - so a client instance attaches itself to the GUI with method call like jalview.ws.MsaWSClient.bind(servicehandle, Desktop.instance, alignframe)
       Vector wsmenu = new Vector();
       final AlignFrame af = this;
       if (msaws != null)
@@ -3480,6 +3481,7 @@ public class AlignFrame
           });
           msawsmenu.add(method);
           // Deal with services that we know accept partial alignments.
+          // TODO: this should be a service property - ie sh.getProperty("submitGaps")==Boolean.true
           if (sh.getName().indexOf("lustal") > -1)
           {
             // We know that ClustalWS can accept partial alignments for refinement.