X-Git-Url: https://juplo.de/gitweb/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fde%2Fjuplo%2Fplugins%2Fhibernate4%2FHbm2DdlMojo.java;h=246f58523889de9b6c69449b57ae566a2d66a8fd;hb=03ed4ead0c3cf337e04de7c1ae0bbec11b68f9c1;hp=81cef938411d21461eeb7e9f79f5b675cca1558c;hpb=b9ab24b21d3eb65e2a2208be658ff447c1846894;p=hibernate4-maven-plugin diff --git a/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java b/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java index 81cef938..246f5852 100644 --- a/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java +++ b/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java @@ -53,6 +53,7 @@ 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.envers.configuration.AuditConfiguration; import org.hibernate.tool.hbm2ddl.SchemaExport; import org.hibernate.tool.hbm2ddl.SchemaExport.Type; import org.hibernate.tool.hbm2ddl.Target; @@ -77,11 +78,14 @@ public class Hbm2DdlMojo extends AbstractMojo public final static String USERNAME = "hibernate.connection.username"; public final static String PASSWORD = "hibernate.connection.password"; public final static String DIALECT = "hibernate.dialect"; + public final static String ENVERS = "hibernate.export.envers"; private final static String MD5S = "schema.md5s"; /** * The maven project. + *

+ * Only needed internally. * * @parameter expression="${project}" * @required @@ -91,20 +95,58 @@ public class Hbm2DdlMojo extends AbstractMojo /** * Build-directory. + *

+ * Only needed internally. * * @parameter expression="${project.build.directory}" + * @required + * @readonly */ private String buildDirectory; /** - * Class-directory to scan. + * Classes-Directory to scan. + *

+ * This parameter defaults to the maven build-output-directory for classes. + * Additonally, all dependencies are scanned for annotated classes. * * @parameter expression="${project.build.outputDirectory}" */ private String outputDirectory; + /** + * Wether 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 expression="${hibernate.export.scann_testclasses}" default-value="false" + */ + private boolean scanTestClasses; + + /** + * Test-Classes-Directory to scan. + *

+ * This parameter defaults to the maven build-output-directory for + * test-classes. + *

+ * This parameter is only used, when scanTestClasses is set + * to true! + * + * @parameter expression="${project.build.testOutputDirectory}" + */ + private String testOutputDirectory; + /** * Skip execution + *

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

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

+ * The excecution is skipped automatically, if no modified or newly added + * annotated classes are found and the dialect was not changed. * * @parameter expression="${maven.test.skip}" default-value="false" */ @@ -114,7 +156,9 @@ public class Hbm2DdlMojo extends AbstractMojo * Force execution *

* Force execution, even if no modified or newly added annotated classes - * where found. skip takes precedence over force. + * where found and the dialect was not changed. + *

+ * skip takes precedence over force. * * @parameter expression="${hibernate.export.force}" default-value="false" */ @@ -165,8 +209,8 @@ public class Hbm2DdlMojo extends AbstractMojo /** * Target of execution: *

@@ -175,12 +219,12 @@ public class Hbm2DdlMojo extends AbstractMojo private String target; /** - * Type of export. + * Type of execution. * * @parameter expression="${hibernate.export.type}" default-value="BOTH" */ @@ -207,6 +251,12 @@ public class Hbm2DdlMojo extends AbstractMojo */ private boolean format; + /** + * Generate envers schema for auditing tables. + * + * @parameter expression="${hibernate.export.envers}" default-value="false" + */ + private Boolean envers; @Override public void execute() @@ -262,6 +312,8 @@ public class Hbm2DdlMojo extends AbstractMojo { 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) { @@ -293,6 +345,15 @@ public class Hbm2DdlMojo extends AbstractMojo getLog().info("Scanning directory " + outputDirectory + " for annotated classes..."); URL dirUrl = dir.toURI().toURL(); db.scanArchives(dirUrl); + if (scanTestClasses) + { + dir = new File(testOutputDirectory); + if (!dir.exists()) + throw new MojoExecutionException("Cannot scan for annotated test-classes in " + testOutputDirectory + ": directory does not exist!"); + getLog().info("Scanning directory " + testOutputDirectory + " for annotated classes..."); + dirUrl = dir.toURI().toURL(); + db.scanArchives(dirUrl); + } Set classNames = new HashSet(); if (db.getAnnotationIndex().containsKey(Entity.class.getName())) @@ -362,7 +423,7 @@ public class Hbm2DdlMojo extends AbstractMojo properties.load(new FileInputStream(file)); } else - getLog().info("No hibernate-properties-file found! Checked path: " + hibernateProperties); + getLog().info("No hibernate-properties-file found! (Checked path: " + hibernateProperties + ")"); } catch (IOException e) { @@ -377,14 +438,10 @@ public class Hbm2DdlMojo extends AbstractMojo getLog().debug( "Overwriting property " + DRIVER_CLASS + "=" + properties.getProperty(DRIVER_CLASS) + - " with the value " + driverClassName + - " from the plugin-configuration-parameter driverClassName!" + " with the value " + driverClassName ); else - getLog().debug( - "Using the value " + driverClassName + - " from the plugin-configuration-parameter driverClassName!" - ); + getLog().debug("Using the value " + driverClassName); properties.setProperty(DRIVER_CLASS, driverClassName); } if (url != null) @@ -393,14 +450,10 @@ public class Hbm2DdlMojo extends AbstractMojo getLog().debug( "Overwriting property " + URL + "=" + properties.getProperty(URL) + - " with the value " + url + - " from the plugin-configuration-parameter url!" + " with the value " + url ); else - getLog().debug( - "Using the value " + url + - " from the plugin-configuration-parameter url!" - ); + getLog().debug("Using the value " + url); properties.setProperty(URL, url); } if (username != null) @@ -409,14 +462,10 @@ public class Hbm2DdlMojo extends AbstractMojo getLog().debug( "Overwriting property " + USERNAME + "=" + properties.getProperty(USERNAME) + - " with the value " + username + - " from the plugin-configuration-parameter username!" + " with the value " + username ); else - getLog().debug( - "Using the value " + username + - " from the plugin-configuration-parameter username!" - ); + getLog().debug("Using the value " + username); properties.setProperty(USERNAME, username); } if (password != null) @@ -425,14 +474,10 @@ public class Hbm2DdlMojo extends AbstractMojo getLog().debug( "Overwriting property " + PASSWORD + "=" + properties.getProperty(PASSWORD) + - " with the value " + password + - " from the plugin-configuration-parameter password!" + " with the value " + password ); else - getLog().debug( - "Using the value " + password + - " from the plugin-configuration-parameter password!" - ); + getLog().debug("Using the value " + password); properties.setProperty(PASSWORD, password); } if (hibernateDialect != null) @@ -441,14 +486,10 @@ public class Hbm2DdlMojo extends AbstractMojo getLog().debug( "Overwriting property " + DIALECT + "=" + properties.getProperty(DIALECT) + - " with the value " + hibernateDialect + - " from the plugin-configuration-parameter hibernateDialect!" + " with the value " + hibernateDialect ); else - getLog().debug( - "Using the value " + hibernateDialect + - " from the plugin-configuration-parameter hibernateDialect!" - ); + getLog().debug("Using the value " + hibernateDialect); properties.setProperty(DIALECT, hibernateDialect); } @@ -471,6 +512,36 @@ public class Hbm2DdlMojo extends AbstractMojo md5s.put(DIALECT, properties.getProperty(DIALECT)); } + if (md5s.containsKey(ENVERS)) + { + String envers = properties.getProperty(ENVERS); + if (md5s.get(ENVERS).equals(envers)) + getLog().debug("Envers unchanged."); + else + { + getLog().debug("Envers changed: " + envers); + modified = true; + md5s.put(ENVERS, envers.toString()); + } + } + else + { + modified = true; + md5s.put(ENVERS, properties.getProperty(ENVERS)); + } + + if (envers != null) { + if (properties.containsKey(ENVERS)) + getLog().debug( + "Overwriting property " + + ENVERS + "=" + properties.getProperty(ENVERS) + + " with the value " + envers + ); + else + getLog().debug("Using the value " + envers); + properties.setProperty(ENVERS, envers.toString()); + } + if (properties.isEmpty()) { getLog().error("No properties set!"); @@ -547,17 +618,28 @@ public class Hbm2DdlMojo extends AbstractMojo case CREATE: case DROP: case BOTH: - Class driverClass = classLoader.loadClass(driverClassName); + 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 " + url + " as " + username + " with password " + password); - connection = DriverManager.getConnection(url, username, password); + 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) + ); } } } catch (ClassNotFoundException e) { - getLog().error("Dependency for driver-class " + driverClassName + " is missing!"); + getLog().error("Dependency for driver-class " + properties.getProperty(DRIVER_CLASS) + " is missing!"); throw new MojoExecutionException(e.getMessage()); } catch (Exception e) @@ -581,6 +663,13 @@ public class Hbm2DdlMojo extends AbstractMojo */ Thread.currentThread().setContextClassLoader(classLoader); + config.buildMappings(); + if ("true".equals(properties.getProperty(ENVERS))) + { + getLog().debug("Using envers"); + AuditConfiguration.getFor(config); + } + SchemaExport export = new SchemaExport(config, connection); export.setOutputFile(outputFile); export.setDelimiter(delimiter); @@ -601,7 +690,8 @@ public class Hbm2DdlMojo extends AbstractMojo /** Close the connection */ try { - connection.close(); + if (connection != null) + connection.close(); } catch (SQLException e) {