Attribute "skip" is no initialized with value of property maven.test.skip

If one runs hibernate4:export with the switch "-Dmaven.test.skip=true", the
configured database was droped, although the execution of the tests was
skipped.

This might lead to serious data-loss, if one builds a project on the
production-machine. Hence, the excecution of hibernate4:export is aborted
now, if maven.test.skip is true.

diff --git a/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java b/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java
index f2fd232..9fe3488 100644 (file)
--- a/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java
+++ b/src/main/java/de/juplo/plugins/hibernate4/Hbm2DdlMojo.java
@@ -77,7 +77,7 @@ public class Hbm2DdlMojo extends AbstractMojo
   /**
    * Skip execution
    *
-   * @parameter default-value="false"
+   * @parameter expression="${maven.test.skip}"
    */
   private boolean skip;