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 f2fd23203652c19133eb59e14677e48dfbbdb94a..9fe34880c0b874985392949ead03edc0674df248 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;