svn commit: r659800 - in /ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse: CHANGES.txt src/java/org/apache/ivyde/eclipse/cpcontainer/IvyResolveJob.java

[hi...@apache.org]

Author: hibou
Date: Sat May 24 04:58:41 2008
New Revision: 659800

URL: http://svn.apache.org/viewvc?rev=659800&view=rev
Log:
IVYDE-95: The dialog box opened for resolve failure should show the status message on several lines
 - use of Eclipse's multi status

Modified:
    ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse/CHANGES.txt
    ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse/src/java/org/apache/ivyde/eclipse/cpcontainer/IvyResolveJob.java

Modified: ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse/CHANGES.txt
URL: http://svn.apache.org/viewvc/ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse/CHANGES.txt?rev=659800&r1=659799&r2=659800&view=diff
==============================================================================
--- ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse/CHANGES.txt (original)
+++ ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse/CHANGES.txt Sat May 24 04:58:41 2008
@@ -6,6 +6,7 @@
 - NEW: [build] Use the latest ivy to resolve dependencies (IVYDE-62) (thanks to Nicolas Lalevée) 
 - NEW: Order alphabetically the entries in the class path container (IVYDE-69) (thanks to Nicolas Lalevée)
 
+- IMPROVE: The dialog box opened for resolve failure should show the status message on several lines (IVYDE-95)
 - IMPROVE: Recognize more source and javadoc artifacts names (IVYDE-66) (thanks to Nicolas Lalevée)
 - IMPROVE: Use Ivy Bundle instead of packaging ivy.jar in IvyDE bundle (IVYDE-67)
 - IMPROVE: enable 'Resolve all' action (IVYDE-42) (thanks to Thomas FRIOL)

Modified: ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse/src/java/org/apache/ivyde/eclipse/cpcontainer/IvyResolveJob.java
URL: http://svn.apache.org/viewvc/ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse/src/java/org/apache/ivyde/eclipse/cpcontainer/IvyResolveJob.java?rev=659800&r1=659799&r2=659800&view=diff
==============================================================================
--- ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse/src/java/org/apache/ivyde/eclipse/cpcontainer/IvyResolveJob.java (original)
+++ ant/ivy/ivyde/trunk/org.apache.ivyde.eclipse/src/java/org/apache/ivyde/eclipse/cpcontainer/IvyResolveJob.java Sat May 24 04:58:41 2008
@@ -60,6 +60,7 @@
 import org.eclipse.core.runtime.IPath;
 import org.eclipse.core.runtime.IProgressMonitor;
 import org.eclipse.core.runtime.IStatus;
+import org.eclipse.core.runtime.MultiStatus;
 import org.eclipse.core.runtime.Path;
 import org.eclipse.core.runtime.Status;
 import org.eclipse.core.runtime.SubProgressMonitor;
@@ -274,15 +275,14 @@
                 }
 
                 if (!problemMessages.isEmpty()) {
-                    StringBuffer problems = new StringBuffer();
+                    MultiStatus multiStatus = new MultiStatus(IvyPlugin.ID, IStatus.ERROR,
+                            "Impossible to resolve dependencies of " + md.getModuleRevisionId(),
+                            null);
                     for (Iterator iter = problemMessages.iterator(); iter.hasNext();) {
-                        String msg = (String) iter.next();
-                        problems.append(msg).append("\n");
+                        multiStatus.add(new Status(IStatus.ERROR, IvyPlugin.ID, (String) iter
+                                .next()));
                     }
-                    status[0] = new Status(IStatus.ERROR, IvyPlugin.ID, IStatus.ERROR,
-                            "Impossible to resolve dependencies of " + md.getModuleRevisionId()
-                                    + ":\n" + problems + "\nSee IvyConsole for further details",
-                            null);
+                    status[0] = multiStatus;
                     return;
                 }