[wise-commits] wise SVN: r474 - in webgui/branches/cdi-jsf/src/main: webapp and 1 other directory.

[wise-commits at lists.jboss.org]

Author: alessio.soldano at jboss.com
Date: 2013-01-29 12:26:12 -0500 (Tue, 29 Jan 2013)
New Revision: 474

Modified:
   webgui/branches/cdi-jsf/src/main/java/org/jboss/wise/gui/ClientConversationBean.java
   webgui/branches/cdi-jsf/src/main/webapp/index.xhtml
Log:
Initial error message handling


Modified: webgui/branches/cdi-jsf/src/main/java/org/jboss/wise/gui/ClientConversationBean.java
===================================================================
--- webgui/branches/cdi-jsf/src/main/java/org/jboss/wise/gui/ClientConversationBean.java	2013-01-29 16:04:52 UTC (rev 473)
+++ webgui/branches/cdi-jsf/src/main/java/org/jboss/wise/gui/ClientConversationBean.java	2013-01-29 17:26:12 UTC (rev 474)
@@ -43,6 +43,8 @@
 import org.jboss.wise.core.client.WebParameter;
 import org.jboss.wise.core.client.builder.WSDynamicClientBuilder;
 import org.jboss.wise.core.client.factories.WSDynamicClientFactory;
+import org.jboss.wise.core.exception.InvocationException;
+import org.jboss.wise.core.exception.WiseRuntimeException;
 import org.jboss.wise.gui.treeElement.GroupWiseTreeElement;
 import org.jboss.wise.gui.treeElement.LazyLoadWiseTreeElement;
 import org.jboss.wise.gui.treeElement.WiseTreeElement;
@@ -68,6 +70,7 @@
     private String currentOperation;
     private TreeNodeImpl inputTree;
     private TreeNodeImpl outputTree;
+    private String error;
     private UITree inTree;
     
     @PostConstruct
@@ -77,24 +80,37 @@
 	conversation.setTimeout(CONVERSATION_TIMEOUT);
     }
     
-    public void readWsdl() throws ConnectException {
+    public void readWsdl() {
 	cleanup();
 	//restart conversation
 	conversation.end();
 	conversation.begin();
-	client = getClientBuilder().verbose(true).keepSource(true).wsdlURL(getWsdlUrl()).maxThreadPoolSize(1).build();
-	cleanupTask.addRef(client, System.currentTimeMillis() + CONVERSATION_TIMEOUT, new CleanupTask.CleanupCallback<WSDynamicClient>() {
-	    @Override
-	    public void cleanup(WSDynamicClient data) {
-		data.close();
+	try {
+	    client = getClientBuilder().verbose(true).keepSource(true).wsdlURL(getWsdlUrl()).maxThreadPoolSize(1).build();
+	    cleanupTask.addRef(client, System.currentTimeMillis() + CONVERSATION_TIMEOUT, new CleanupTask.CleanupCallback<WSDynamicClient>() {
+		@Override
+		public void cleanup(WSDynamicClient data) {
+		    data.close();
+		}
+	    });
+	} catch (Exception e) {
+	    error = "Could not read WSDL from specified URL. Please check logs for further information.";
+	    logException(e);
+	}
+	if (client != null) {
+	    try {
+		services = convertServicesToGui(client.processServices());
+	    } catch (Exception e) {
+		error = "Could not parse WSDL from specified URL. Please check logs for further information.";
+		logException(e);
 	    }
-	});
-	
-	services = convertServicesToGui(client.processServices());
+	}
     }
     
     public void parseOperationParameters() {
 	if (currentOperation == null) return;
+	outputTree = null;
+	error = null;
 	StringTokenizer st = new StringTokenizer(currentOperation, ";");
 	String serviceName = st.nextToken();
 	String portName = st.nextToken();
@@ -102,12 +118,14 @@
 	try {
 	    inputTree = convertOperationParametersToGui(client.getWSMethod(serviceName, portName, operationName), client);
 	} catch (Exception e) {
-	    throw new RuntimeException(e);
+	    error = toErrorMessage(e);
+	    logException(e);
 	}
-	outputTree = null;
     }
     
     public void performInvocation() {
+	outputTree = null;
+	error = null;
 	StringTokenizer st = new StringTokenizer(currentOperation, ";");
 	String serviceName = st.nextToken();
 	String portName = st.nextToken();
@@ -119,9 +137,20 @@
 		WiseTreeElement wte = (WiseTreeElement)inputTree.getChild(it.next());
 		params.put(wte.getName(), wte.isNil() ? null : wte.toObject());
 	    }
-	    outputTree = convertOperationResultToGui(wsMethod.invoke(params), client);
+	    InvocationResult result = null;
+	    try {
+		result = wsMethod.invoke(params);
+	    } catch (InvocationException e) {
+		logException(e);
+		error = "Unexpected fault / error received from target endpoint";
+	    }
+	    if (result != null) {
+		outputTree = convertOperationResultToGui(result, client);
+		error = null;
+	    }
 	} catch (Exception e) {
-	    throw new RuntimeException(e);
+	    error = toErrorMessage(e);
+	    logException(e);
 	}
     }
     
@@ -137,7 +166,8 @@
 	try {
 	    el.resolveReference();
 	} catch (Exception e) {
-	    throw new RuntimeException(e);
+	    error = toErrorMessage(e);
+	    logException(e);
 	}
     }
     
@@ -231,6 +261,7 @@
 	    inTree.clearInitialState();
 	}
 	inputTree = null;
+	error = null;
     }
     
     public String getWsdlUrl() {
@@ -280,11 +311,38 @@
     public void setOutputTree(TreeNodeImpl outputTree) {
         this.outputTree = outputTree;
     }
+    
+    public String getError() {
+        return error;
+    }
 
+    public void setError(String error) {
+        this.error = error;
+    }
+
     private static synchronized WSDynamicClientBuilder getClientBuilder() {
 	if (clientBuilder == null) {
 	    clientBuilder = WSDynamicClientFactory.getJAXWSClientBuilder();
 	}
 	return clientBuilder;
     }
+    
+    private static String toErrorMessage(Exception e) {
+	StringBuilder sb = new StringBuilder();
+	if (e instanceof WiseRuntimeException) {
+	    sb.append(e.getMessage());
+	} else {
+	    sb.append(e.toString());
+	}
+	if (e.getCause() != null) {
+	    sb.append(", caused by ");
+	    sb.append(e.getCause());
+	}
+	sb.append(". Please check logs for further information.");
+	return sb.toString();
+    }
+    
+    private static void logException(Exception e) {
+	e.printStackTrace(); //TODO!!
+    }
 }

Modified: webgui/branches/cdi-jsf/src/main/webapp/index.xhtml
===================================================================
--- webgui/branches/cdi-jsf/src/main/webapp/index.xhtml	2013-01-29 16:04:52 UTC (rev 473)
+++ webgui/branches/cdi-jsf/src/main/webapp/index.xhtml	2013-01-29 17:26:12 UTC (rev 474)
@@ -215,6 +215,9 @@
 		        	</rich:tree>
                 </h:form>
             </rich:panel>
+            <rich:panel header="Error" rendered="#{not empty clientConversationBean.error}">
+               <h:outputText value="#{clientConversationBean.error}" />
+            </rich:panel>
             </a4j:outputPanel>
             
             </a4j:outputPanel>