[jbpm-commits] JBoss JBPM SVN: r3014 - in projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client: process and 1 other directory.

Author: heiko.braun(a)jboss.com Date: 2008-11-20 09:29:46 -0500 (Thu, 20 Nov 2008) New Revision: 3014 Modified: projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/MainView.java projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/process/ProcessDefinitionList.java projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/process/ProcessInstanceList.java Log: Cleanup logging Modified: projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/MainView.java =================================================================== --- projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/MainView.java 2008-11-20 14:24:56 UTC (rev 3013) +++ projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/MainView.java 2008-11-20 14:29:46 UTC (rev 3014) @@ -21,6 +21,7 @@ */ package org.jboss.bpm.console.client; +import com.allen_sauer.gwt.log.client.Log; import com.google.gwt.core.client.GWT; import com.google.gwt.http.client.Request; import com.google.gwt.http.client.Response; @@ -37,7 +38,6 @@ import com.gwtext.client.widgets.layout.BorderLayoutData; import com.gwtext.client.widgets.layout.FitLayout; import com.gwtext.client.widgets.layout.VerticalLayout; -import com.allen_sauer.gwt.log.client.Log; import org.jboss.bpm.console.client.metric.MetricOverviewEditor; import org.jboss.bpm.console.client.process.ProcessDefinitionListEditor; import org.jboss.bpm.console.client.task.TaskListEditor; @@ -225,18 +225,13 @@ return this.urlBuilder; } - public void setError(final String error) - { - displayMessage(error, true); - } - public void displayMessage(final String message, final boolean isError) { status.setText(message); MessageBox.show(new MessageBoxConfig() { { - String title = isError ? "Unknown error" : "System Message"; + String title = isError ? "Error" : "System Message"; setTitle(title); setMsg(message); setButtons(MessageBox.OK); Modified: projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/process/ProcessDefinitionList.java =================================================================== --- projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/process/ProcessDefinitionList.java 2008-11-20 14:24:56 UTC (rev 3013) +++ projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/process/ProcessDefinitionList.java 2008-11-20 14:29:46 UTC (rev 3014) @@ -131,8 +131,7 @@ } catch (RequestException e1) { - view.setError(e1.getMessage()); - e1.printStackTrace(); + Log.error("Request failed", e1); } } } Modified: projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/process/ProcessInstanceList.java =================================================================== --- projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/process/ProcessInstanceList.java 2008-11-20 14:24:56 UTC (rev 3013) +++ projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/process/ProcessInstanceList.java 2008-11-20 14:29:46 UTC (rev 3014) @@ -101,14 +101,13 @@ public void onError(Request request, Throwable t) { // auth failed - Log.error("Failed to remove process instance: "+t.getMessage()); + Log.error("Failed to remove process instance", t); } }); } catch (RequestException e1) { - view.setError(e1.getMessage()); - e1.printStackTrace(); + Log.error("Request failed", e1); } } } @@ -153,8 +152,7 @@ } catch (RequestException e1) { - view.setError(e1.getMessage()); - e1.printStackTrace(); + Log.error("Request failed", e1); } } }