[Openutils-svn] SF.net SVN: openutils: [350] trunk/openutils-dbmigration/src/main/java/it/ openutils/migration/task/setup/GenericScriptBasedConditionalTask.java

[<fg...@us...>]

Revision: 350
          http://svn.sourceforge.net/openutils/?rev=350&view=rev
Author:   fgiust
Date:     2007-07-03 14:47:37 -0700 (Tue, 03 Jul 2007)

Log Message:
-----------
fix getDescription()

Modified Paths:
--------------
    trunk/openutils-dbmigration/src/main/java/it/openutils/migration/task/setup/GenericScriptBasedConditionalTask.java

Modified: trunk/openutils-dbmigration/src/main/java/it/openutils/migration/task/setup/GenericScriptBasedConditionalTask.java
===================================================================
--- trunk/openutils-dbmigration/src/main/java/it/openutils/migration/task/setup/GenericScriptBasedConditionalTask.java	2007-07-03 07:58:42 UTC (rev 349)
+++ trunk/openutils-dbmigration/src/main/java/it/openutils/migration/task/setup/GenericScriptBasedConditionalTask.java	2007-07-03 21:47:37 UTC (rev 350)
@@ -68,8 +68,22 @@
      */
     public String getDescription()
     {
-        // @todo add better description and avoid NPEs
-        return "Checking " + objectNameFromFileName(scripts.get(0));
+        if (scripts == null)
+        {
+            return "Nothing to do, no scripts configured";
+        }
+
+        StringBuffer result = new StringBuffer();
+        if (!scripts.isEmpty())
+        {
+            result.append("Checking :\n");
+            for (Resource script : scripts)
+            {
+                result.append("  - " + objectNameFromFileName(script) + "\n");
+            }
+
+        }
+        return result.toString();
     }
 
     /**


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.