X-Git-Url: https://juplo.de/gitweb/?p=hibernate4-maven-plugin;a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fde%2Fjuplo%2Fplugins%2Fhibernate4%2FHbm2DdlMojo.java;h=f8c0634424772e8d374093fbd0f6a01bbc47ea74;hp=2e232bf1cbb7080be6f917bfc85ae2296e865acb;hb=7e3e9b90d61b077e48b59fc0eb63059886c68cf5;hpb=d38b2386641c7ca00f54d69cb3f576c20b0cdccc diff --git a/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java b/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java index 2e232bf1..f8c06344 100644 --- a/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java +++ b/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java @@ -30,16 +30,16 @@ import java.net.URL; import java.net.URLClassLoader; import java.security.MessageDigest; import java.security.NoSuchAlgorithmException; -import java.sql.Connection; import java.sql.Driver; -import java.sql.DriverManager; import java.sql.DriverPropertyInfo; import java.sql.SQLException; import java.sql.SQLFeatureNotSupportedException; +import java.util.Collections; import java.util.Comparator; -import java.util.Enumeration; import java.util.HashMap; import java.util.HashSet; +import java.util.LinkedHashSet; +import java.util.LinkedList; import java.util.List; import java.util.Map; import java.util.Map.Entry; @@ -47,17 +47,29 @@ import java.util.Properties; import java.util.Set; import java.util.TreeSet; import java.util.logging.Logger; +import java.util.regex.Matcher; +import java.util.regex.Pattern; import javax.persistence.Embeddable; import javax.persistence.Entity; import javax.persistence.MappedSuperclass; +import javax.persistence.spi.PersistenceUnitTransactionType; +import org.apache.maven.artifact.Artifact; import org.apache.maven.model.Resource; import org.apache.maven.plugin.AbstractMojo; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugin.MojoFailureException; import org.apache.maven.project.MavenProject; -import org.hibernate.cfg.Configuration; +import org.hibernate.boot.registry.StandardServiceRegistryBuilder; +import org.hibernate.boot.registry.classloading.spi.ClassLoaderService; +import org.hibernate.boot.registry.classloading.spi.ClassLoadingException; +import org.hibernate.boot.registry.internal.StandardServiceRegistryImpl; +import org.hibernate.cfg.Environment; import org.hibernate.cfg.NamingStrategy; -import org.hibernate.envers.configuration.AuditConfiguration; +import org.hibernate.envers.configuration.spi.AuditConfiguration; +import org.hibernate.internal.util.config.ConfigurationHelper; +import org.hibernate.jpa.boot.internal.ParsedPersistenceXmlDescriptor; +import org.hibernate.jpa.boot.internal.PersistenceXmlParser; +import org.hibernate.jpa.boot.spi.ProviderChecker; import org.hibernate.tool.hbm2ddl.SchemaExport; import org.hibernate.tool.hbm2ddl.SchemaExport.Type; import org.hibernate.tool.hbm2ddl.Target; @@ -85,7 +97,15 @@ public class Hbm2DdlMojo extends AbstractMojo public final static String NAMING_STRATEGY="hibernate.ejb.naming_strategy"; public final static String ENVERS = "hibernate.export.envers"; - private final static String MD5S = "schema.md5s"; + public final static String JPA_DRIVER = "javax.persistence.jdbc.driver"; + public final static String JPA_URL = "javax.persistence.jdbc.url"; + public final static String JPA_USERNAME = "javax.persistence.jdbc.user"; + public final static String JPA_PASSWORD = "javax.persistence.jdbc.password"; + + public final static String MD5S = "hibernate4-generatedschema.md5s"; + + private final static Pattern split = Pattern.compile("[^,\\s]+"); + /** * The maven project. @@ -113,22 +133,43 @@ public class Hbm2DdlMojo extends AbstractMojo * Classes-Directory to scan. *

* This parameter defaults to the maven build-output-directory for classes. - * Additonally, all dependencies are scanned for annotated classes. + * Additionally, all dependencies are scanned for annotated classes. * * @parameter property="project.build.outputDirectory" + * @since 1.0 */ private String outputDirectory; /** - * Wether to scan test-classes too, or not. + * Whether to scan test-classes too, or not. *

* If this parameter is set to true the test-classes of the * artifact will be scanned for hibernate-annotated classes additionally. * * @parameter property="hibernate.export.scan_testclasses" default-value="false" + * @since 1.0.1 */ private boolean scanTestClasses; + /** + * Dependency-Scopes, that should be scanned for annotated classes. + *

+ * By default, only dependencies in the scope compile are + * scanned for annotated classes. Multiple scopes can be seperated by + * white space or commas. + *

+ * If you do not want any dependencies to be scanned for annotated + * classes, set this parameter to none. + *

+ * The plugin does not scan for annotated classes in transitive + * dependencies. If some of your annotated classes are hidden in a + * transitive dependency, you can simply add that dependency explicitly. + * + * @parameter property="hibernate.export.scan_dependencies" default-value="compile" + * @since 1.0.3 + */ + private String scanDependencies; + /** * Test-Classes-Directory to scan. *

@@ -139,6 +180,7 @@ public class Hbm2DdlMojo extends AbstractMojo * to true! * * @parameter property="project.build.testOutputDirectory" + * @since 1.0.2 */ private String testOutputDirectory; @@ -147,13 +189,14 @@ public class Hbm2DdlMojo extends AbstractMojo *

* If set to true, the execution is skipped. *

- * A skipped excecution is signaled via the maven-property + * A skipped execution is signaled via the maven-property * ${hibernate.export.skipped}. *

- * The excecution is skipped automatically, if no modified or newly added + * The execution is skipped automatically, if no modified or newly added * annotated classes are found and the dialect was not changed. * - * @parameter property="maven.test.skip" default-value="false" + * @parameter property="hibernate.skip" default-value="${maven.test.skip}" + * @since 1.0 */ private boolean skip; @@ -166,6 +209,7 @@ public class Hbm2DdlMojo extends AbstractMojo * skip takes precedence over force. * * @parameter property="hibernate.export.force" default-value="false" + * @since 1.0 */ private boolean force; @@ -173,6 +217,7 @@ public class Hbm2DdlMojo extends AbstractMojo * SQL-Driver name. * * @parameter property="hibernate.connection.driver_class" + * @since 1.0 */ private String driverClassName; @@ -180,6 +225,7 @@ public class Hbm2DdlMojo extends AbstractMojo * Database URL. * * @parameter property="hibernate.connection.url" + * @since 1.0 */ private String url; @@ -187,6 +233,7 @@ public class Hbm2DdlMojo extends AbstractMojo * Database username * * @parameter property="hibernate.connection.username" + * @since 1.0 */ private String username; @@ -194,6 +241,7 @@ public class Hbm2DdlMojo extends AbstractMojo * Database password * * @parameter property="hibernate.connection.password" + * @since 1.0 */ private String password; @@ -201,43 +249,76 @@ public class Hbm2DdlMojo extends AbstractMojo * Hibernate dialect. * * @parameter property="hibernate.dialect" + * @since 1.0 */ private String hibernateDialect; /** * Hibernate Naming Strategy + * * @parameter property="hibernate.ejb.naming_strategy" + * @since 1.0.2 */ private String hibernateNamingStrategy; /** - * Path to Hibernate configuration file. + * Path to Hibernate properties file. + * If this parameter is not set the plugin will try to load the configuration + * from a file hibernate.properties on the classpath. The + * test-classpath takes precedence. * - * @parameter default-value="${project.build.outputDirectory}/hibernate.properties" + * @parameter + * @since 1.0 */ private String hibernateProperties; + /** + * Path to Hibernate configuration file (.cfg.xml). + * Settings in this file will overwrite settings in the properties file. + * If this parameter is not set the plugin will try to load the configuration + * from a file hibernate.cfg.xml on the classpath. The + * test-classpath takes precedence. + * + * @parameter + * @since 1.1.0 + */ + private String hibernateConfig; + + /** + * Name of the persistence-unit. + * If there is only one persistence-unit available, that unit will be used + * automatically. + * Settings in this file will overwrite settings in the properties or the + * configuration file. + * + * @parameter + * @since 1.1.0 + */ + private String persistenceUnit; + /** * List of Hibernate-Mapping-Files (XML). * Multiple files can be separated with white-spaces and/or commas. * * @parameter property="hibernate.mapping" + * @since 1.0.2 */ private String hibernateMapping; /** * Target of execution: *

* - * A databaseconnection is only needed for EXPORT and BOTH, but a - * Hibernate-Dialect must always be choosen. + * A database connection is only needed for EXPORT and BOTH, but a + * Hibernate-Dialect must always be chosen. * * @parameter property="hibernate.export.target" default-value="EXPORT" + * @since 1.0 */ private String target; @@ -253,6 +334,7 @@ public class Hbm2DdlMojo extends AbstractMojo * If NONE is choosen, no databaseconnection is needed. * * @parameter property="hibernate.export.type" default-value="BOTH" + * @since 1.0 */ private String type; @@ -260,6 +342,7 @@ public class Hbm2DdlMojo extends AbstractMojo * Output file. * * @parameter property="hibernate.export.schema.filename" default-value="${project.build.directory}/schema.sql" + * @since 1.0 */ private String outputFile; @@ -267,6 +350,7 @@ public class Hbm2DdlMojo extends AbstractMojo * Delimiter in output-file. * * @parameter property="hibernate.export.schema.delimiter" default-value=";" + * @since 1.0 */ private String delimiter; @@ -274,16 +358,19 @@ public class Hbm2DdlMojo extends AbstractMojo * Format output-file. * * @parameter property="hibernate.export.schema.format" default-value="true" + * @since 1.0 */ private boolean format; /** * Generate envers schema for auditing tables. * - * @parameter property="hibernate.export.envers" default-value="false" + * @parameter property="hibernate.export.envers" default-value="true" + * @since 1.0.3 */ private boolean envers; + @Override public void execute() throws @@ -292,7 +379,7 @@ public class Hbm2DdlMojo extends AbstractMojo { if (skip) { - getLog().info("Exectuion of hibernate4-maven-plugin:export was skipped!"); + getLog().info("Execution of hibernate4-maven-plugin:export was skipped!"); project.getProperties().setProperty(EXPORT_SKIPPED_PROPERTY, "true"); return; } @@ -301,7 +388,7 @@ public class Hbm2DdlMojo extends AbstractMojo boolean modified = false; File saved = new File(buildDirectory + File.separator + MD5S); - if (saved.exists()) + if (saved.isFile() && saved.length() > 0) { try { @@ -329,20 +416,39 @@ public class Hbm2DdlMojo extends AbstractMojo } } - ClassLoader classLoader = null; + URLClassLoader classLoader = null; try { getLog().debug("Creating ClassLoader for project-dependencies..."); List classpathFiles = project.getCompileClasspathElements(); if (scanTestClasses) classpathFiles.addAll(project.getTestClasspathElements()); - URL[] urls = new URL[classpathFiles.size()]; - for (int i = 0; i < classpathFiles.size(); ++i) + List urls = new LinkedList(); + File file; + file = new File(testOutputDirectory); + if (!file.exists()) + { + getLog().info("creating test-output-directory: " + testOutputDirectory); + file.mkdirs(); + } + urls.add(file.toURI().toURL()); + file = new File(outputDirectory); + if (!file.exists()) + { + getLog().info("creating output-directory: " + outputDirectory); + file.mkdirs(); + } + urls.add(file.toURI().toURL()); + for (String pathElement : classpathFiles) { - getLog().debug("Dependency: " + classpathFiles.get(i)); - urls[i] = new File(classpathFiles.get(i)).toURI().toURL(); + getLog().debug("Dependency: " + pathElement); + urls.add(new File(pathElement).toURI().toURL()); } - classLoader = new URLClassLoader(urls, getClass().getClassLoader()); + classLoader = + new URLClassLoader( + urls.toArray(new URL[urls.size()]), + getClass().getClassLoader() + ); } catch (Exception e) { @@ -381,6 +487,34 @@ public class Hbm2DdlMojo extends AbstractMojo db.scanArchives(dirUrl); } } + if (scanDependencies != null) + { + Matcher matcher = split.matcher(scanDependencies); + while (matcher.find()) + { + getLog().info("Scanning dependencies for scope " + matcher.group()); + for (Artifact artifact : project.getDependencyArtifacts()) + { + if (!artifact.getScope().equalsIgnoreCase(matcher.group())) + continue; + if (artifact.getFile() == null) + { + getLog().warn( + "Cannot scan dependency " + + artifact.getId() + + ": no JAR-file available!" + ); + continue; + } + getLog().info( + "Scanning dependency " + + artifact.getId() + + " for annotated classes..." + ); + db.scanArchives(artifact.getFile().toURI().toURL()); + } + } + } Set classNames = new HashSet(); if (db.getAnnotationIndex().containsKey(Entity.class.getName())) @@ -432,368 +566,459 @@ public class Hbm2DdlMojo extends AbstractMojo throw new MojoFailureException(e.getMessage()); } - if (classes.isEmpty()) - { - if (hibernateMapping == null || hibernateMapping.isEmpty()) - throw new MojoFailureException("No annotated classes found in directory " + outputDirectory); - } - else - { - getLog().debug("Detected classes with mapping-annotations:"); - for (Class annotatedClass : classes) - getLog().debug(" " + annotatedClass.getName()); - } + ValidationConfiguration config = new ValidationConfiguration(); + // Clear unused system-properties + config.setProperties(new Properties()); - Properties properties = new Properties(); - /** Try to read configuration from properties-file */ + ClassLoader contextClassLoader = Thread.currentThread().getContextClassLoader(); + StandardServiceRegistryImpl registry = null; + MavenLogAppender.startPluginLog(this); + try { - File file = new File(hibernateProperties); - if (file.exists()) + /** + * Change class-loader of current thread, so that hibernate can + * see all dependencies! + */ + Thread.currentThread().setContextClassLoader(classLoader); + + + /** Try to read configuration from properties-file */ + try { - getLog().info("Reading properties from file " + hibernateProperties + "..."); - properties.load(new FileInputStream(file)); + if (hibernateProperties == null) + { + URL url = classLoader.findResource("hibernate.properties"); + if (url == null) + { + getLog().info("No hibernate.properties on the classpath!"); + } + else + { + getLog().info("Reading settings from hibernate.properties on the classpath."); + Properties properties = new Properties(); + properties.load(url.openStream()); + config.setProperties(properties); + } + } + else + { + File file = new File(hibernateProperties); + if (file.exists()) + { + getLog().info("Reading settings from file " + hibernateProperties + "..."); + Properties properties = new Properties(); + properties.load(new FileInputStream(file)); + config.setProperties(properties); + } + else + getLog().info("No hibernate-properties-file found! (Checked path: " + hibernateProperties + ")"); + } + } + catch (IOException e) + { + getLog().error("Error while reading properties!", e); + throw new MojoExecutionException(e.getMessage()); } - else - getLog().info("No hibernate-properties-file found! (Checked path: " + hibernateProperties + ")"); - } - catch (IOException e) - { - getLog().error("Error while reading properties!", e); - throw new MojoExecutionException(e.getMessage()); - } - /** Overwrite values from propertie-file or set, if given */ - if (driverClassName != null) - { - if (properties.containsKey(DRIVER_CLASS)) - getLog().debug( - "Overwriting property " + - DRIVER_CLASS + "=" + properties.getProperty(DRIVER_CLASS) + - " with the value " + driverClassName - ); - else - getLog().debug("Using the value " + driverClassName); - properties.setProperty(DRIVER_CLASS, driverClassName); - } - if (url != null) - { - if (properties.containsKey(URL)) - getLog().debug( - "Overwriting property " + - URL + "=" + properties.getProperty(URL) + - " with the value " + url - ); - else - getLog().debug("Using the value " + url); - properties.setProperty(URL, url); - } - if (username != null) - { - if (properties.containsKey(USERNAME)) - getLog().debug( - "Overwriting property " + - USERNAME + "=" + properties.getProperty(USERNAME) + - " with the value " + username - ); - else - getLog().debug("Using the value " + username); - properties.setProperty(USERNAME, username); - } - if (password != null) - { - if (properties.containsKey(PASSWORD)) - getLog().debug( - "Overwriting property " + - PASSWORD + "=" + properties.getProperty(PASSWORD) + - " with value " + password - ); - else - getLog().debug("Using value " + password + " for property " + PASSWORD); - properties.setProperty(PASSWORD, password); - } - if (hibernateDialect != null) - { - if (properties.containsKey(DIALECT)) - getLog().debug( - "Overwriting property " + - DIALECT + "=" + properties.getProperty(DIALECT) + - " with value " + hibernateDialect - ); - else - getLog().debug( - "Using value " + hibernateDialect + " for property " + DIALECT - ); - properties.setProperty(DIALECT, hibernateDialect); - } - if ( hibernateNamingStrategy != null ) - { - if ( properties.contains(NAMING_STRATEGY)) - getLog().debug( - "Overwriting property " + - NAMING_STRATEGY + "=" + properties.getProperty(NAMING_STRATEGY) + - " with value " + hibernateNamingStrategy - ); - else - getLog().debug( - "Using value " + hibernateNamingStrategy + " for property " + - NAMING_STRATEGY + /** Try to read configuration from configuration-file */ + try + { + if (hibernateConfig == null) + { + URL url = classLoader.findResource("hibernate.cfg.xml"); + if (url == null) + { + getLog().info("No hibernate.cfg.xml on the classpath!"); + } + else + { + getLog().info("Reading settings from hibernate.cfg.xml on the classpath."); + config.configure(url); + } + } + else + { + File file = new File(hibernateConfig); + if (file.exists()) + { + getLog().info("Reading configuration from file " + hibernateConfig + "..."); + config.configure(file); + } + else + getLog().info("No hibernate-configuration-file found! (Checked path: " + hibernateConfig + ")"); + } + } + catch (Exception e) + { + getLog().error("Error while reading configuration!", e); + throw new MojoExecutionException(e.getMessage()); + } + + ParsedPersistenceXmlDescriptor persistenceUnitDescriptor = + getPersistenceUnitDescriptor( + persistenceUnit, + config.getProperties(), + new MavenProjectClassLoaderService(classLoader) + ); + if (persistenceUnitDescriptor != null) + config.setProperties(persistenceUnitDescriptor.getProperties()); + + + /** Overwrite values from properties-file or set, if given */ + + if (driverClassName != null) + { + if (config.getProperties().containsKey(DRIVER_CLASS)) + getLog().debug( + "Overwriting property " + + DRIVER_CLASS + "=" + config.getProperty(DRIVER_CLASS) + + " with the value " + driverClassName ); - properties.setProperty(NAMING_STRATEGY, hibernateNamingStrategy); - } + else + getLog().debug("Using the value " + driverClassName); + config.setProperty(DRIVER_CLASS, driverClassName); + } + if (config.getProperty(DRIVER_CLASS) == null) + { + String driver = config.getProperty(JPA_DRIVER); + if (driver != null) + { + getLog().info( + DRIVER_CLASS + + " is not set. Borrow setting from " + + JPA_DRIVER + + ": " + + driver); + config.setProperty(DRIVER_CLASS, driver); + } + } - /** The generated SQL varies with the dialect! */ - if (md5s.containsKey(DIALECT)) - { - String dialect = properties.getProperty(DIALECT); - if (md5s.get(DIALECT).equals(dialect)) - getLog().debug("SQL-dialect unchanged."); - else + if (url != null) { - modified = true; - if (dialect == null) + if (config.getProperties().containsKey(URL)) + getLog().debug( + "Overwriting property " + + URL + "=" + config.getProperty(URL) + + " with the value " + url + ); + else + getLog().debug("Using the value " + url); + config.setProperty(URL, url); + } + if (config.getProperty(URL) == null) + { + String url = config.getProperty(JPA_URL); + if (url != null) { - getLog().debug("SQL-dialect was unset."); - md5s.remove(DIALECT); + getLog().info( + URL + + " is not set. Borrow setting from " + + JPA_URL + + ": " + + url); + config.setProperty(URL, url); } + } + + if (username != null) + { + if (config.getProperties().containsKey(USERNAME)) + getLog().debug( + "Overwriting property " + + USERNAME + "=" + config.getProperty(USERNAME) + + " with the value " + username + ); else + getLog().debug("Using the value " + username); + config.setProperty(USERNAME, username); + } + if (config.getProperty(USERNAME) == null) + { + String username = config.getProperty(JPA_USERNAME); + if (username != null) { - getLog().debug("SQL-dialect changed: " + dialect); - md5s.put(DIALECT, dialect); + getLog().info( + USERNAME + + " is not set. Borrow setting from " + + JPA_USERNAME + + ": " + + username); + config.setProperty(USERNAME, username); } } - } - else - { - String dialect = properties.getProperty(DIALECT); - if (dialect != null) + + if (password != null) { - modified = true; - md5s.put(DIALECT, properties.getProperty(DIALECT)); + if (config.getProperties().containsKey(PASSWORD)) + getLog().debug( + "Overwriting property " + + PASSWORD + "=" + config.getProperty(PASSWORD) + + " with value " + password + ); + else + getLog().debug("Using value " + password + " for property " + PASSWORD); + config.setProperty(PASSWORD, password); + } + if (config.getProperty(PASSWORD) == null) + { + String password = config.getProperty(JPA_PASSWORD); + if (password != null) + { + getLog().info( + PASSWORD + + " is not set. Borrow setting from " + + JPA_PASSWORD + + ": " + + password); + config.setProperty(PASSWORD, password); + } } - } - /** The generated SQL varies with the envers-configuration */ - if (md5s.get(ENVERS) != null) - { - if (md5s.get(ENVERS).equals(Boolean.toString(envers))) - getLog().debug("Envers-Configuration unchanged. Enabled: " + envers); - else + if (hibernateDialect != null) { - getLog().debug("Envers-Configuration changed. Enabled: " + envers); - modified = true; - md5s.put(ENVERS, Boolean.toString(envers)); + if (config.getProperties().containsKey(DIALECT)) + getLog().debug( + "Overwriting property " + + DIALECT + "=" + config.getProperty(DIALECT) + + " with value " + hibernateDialect + ); + else + getLog().debug( + "Using value " + hibernateDialect + " for property " + DIALECT + ); + config.setProperty(DIALECT, hibernateDialect); } - } - else - { - modified = true; - md5s.put(ENVERS, Boolean.toString(envers)); - } - if (properties.isEmpty()) - { - getLog().error("No properties set!"); - throw new MojoFailureException("Hibernate-Configuration is missing!"); - } + if ( hibernateNamingStrategy != null ) + { + if ( config.getProperties().contains(NAMING_STRATEGY)) + getLog().debug( + "Overwriting property " + + NAMING_STRATEGY + "=" + config.getProperty(NAMING_STRATEGY) + + " with value " + hibernateNamingStrategy + ); + else + getLog().debug( + "Using value " + hibernateNamingStrategy + " for property " + + NAMING_STRATEGY + ); + config.setProperty(NAMING_STRATEGY, hibernateNamingStrategy); + } - Configuration config = new Configuration(); - config.setProperties(properties); + /** The generated SQL varies with the dialect! */ + if (md5s.containsKey(DIALECT)) + { + String dialect = config.getProperty(DIALECT); + if (md5s.get(DIALECT).equals(dialect)) + getLog().debug("SQL-dialect unchanged."); + else + { + modified = true; + if (dialect == null) + { + getLog().debug("SQL-dialect was unset."); + md5s.remove(DIALECT); + } + else + { + getLog().debug("SQL-dialect changed: " + dialect); + md5s.put(DIALECT, dialect); + } + } + } + else + { + String dialect = config.getProperty(DIALECT); + if (dialect != null) + { + modified = true; + md5s.put(DIALECT, config.getProperty(DIALECT)); + } + } - if ( properties.containsKey(NAMING_STRATEGY)) - { - String namingStrategy = properties.getProperty(NAMING_STRATEGY); - getLog().debug("Explicitly set NamingStrategy: " + namingStrategy); - try + /** The generated SQL varies with the envers-configuration */ + if (md5s.get(ENVERS) != null) { - @SuppressWarnings("unchecked") - Class namingStrategyClass = (Class) Class.forName(namingStrategy); - config.setNamingStrategy(namingStrategyClass.newInstance()); + if (md5s.get(ENVERS).equals(Boolean.toString(envers))) + getLog().debug("Envers-Configuration unchanged. Enabled: " + envers); + else + { + getLog().debug("Envers-Configuration changed. Enabled: " + envers); + modified = true; + md5s.put(ENVERS, Boolean.toString(envers)); + } } - catch (Exception e) + else { - getLog().error("Error setting NamingStrategy", e); - throw new MojoExecutionException(e.getMessage()); + modified = true; + md5s.put(ENVERS, Boolean.toString(envers)); } - } - getLog().debug("Adding annotated classes to hibernate-mapping-configuration..."); - for (Class annotatedClass : classes) - { - getLog().debug("Class " + annotatedClass); - config.addAnnotatedClass(annotatedClass); - } + if (config.getProperties().isEmpty()) + { + getLog().error("No properties set!"); + throw new MojoFailureException("Hibernate configuration is missing!"); + } - if (hibernateMapping != null) - { - try + getLog().info("Gathered hibernate-configuration (turn on debugging for details):"); + for (Entry entry : config.getProperties().entrySet()) + getLog().info(" " + entry.getKey() + " = " + entry.getValue()); + + + getLog().debug("Adding explicitly configured mappings..."); + if (hibernateMapping != null) { - MessageDigest digest = java.security.MessageDigest.getInstance("MD5"); - for (String filename : hibernateMapping.split("[\\s,]+")) + try { - // First try the filename as absolute/relative path - File file = new File(filename); - if (!file.exists()) + MessageDigest digest = java.security.MessageDigest.getInstance("MD5"); + for (String filename : hibernateMapping.split("[\\s,]+")) { - // If the file was not found, search for it in the resource-directories - for (Resource resource : project.getResources()) + // First try the filename as absolute/relative path + File file = new File(filename); + if (!file.exists()) { - file = new File(resource.getDirectory() + File.separator + filename); - if (file.exists()) - break; + // If the file was not found, search for it in the resource-directories + for (Resource resource : project.getResources()) + { + file = new File(resource.getDirectory() + File.separator + filename); + if (file.exists()) + break; + } } - } - if (file != null && file.exists()) - { - InputStream is = new FileInputStream(file); - byte[] buffer = new byte[1024*4]; // copy data in 4MB-chunks - int i; - while((i = is.read(buffer)) > -1) - digest.update(buffer, 0, i); - is.close(); - byte[] bytes = digest.digest(); - BigInteger bi = new BigInteger(1, bytes); - String newMd5 = String.format("%0" + (bytes.length << 1) + "x", bi); - String oldMd5 = !md5s.containsKey(filename) ? "" : md5s.get(filename); - if (!newMd5.equals(oldMd5)) + if (file != null && file.exists()) { - getLog().debug("Found new or modified mapping-file: " + filename); - modified = true; - md5s.put(filename, newMd5); + InputStream is = new FileInputStream(file); + byte[] buffer = new byte[1024*4]; // copy data in 4MB-chunks + int i; + while((i = is.read(buffer)) > -1) + digest.update(buffer, 0, i); + is.close(); + byte[] bytes = digest.digest(); + BigInteger bi = new BigInteger(1, bytes); + String newMd5 = String.format("%0" + (bytes.length << 1) + "x", bi); + String oldMd5 = !md5s.containsKey(filename) ? "" : md5s.get(filename); + if (!newMd5.equals(oldMd5)) + { + getLog().debug("Found new or modified mapping-file: " + filename); + modified = true; + md5s.put(filename, newMd5); + } + else + { + getLog().debug(oldMd5 + " -> mapping-file unchanged: " + filename); + } + getLog().debug("Adding mappings from XML-configurationfile: " + file); + config.addFile(file); } else - { - getLog().debug(oldMd5 + " -> mapping-file unchanged: " + filename); - } - getLog().debug("Adding mappings from XML-configurationfile: " + file); - config.addFile(file); + throw new MojoFailureException("File " + filename + " could not be found in any of the configured resource-directories!"); } - else - throw new MojoFailureException("File " + filename + " could not be found in any of the configured resource-directories!"); + } + catch (NoSuchAlgorithmException e) + { + throw new MojoFailureException("Cannot calculate MD5 sums!", e); + } + catch (FileNotFoundException e) + { + throw new MojoFailureException("Cannot calculate MD5 sums!", e); + } + catch (IOException e) + { + throw new MojoFailureException("Cannot calculate MD5 sums!", e); } } - catch (NoSuchAlgorithmException e) + + getLog().debug("Adding annotated classes to hibernate-mapping-configuration..."); + // build annotated packages + Set packages = new HashSet(); + for (Class annotatedClass : classes) { - throw new MojoFailureException("Cannot calculate MD5-summs!", e); + String packageName = annotatedClass.getPackage().getName(); + if (!packages.contains(packageName)) + { + getLog().debug("Add package " + packageName); + packages.add(packageName); + config.addPackage(packageName); + getLog().debug("type definintions" + config.getTypeDefs()); + } + getLog().debug("Class " + annotatedClass); + config.addAnnotatedClass(annotatedClass); } - catch (FileNotFoundException e) + + Target target = null; + try { - throw new MojoFailureException("Cannot calculate MD5-summs!", e); + target = Target.valueOf(this.target.toUpperCase()); } - catch (IOException e) + catch (IllegalArgumentException e) { - throw new MojoFailureException("Cannot calculate MD5-summs!", e); + getLog().error("Invalid value for configuration-option \"target\": " + this.target); + getLog().error("Valid values are: NONE, SCRIPT, EXPORT, BOTH"); + throw new MojoExecutionException("Invalid value for configuration-option \"target\""); + } + Type type = null; + try + { + type = Type.valueOf(this.type.toUpperCase()); + } + catch (IllegalArgumentException e) + { + getLog().error("Invalid value for configuration-option \"type\": " + this.type); + getLog().error("Valid values are: NONE, CREATE, DROP, BOTH"); + throw new MojoExecutionException("Invalid value for configuration-option \"type\""); } - } - Target target = null; - try - { - target = Target.valueOf(this.target.toUpperCase()); - } - catch (IllegalArgumentException e) - { - getLog().error("Invalid value for configuration-option \"target\": " + this.target); - getLog().error("Valid values are: NONE, SCRIPT, EXPORT, BOTH"); - throw new MojoExecutionException("Invalid value for configuration-option \"target\""); - } - Type type = null; - try - { - type = Type.valueOf(this.type.toUpperCase()); - } - catch (IllegalArgumentException e) - { - getLog().error("Invalid value for configuration-option \"type\": " + this.type); - getLog().error("Valid values are: NONE, CREATE, DROP, BOTH"); - throw new MojoExecutionException("Invalid value for configuration-option \"type\""); - } - if (target.equals(Target.SCRIPT) || target.equals(Target.NONE)) - { - project.getProperties().setProperty(EXPORT_SKIPPED_PROPERTY, "true"); - } - if ( - !modified - && !target.equals(Target.SCRIPT) - && !target.equals(Target.NONE) - && !force - ) - { - getLog().info("No modified annotated classes or mapping-files found and dialect unchanged."); - getLog().info("Skipping schema generation!"); - project.getProperties().setProperty(EXPORT_SKIPPED_PROPERTY, "true"); - return; - } + if (config.getProperty(DIALECT) == null) + throw new MojoFailureException("hibernate-dialect must be set!"); - getLog().info("Gathered hibernate-configuration (turn on debugging for details):"); - for (Entry entry : properties.entrySet()) - getLog().info(" " + entry.getKey() + " = " + entry.getValue()); - Connection connection = null; - try - { - /** - * The connection must be established outside of hibernate, because - * hibernate does not use the context-classloader of the current - * thread and, hence, would not be able to resolve the driver-class! - */ - getLog().debug("Target: " + target + ", Type: " + type); - switch (target) + if (target.equals(Target.SCRIPT) || target.equals(Target.NONE)) { - case EXPORT: - case BOTH: - switch (type) - { - case CREATE: - case DROP: - case BOTH: - Class driverClass = classLoader.loadClass(properties.getProperty(DRIVER_CLASS)); - getLog().debug("Registering JDBC-driver " + driverClass.getName()); - DriverManager.registerDriver(new DriverProxy((Driver)driverClass.newInstance())); - getLog().debug( - "Opening JDBC-connection to " - + properties.getProperty(URL) - + " as " - + properties.getProperty(USERNAME) - + " with password " - + properties.getProperty(PASSWORD) - ); - connection = DriverManager.getConnection( - properties.getProperty(URL), - properties.getProperty(USERNAME), - properties.getProperty(PASSWORD) - ); - } + project.getProperties().setProperty(EXPORT_SKIPPED_PROPERTY, "true"); + } + if ( + !modified + && !target.equals(Target.SCRIPT) + && !target.equals(Target.NONE) + && !force + ) + { + getLog().info("No modified annotated classes or mapping-files found and dialect unchanged."); + getLog().info("Skipping schema generation!"); + project.getProperties().setProperty(EXPORT_SKIPPED_PROPERTY, "true"); + return; } - } - catch (ClassNotFoundException e) - { - getLog().error("Dependency for driver-class " + properties.getProperty(DRIVER_CLASS) + " is missing!"); - throw new MojoExecutionException(e.getMessage()); - } - catch (Exception e) - { - getLog().error("Cannot establish connection to database!"); - Enumeration drivers = DriverManager.getDrivers(); - if (!drivers.hasMoreElements()) - getLog().error("No drivers registered!"); - while (drivers.hasMoreElements()) - getLog().debug("Driver: " + drivers.nextElement()); - throw new MojoExecutionException(e.getMessage()); - } - ClassLoader contextClassLoader = Thread.currentThread().getContextClassLoader(); - MavenLogAppender.startPluginLog(this); - try - { - /** - * Change class-loader of current thread, so that hibernate can - * see all dependencies! - */ - Thread.currentThread().setContextClassLoader(classLoader); + + if ( config.getProperties().containsKey(NAMING_STRATEGY)) + { + String namingStrategy = config.getProperty(NAMING_STRATEGY); + getLog().debug("Explicitly set NamingStrategy: " + namingStrategy); + try + { + @SuppressWarnings("unchecked") + Class namingStrategyClass = (Class) Class.forName(namingStrategy); + config.setNamingStrategy(namingStrategyClass.newInstance()); + } + catch (Exception e) + { + getLog().error("Error setting NamingStrategy", e); + throw new MojoExecutionException(e.getMessage()); + } + } + + + Environment.verifyProperties(config.getProperties()); + ConfigurationHelper.resolvePlaceHolders(config.getProperties()); + registry = + (StandardServiceRegistryImpl) + new StandardServiceRegistryBuilder() + .applySettings(config.getProperties()) + .build(); config.buildMappings(); @@ -803,10 +1028,35 @@ public class Hbm2DdlMojo extends AbstractMojo AuditConfiguration.getFor(config); } - SchemaExport export = new SchemaExport(config, connection); - export.setOutputFile(outputFile); + SchemaExport export = new SchemaExport(registry, config); export.setDelimiter(delimiter); export.setFormat(format); + + File outF = new File(outputFile); + + if (!outF.isAbsolute()) + { + // Interpret relative file path relative to build directory + outF = new File(buildDirectory, outputFile); + getLog().info("Adjusted relative path, resulting path is " + outF.getPath()); + } + + // Ensure that directory path for specified file exists + File outFileParentDir = outF.getParentFile(); + if (null != outFileParentDir && !outFileParentDir.exists()) + { + try + { + getLog().info("Creating directory path for output file:" + outFileParentDir.getPath()); + outFileParentDir.mkdirs(); + } + catch (Exception e) + { + getLog().error("Error creating directory path for output file: " + e.getLocalizedMessage()); + } + } + + export.setOutputFile(outF.getPath()); export.execute(target, type); for (Object exception : export.getExceptions()) @@ -820,16 +1070,8 @@ public class Hbm2DdlMojo extends AbstractMojo /** Restore the old class-loader (TODO: is this really necessary?) */ Thread.currentThread().setContextClassLoader(contextClassLoader); - /** Close the connection */ - try - { - if (connection != null) - connection.close(); - } - catch (SQLException e) - { - getLog().error("Error while closing connection: " + e.getMessage()); - } + if (registry != null) + registry.destroy(); } /** Write md5-sums for annotated classes to file */ @@ -847,6 +1089,123 @@ public class Hbm2DdlMojo extends AbstractMojo } } + private ParsedPersistenceXmlDescriptor getPersistenceUnitDescriptor( + String name, + Properties properties, + ClassLoaderService loader + ) + throws + MojoFailureException + { + PersistenceXmlParser parser = + new PersistenceXmlParser( + loader, + PersistenceUnitTransactionType.RESOURCE_LOCAL + ); + + List units = parser.doResolve(properties); + + if (name == null) + { + switch (units.size()) + { + case 0: + getLog().info("Found no META-INF/persistence.xml."); + return null; + case 1: + getLog().info("Using persistence-unit " + units.get(0).getName()); + return units.get(0); + default: + getLog().warn("No name provided and multiple persistence units found:"); + for (ParsedPersistenceXmlDescriptor unit : units) + getLog().warn(" - " + unit.getName()); + return null; + } + + } + + for (ParsedPersistenceXmlDescriptor unit : units) + { + getLog().debug("Found persistence-unit " + unit.getName()); + if (!unit.getName().equals(name)) + continue; + + // See if we (Hibernate) are the persistence provider + if (!ProviderChecker.isProvider(unit, properties)) + { + getLog().debug("Wrong provider: " + unit.getProviderClassName()); + continue; + } + + getLog().info("Using persistence-unit " + unit.getName()); + return unit; + } + + throw new MojoFailureException("Could not find persistence-unit " + name); + } + + + static final class MavenProjectClassLoaderService implements ClassLoaderService + { + final private ClassLoader loader; + + + public MavenProjectClassLoaderService(ClassLoader loader) + { + this.loader = loader; + } + + + @Override + public Class classForName(String name) + { + try + { + return (Class)loader.loadClass(name); + } + catch (ClassNotFoundException e) + { + throw new ClassLoadingException( "Unable to load class [" + name + "]", e ); + } + } + + @Override + public URL locateResource(String name) + { + return loader.getResource(name); + } + + @Override + public InputStream locateResourceStream(String name) + { + return loader.getResourceAsStream(name); + } + + @Override + public List locateResources(String name) + { + try + { + return Collections.list(loader.getResources(name)); + } + catch (IOException e) + { + return Collections.EMPTY_LIST; + } + } + + @Override + public LinkedHashSet loadJavaServices(Class serviceContract) + { + throw new UnsupportedOperationException("Not supported yet."); //To change body of generated methods, choose Tools | Templates. + } + + @Override + public void stop() { } + + } + + /** * Needed, because DriverManager won't pick up drivers, that were not * loaded by the system-classloader!