X-Git-Url: http://juplo.de/gitweb/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fde%2Fjuplo%2Fplugins%2Fhibernate%2FAbstractSchemaMojo.java;h=edf8d125f58277b09d9b527702b1e9512381f644;hb=5504979ec926b927755a3e4bfd93a406bb4d07b9;hp=20c326aa036395c88b69fbd4261cb5912a27d6c3;hpb=2bc469097f6ff01d83f7a361d2c33e2a3fafef1c;p=hibernate4-maven-plugin

diff --git a/src/main/java/de/juplo/plugins/hibernate/AbstractSchemaMojo.java b/src/main/java/de/juplo/plugins/hibernate/AbstractSchemaMojo.java
index 20c326aa..edf8d125 100644
--- a/src/main/java/de/juplo/plugins/hibernate/AbstractSchemaMojo.java
+++ b/src/main/java/de/juplo/plugins/hibernate/AbstractSchemaMojo.java
@@ -490,8 +490,7 @@ public abstract class AbstractSchemaMojo extends AbstractMojo
       LoadedConfig config = loadConfig(configLoader);
       if (config != null)
         properties.putAll(config.getConfigurationValues());
-      ParsedPersistenceXmlDescriptor unit =
-          loadPersistenceUnit(classLoaderService, properties);
+      ParsedPersistenceXmlDescriptor unit = loadPersistenceUnit(properties);
       if (unit != null)
         properties.putAll(unit.getProperties());
 
@@ -733,12 +732,25 @@ public abstract class AbstractSchemaMojo extends AbstractMojo
       {
         thread.setContextClassLoader(classLoader);
         build((MetadataImplementor)metadataBuilder.build(), options, target);
+        if (handler.getExceptions().size() > 0)
+        {
+          StringBuilder builder = new StringBuilder();
+          builder.append("Hibernate failed:");
+          for (Exception e : handler.getExceptions())
+          {
+            builder.append("\n * ");
+            builder.append(e.getMessage());
+            AbstractSchemaMojo.printStrackTrace(builder, e);
+            builder.append("\n");
+          }
+          String error = builder.toString();
+          getLog().error(error);
+          throw new MojoFailureException(error);
+        }
       }
       finally
       {
         thread.setContextClassLoader(contextClassLoader);
-        for (Exception e : handler.getExceptions())
-          getLog().error(e.getMessage());
         /** Track, the content of the generated script */
         checkOutputFile(output, tracker);
       }
@@ -1168,12 +1180,18 @@ public abstract class AbstractSchemaMojo extends AbstractMojo
   {
     try
     {
-      File dir = new File(outputDirectory);
+      File dir = new File(path);
       if (dir.exists())
       {
         getLog().info("Adding " + dir.getAbsolutePath() + " to the list of roots to scan...");
         urls.add(dir.toURI().toURL());
       }
+      else
+        getLog().warn(
+            "the directory cannot be scanned for annotated classes, " +
+            "because it does not exist: " +
+            dir.getAbsolutePath()
+            );
     }
     catch (MalformedURLException e)
     {
@@ -1327,55 +1345,73 @@ public abstract class AbstractSchemaMojo extends AbstractMojo
   }
 
   private ParsedPersistenceXmlDescriptor loadPersistenceUnit(
-      ClassLoaderService classLoaderService,
       Properties properties
       )
       throws
         MojoFailureException
   {
-    PersistenceXmlParser parser =
-        new PersistenceXmlParser(
-            classLoaderService,
-            PersistenceUnitTransactionType.RESOURCE_LOCAL
-             );
-
-    Map<String, ParsedPersistenceXmlDescriptor> units =
-        parser.doResolve(properties);
+    List<ParsedPersistenceXmlDescriptor> units =
+        PersistenceXmlParser.locatePersistenceUnits(properties);
 
     if (persistenceUnit == null)
     {
-      Iterator<String> names = units.keySet().iterator();
-      if (!names.hasNext())
+      if (units.isEmpty())
       {
         getLog().info("Found no META-INF/persistence.xml.");
         return null;
       }
 
-      String name = names.next();
-      if (!names.hasNext())
+      if (units.size() == 1)
       {
-          getLog().info("Using persistence-unit " + name);
-          return units.get(name);
+        ParsedPersistenceXmlDescriptor descriptor = units.get(0);
+        getLog().info("Using persistence-unit " + descriptor.getName());
+        return descriptor;
       }
 
       StringBuilder builder = new StringBuilder();
       builder.append("No name provided and multiple persistence units found: ");
-      builder.append(name);
-      while(names.hasNext())
+      Iterator<ParsedPersistenceXmlDescriptor> it = units.iterator();
+      while(it.hasNext())
       {
-        builder.append(", ");
-        builder.append(names.next());
+        builder.append(it.next().getName());
+        if (it.hasNext())
+          builder.append(", ");
       }
       builder.append('.');
       throw new MojoFailureException(builder.toString());
     }
 
-    if (units.containsKey(persistenceUnit))
+    Iterator<ParsedPersistenceXmlDescriptor> it = units.iterator();
+    while(it.hasNext())
     {
-      getLog().info("Using configured persistence-unit " + persistenceUnit);
-      return units.get(persistenceUnit);
+      ParsedPersistenceXmlDescriptor descriptor =  it.next();
+      if (descriptor.getName().equals(persistenceUnit))
+      {
+        getLog().info("Using configured persistence-unit " + persistenceUnit);
+        return descriptor;
+      }
     }
 
     throw new MojoFailureException("Could not find persistence-unit " + persistenceUnit);
   }
+
+
+  public static void printStrackTrace(StringBuilder builder, Throwable t)
+  {
+    while (t != null)
+    {
+      builder.append("\n\tCause: ");
+      builder.append(t.getMessage() == null ? "" : t.getMessage().replaceAll("\\s+", " "));
+      for (StackTraceElement trace : t.getStackTrace())
+      {
+        builder.append("\n\t");
+        builder.append(trace.getClassName());
+        builder.append(".");
+        builder.append(trace.getMethodName());
+        builder.append("():");
+        builder.append(trace.getLineNumber());
+      }
+      t = t.getCause();
+    }
+  }
 }