You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ace.apache.org by ja...@apache.org on 2012/04/10 11:55:35 UTC
svn commit: r1311660 -
/ace/trunk/ace-webui-vaadin/src/main/java/org/apache/ace/webui/vaadin/AddArtifactWindow.java
Author: jawi
Date: Tue Apr 10 09:55:34 2012
New Revision: 1311660
URL: http://svn.apache.org/viewvc?rev=1311660&view=rev
Log:
ACE-212: notify user in case of errors during adding of artifacts did not work correctly.
Modified:
ace/trunk/ace-webui-vaadin/src/main/java/org/apache/ace/webui/vaadin/AddArtifactWindow.java
Modified: ace/trunk/ace-webui-vaadin/src/main/java/org/apache/ace/webui/vaadin/AddArtifactWindow.java
URL: http://svn.apache.org/viewvc/ace/trunk/ace-webui-vaadin/src/main/java/org/apache/ace/webui/vaadin/AddArtifactWindow.java?rev=1311660&r1=1311659&r2=1311660&view=diff
==============================================================================
--- ace/trunk/ace-webui-vaadin/src/main/java/org/apache/ace/webui/vaadin/AddArtifactWindow.java (original)
+++ ace/trunk/ace-webui-vaadin/src/main/java/org/apache/ace/webui/vaadin/AddArtifactWindow.java Tue Apr 10 09:55:34 2012
@@ -495,9 +495,9 @@ abstract class AddArtifactWindow extends
added.add(importRemoteBundle(artifact.toURI().toURL()));
}
catch (Exception exception) {
- showErrorNotification("Import artifact failed", "Artifact "
- + artifact.getAbsolutePath()
- + "<br />could not be imported into the repository.<br />"
+ showErrorNotification("Import artifact failed", "<br/>Artifact '"
+ + artifact.getName()
+ + "' could not be imported into the repository.<br />"
+ "Reason: " + exception.getMessage());
logError("Import of " + artifact.getAbsolutePath() + " failed.", exception);
@@ -517,7 +517,7 @@ abstract class AddArtifactWindow extends
* @param aMessage the error message itself.
*/
final void showErrorNotification(final String aTitle, final String aMessage) {
- showNotification(aTitle, aMessage, Notification.TYPE_ERROR_MESSAGE);
+ getParent().showNotification(aTitle, aMessage, Notification.TYPE_ERROR_MESSAGE);
}
/**