From ac953e37d0f149d5a010c1cb719da9e2b9057671 Mon Sep 17 00:00:00 2001
From: aheritier <aheritier@13f79535-47bb-0310-9956-ffa450edef68>
Date: Sun, 22 Apr 2007 01:39:39 +0000
Subject: [PATCH] Move the version to 1.12 MPECLIPSE-127 : add support for wtp
 1.5 Dynamically set java and servlet versions in the facet descriptor Add a
 new property maven.eclipse.enable to avoid to have to define some natures to
 enable wtp Update documentation Generates project settings to store the
 compiler source and target levels

git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@531133 13f79535-47bb-0310-9956-ffa450edef68
---
 eclipse/plugin.jelly                          |  27 +-
 eclipse/plugin.properties                     |   5 +-
 eclipse/project.xml                           |   2 +-
 .../org.eclipse.jdt.core.prefs.jelly          |  30 +++
 .../plugin-resources/templates/project.jelly  |  20 ++
 .../org.eclipse.wst.common.component.jelly    | 238 ++++++++----------
 ...se.wst.common.project.facet.core.xml.jelly |  78 ++++--
 .../plugin-test/wtptest/project.properties    |   5 +-
 eclipse/xdocs/changes.xml                     |   4 +-
 eclipse/xdocs/faq.fml                         |  22 ++
 eclipse/xdocs/properties.xml                  |  15 ++
 11 files changed, 276 insertions(+), 170 deletions(-)
 create mode 100644 eclipse/src/plugin-resources/templates/org.eclipse.jdt.core.prefs.jelly

diff --git a/eclipse/plugin.jelly b/eclipse/plugin.jelly
index aaf97b99..e49a5bfc 100644
--- a/eclipse/plugin.jelly
+++ b/eclipse/plugin.jelly
@@ -116,11 +116,7 @@
     <j:if test="${sourcesPresent or !pom.build.resources.isEmpty()}">
       <attainGoal name="eclipse:generate-classpath" />
     </j:if>
-    <!-- jelly trick, you can't use "-1" in comparisons -->
-    <j:set var="indexnotfound" value="${0-1}" />
-    <!-- another jelly trick, you can't call methods on variables with a dot -->
-    <j:set var="mepn" value="${maven.eclipse.projectnatures}x" />
-    <j:if test="${mepn.indexOf('org.eclipse.wst.common.modulecore.ModuleCoreNature') != indexnotfound}">
+    <j:if test="${maven.eclipse.wtp.enable==true}">
       <attainGoal name="eclipse:generate-wtpmodules" />
     </j:if>
     <ant:echo>Now refresh your project in Eclipse (right click on the project and select "Refresh")</ant:echo>
@@ -149,6 +145,11 @@
     <j:file name="${basedir}/.classpath" prettyPrint="true" outputMode="xml" xmlns="dummy">
       <j:import file="${plugin.resources}/templates/classpath.jelly" inherit="true" />
     </j:file>
+    <ant:mkdir dir="${basedir}/.settings"/>
+    <ant:echo>Creating ${basedir}/.settings/org.eclipse.jdt.core.prefs ...</ant:echo>
+    <j:file name="${basedir}/.settings/org.eclipse.jdt.core.prefs" omitXmlDeclaration="true" outputMode="txt" xmlns="dummy">
+      <j:import file="${plugin.resources}/templates/org.eclipse.jdt.core.prefs.jelly" inherit="true" />
+    </j:file>
   </goal>
 
   <!--==================================================================-->
@@ -164,14 +165,14 @@
       </j:when>
       <j:when test="${maven.eclipse.wtp.version==1.5}">
         <ant:mkdir dir="${basedir}/.settings"/>
-	    <ant:echo>Creating ${basedir}/.settings/org.eclipse.wst.common.component ...</ant:echo>
-	    <j:file name="${basedir}/.settings/org.eclipse.wst.common.component" prettyPrint="true" outputMode="xml" xmlns="dummy">
-	      <j:import file="${plugin.resources}/templates/wtp/1.5/org.eclipse.wst.common.component.jelly" inherit="true" />
-	    </j:file>
-	    <ant:echo>Creating ${basedir}/.settings/org.eclipse.wst.common.project.facet.core.xml ...</ant:echo>
-	    <j:file name="${basedir}/.settings/org.eclipse.wst.common.project.facet.core.xml" prettyPrint="true" outputMode="xml" xmlns="dummy">
-	      <j:import file="${plugin.resources}/templates/wtp/1.5/org.eclipse.wst.common.project.facet.core.xml.jelly" inherit="true" />
-	    </j:file>
+  	    <ant:echo>Creating ${basedir}/.settings/org.eclipse.wst.common.component ...</ant:echo>
+  	    <j:file name="${basedir}/.settings/org.eclipse.wst.common.component" prettyPrint="true" outputMode="xml" xmlns="dummy">
+  	      <j:import file="${plugin.resources}/templates/wtp/1.5/org.eclipse.wst.common.component.jelly" inherit="true" />
+  	    </j:file>
+  	    <ant:echo>Creating ${basedir}/.settings/org.eclipse.wst.common.project.facet.core.xml ...</ant:echo>
+  	    <j:file name="${basedir}/.settings/org.eclipse.wst.common.project.facet.core.xml" prettyPrint="true" outputMode="xml" xmlns="dummy">
+  	      <j:import file="${plugin.resources}/templates/wtp/1.5/org.eclipse.wst.common.project.facet.core.xml.jelly" inherit="true" />
+  	    </j:file>
       </j:when>
       <j:otherwise>
         <ant:fail>maven.eclipse.wtp.version=${maven.eclipse.wtp.version} not supported. Please check the plugin documentation to have the list of supported values.</ant:fail>
diff --git a/eclipse/plugin.properties b/eclipse/plugin.properties
index b8d3814c..e49d1d58 100644
--- a/eclipse/plugin.properties
+++ b/eclipse/plugin.properties
@@ -91,4 +91,7 @@ maven.eclipse.javadoc.download = true
 maven.eclipse.servletapilist=javax.servlet:servlet-api,servletapi:servletapi,geronimo-spec:geronimo-spec-servlet
 
 # For which version of WTP we have to generate the configuration
-maven.eclipse.wtp.version=1.5
\ No newline at end of file
+maven.eclipse.wtp.version=1.5
+
+# Do you want to generate WTP configuration files
+maven.eclipse.wtp.enable=false
\ No newline at end of file
diff --git a/eclipse/project.xml b/eclipse/project.xml
index 2d6f42a8..44c65f74 100644
--- a/eclipse/project.xml
+++ b/eclipse/project.xml
@@ -22,7 +22,7 @@
   <pomVersion>3</pomVersion>
   <id>maven-eclipse-plugin</id>
   <name>Maven Eclipse Plugin</name>
-  <currentVersion>1.11.1-SNAPSHOT</currentVersion>
+  <currentVersion>1.12-SNAPSHOT</currentVersion>
   <description>A plugin to generate various files for the Eclipse IDE and ease the use of Maven within that environment.</description>
   <shortDescription>Eclipse Plugin for Maven</shortDescription>
   <versions>
diff --git a/eclipse/src/plugin-resources/templates/org.eclipse.jdt.core.prefs.jelly b/eclipse/src/plugin-resources/templates/org.eclipse.jdt.core.prefs.jelly
new file mode 100644
index 00000000..a5989042
--- /dev/null
+++ b/eclipse/src/plugin-resources/templates/org.eclipse.jdt.core.prefs.jelly
@@ -0,0 +1,30 @@
+<?xml version="1.0" encoding="ISO-8859-1"?>
+<!--
+  /*
+  * Copyright 2001-2004 The Apache Software Foundation.
+  *
+  * Licensed under the Apache License, Version 2.0 (the "License");
+  * you may not use this file except in compliance with the License.
+  * You may obtain a copy of the License at
+  *
+  *      http://www.apache.org/licenses/LICENSE-2.0
+  *
+  * Unless required by applicable law or agreed to in writing, software
+  * distributed under the License is distributed on an "AS IS" BASIS,
+  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  * See the License for the specific language governing permissions and
+  * limitations under the License.
+  */
+-->
+<j:whitespace xmlns:j="jelly:core" xmlns="dummy" trim="false">
+<j:set var="target" value="${maven.compile.target}" />
+<j:set var="source" value="${maven.compile.source}" />
+eclipse.preferences.version=1
+<j:if test="${target!=null}">
+org.eclipse.jdt.core.compiler.codegen.targetPlatform=${target}
+org.eclipse.jdt.core.compiler.compliance=${target}
+</j:if>
+<j:if test="${source!=null}">
+org.eclipse.jdt.core.compiler.source=${source}
+</j:if>
+</j:whitespace>
\ No newline at end of file
diff --git a/eclipse/src/plugin-resources/templates/project.jelly b/eclipse/src/plugin-resources/templates/project.jelly
index 8b959d40..835b8a02 100644
--- a/eclipse/src/plugin-resources/templates/project.jelly
+++ b/eclipse/src/plugin-resources/templates/project.jelly
@@ -44,6 +44,26 @@
     <j:if test="${mebc.indexOf('org.eclipse.jdt.core.javabuilder') == indexnotfound}">
       <j:set var="fullbuildcommands" value="org.eclipse.jdt.core.javabuilder,${maven.eclipse.buildcommands}" />
     </j:if>
+    
+    <j:if test="${maven.eclipse.wtp.enable==true}">
+      <!-- set required natures for WTP -->
+      <j:if test="${mepn.indexOf('org.eclipse.wst.common.project.facet.core.nature') == indexnotfound}">
+        <j:set var="fullprojectnatures" value="org.eclipse.wst.common.project.facet.core.nature,${fullprojectnatures}" />
+      </j:if>
+      <j:if test="${mepn.indexOf('org.eclipse.wst.common.modulecore.ModuleCoreNature') == indexnotfound}">
+        <j:set var="fullprojectnatures" value="org.eclipse.wst.common.modulecore.ModuleCoreNature,${fullprojectnatures}" />
+      </j:if>
+      <j:if test="${mepn.indexOf('org.eclipse.jem.workbench.JavaEMFNature') == indexnotfound}">
+        <j:set var="fullprojectnatures" value="org.eclipse.jem.workbench.JavaEMFNature,${fullprojectnatures}" />
+      </j:if>
+      <!-- set required build commands for WTP -->
+      <j:if test="${mebc.indexOf('org.eclipse.wst.common.project.facet.core.builder') == indexnotfound}">
+        <j:set var="fullbuildcommands" value="org.eclipse.wst.common.project.facet.core.builder,${fullbuildcommands}" />
+      </j:if>
+      <j:if test="${mebc.indexOf('org.eclipse.wst.validation.validationbuilder') == indexnotfound}">
+        <j:set var="fullbuildcommands" value="org.eclipse.wst.validation.validationbuilder,${fullbuildcommands}" />
+      </j:if>
+    </j:if>
   </j:if>
 
   <name>${maven.eclipse.project.name}</name>
diff --git a/eclipse/src/plugin-resources/templates/wtp/1.5/org.eclipse.wst.common.component.jelly b/eclipse/src/plugin-resources/templates/wtp/1.5/org.eclipse.wst.common.component.jelly
index eaf16b98..f0a3e4e6 100644
--- a/eclipse/src/plugin-resources/templates/wtp/1.5/org.eclipse.wst.common.component.jelly
+++ b/eclipse/src/plugin-resources/templates/wtp/1.5/org.eclipse.wst.common.component.jelly
@@ -1,132 +1,116 @@
 <?xml version="1.0" encoding="ISO-8859-1"?>
 <!--
-    /*
-    * Copyright 2001-2006 The Apache Software Foundation.
-    *
-    * Licensed under the Apache License, Version 2.0 (the "License");
-    * you may not use this file except in compliance with the License.
-    * You may obtain a copy of the License at
-    *
-    *      http://www.apache.org/licenses/LICENSE-2.0
-    *
-    * Unless required by applicable law or agreed to in writing, software
-    * distributed under the License is distributed on an "AS IS" BASIS,
-    * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-    * See the License for the specific language governing permissions and
-    * limitations under the License.
-    */
+  /*
+  * Copyright 2001-2006 The Apache Software Foundation.
+  *
+  * Licensed under the Apache License, Version 2.0 (the "License");
+  * you may not use this file except in compliance with the License.
+  * You may obtain a copy of the License at
+  *
+  *      http://www.apache.org/licenses/LICENSE-2.0
+  *
+  * Unless required by applicable law or agreed to in writing, software
+  * distributed under the License is distributed on an "AS IS" BASIS,
+  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  * See the License for the specific language governing permissions and
+  * limitations under the License.
+  */
 -->
-<j:whitespace xmlns:j="jelly:core" xmlns:maven="jelly:maven" xmlns:u="jelly:util" xmlns:eclipse="eclipse" xmlns="dummy" trim="true">
-    <project-modules id="moduleCoreId">
-        <j:set var="deployDir" value="/" />
-        <j:set var="moduletype" value="jst.utility" />
-        <j:set var="deployname" value="${pom.artifactId}" />
-        <j:set var="mmpt" value="${maven.multiproject.type}" trim="true" />
-        <j:choose>
-            <j:when test="${mmpt == 'war'}">
-                <!-- war -->
-                <j:set var="moduletype" value="jst.web" />
-                <j:set var="deployDir" value="/WEB-INF/classes" />
+<j:whitespace xmlns:j="jelly:core" xmlns:maven="jelly:maven" xmlns:u="jelly:util" xmlns:eclipse="eclipse" xmlns="dummy"
+  trim="true">
+  <project-modules id="moduleCoreId" project-version="1.5.0">
+    <j:set var="deployDir" value="/" />
+    <j:set var="moduletype" value="jst.utility" />
+    <j:set var="deployname" value="${pom.artifactId}" />
+    <j:set var="mmpt" value="${maven.multiproject.type}" trim="true" />
+    <j:choose>
+      <j:when test="${mmpt == 'war'}">
+        <!-- war -->
+        <j:set var="moduletype" value="jst.web" />
+        <j:set var="deployDir" value="/WEB-INF/classes" />
+      </j:when>
+    </j:choose>
+    <wb-module deploy-name="${deployname}">
+      <j:if test="${moduletype == 'jst.web'}">
+        <!-- war -->
+        <maven:get var="warPath" plugin='maven-war-plugin' property='maven.war.src' />
+        <maven:makeRelativePath var="relWarPath" basedir="${basedir}" path="${warPath}" separator="/" />
+        <wb-resource deploy-path="/" source-path="/${relWarPath}" />
+        <property name="context-root" value="${pom.artifactId}" />
+      </j:if>
+      <property name="java-output-path" value="/target/classes"/>
+      <j:set var="srcDir" value="" />
+      <!-- main src dir -->
+      <j:if test="${sourcesPresent}">
+        <maven:makeRelativePath var="srcDir" basedir="${basedir}" path="${pom.build.sourceDirectory}" separator="/" />
+        <wb-resource deploy-path="${deployDir}" source-path="/${srcDir}" />
+      </j:if>
+      <!-- resources -->
+      <j:if test="${maven.eclipse.resources.addtoclasspath}">
+        <j:if test="${!pom.build.resources.isEmpty()}">
+          <j:forEach var="resource" items="${pom.build.resources}">
+            <maven:makeRelativePath var="resourceDirectory" basedir="${basedir}" path="${resource.directory}"
+              separator="/" />
+            <!-- don't add duplicate directories -->
+            <j:if test="${!resourceDirectory.equals(srcDir)}">
+              <wb-resource deploy-path="${deployDir}" source-path="/${resourceDirectory}" />
+            </j:if>
+          </j:forEach>
+        </j:if>
+      </j:if>
+      <!-- generated source directories -->
+      <u:available file="${maven.gen.src}">
+        <u:file var="genSrcRootDir" name="${maven.gen.src}" />
+        <j:forEach var="genSrcDir" items="${genSrcRootDir.listFiles()}">
+          <maven:makeRelativePath var="srcDir" basedir="${basedir}" path="${genSrcDir}" separator="/" />
+          <wb-resource deploy-path="${deployDir}" source-path="/${srcDir}" />
+        </j:forEach>
+      </u:available>
+      <j:if test="${moduletype == 'jst.web'}">
+        <j:forEach var="lib" items="${pom.artifacts}">
+          <j:set var="eclipseDependency" value="${lib.dependency.getProperty('eclipse.dependency')}" />
+          <j:choose>
+            <j:when test="${eclipseDependency == 'true'}">
+              <dependent-module deploy-path="/WEB-INF/lib"
+                handle="module:/resource/${lib.dependency.artifactId}/${lib.dependency.artifactId}">
+                <dependency-type>uses</dependency-type>
+              </dependent-module>
             </j:when>
-        </j:choose>
-        <wb-module deploy-name="${deployname}">
-            <j:choose>
-                <j:when test="${moduletype == 'jst.web'}">
-                    <!-- war -->
-                    <maven:get var="warPath" plugin='maven-war-plugin' property='maven.war.src' />
-                    <maven:makeRelativePath var="relWarPath" basedir="${basedir}" path="${warPath}" separator="/" />
-                    <module-type module-type-id="jst.web">
-                        <version>
-                            <!-- try to detect servlet dependency, maven.eclipse.servletapilist can hold different names -->
-                            <j:set var="moduleversion" value="x" />
-                            <u:tokenize var="servletapilist" delim=",">${maven.eclipse.servletapilist}</u:tokenize>
-                            <j:forEach var="servletapi" items="${servletapilist}" trim="true">
-                                <j:if test="${pom.getDependency(servletapi) != null}">
-                                    <j:set var="moduleversion" value="${pom.getDependency(servletapi).getVersion()}" />
-                                </j:if>
-                            </j:forEach>
-                            <!-- defaults to 2.2 -->
-                            <j:choose>
-                                <j:when test="${moduleversion.startsWith('2.4')}">2.4</j:when>
-                                <j:when test="${moduleversion.startsWith('2.3')}">2.3</j:when>
-                                <j:otherwise>2.2</j:otherwise>
-                            </j:choose>
-                        </version>
-                        <property name="context-root" value="${pom.artifactId}" />
-                    </module-type>
-                    <wb-resource deploy-path="/" source-path="/${relWarPath}" />
-                </j:when>
-                <j:otherwise>
-                    <!-- jar -->
-                    <module-type module-type-id="jst.utility" />
-                </j:otherwise>
-            </j:choose>
-            <j:set var="srcDir" value="" />
-            <!-- main src dir -->
-            <j:if test="${sourcesPresent}">
-                <maven:makeRelativePath var="srcDir" basedir="${basedir}" path="${pom.build.sourceDirectory}" separator="/" />
-                <wb-resource deploy-path="${deployDir}" source-path="/${srcDir}" />
-            </j:if>
-            <!-- resources -->
-            <j:if test="${maven.eclipse.resources.addtoclasspath}">
-                <j:if test="${!pom.build.resources.isEmpty()}">
-                    <j:forEach var="resource" items="${pom.build.resources}">
-                        <maven:makeRelativePath var="resourceDirectory" basedir="${basedir}" path="${resource.directory}" separator="/" />
-                        <!-- don't add duplicate directories -->
-                        <j:if test="${!resourceDirectory.equals(srcDir)}">
-                            <wb-resource deploy-path="${deployDir}" source-path="/${resourceDirectory}" />
-                        </j:if>
-                    </j:forEach>
-                </j:if>
-            </j:if>
-            <!-- generated source directories -->
-            <u:available file="${maven.gen.src}">
-                <u:file var="genSrcRootDir" name="${maven.gen.src}" />
-                <j:forEach var="genSrcDir" items="${genSrcRootDir.listFiles()}">
-                    <maven:makeRelativePath var="srcDir" basedir="${basedir}" path="${genSrcDir}" separator="/" />
-                    <wb-resource deploy-path="${deployDir}" source-path="/${srcDir}" />
-                </j:forEach>
-            </u:available>
-            <j:if test="${moduletype == 'jst.web'}">
-                <j:forEach var="lib" items="${pom.artifacts}">
-                    <j:set var="eclipseDependency" value="${lib.dependency.getProperty('eclipse.dependency')}" />
-                    <j:choose>
-                        <j:when test="${eclipseDependency == 'true'}">
-                            <dependent-module deploy-path="/WEB-INF/lib"
-                                handle="module:/resource/${lib.dependency.artifactId}/${lib.dependency.artifactId}">
-                                <dependency-type>uses</dependency-type>
-                            </dependent-module>
-                        </j:when>
-                        <j:otherwise>
-                            <!-- make sure it's a webapp library -->
-                            <j:if test="${lib.dependency.getProperty('war.bundle')=='true'}">
-                                <maven:makeRelativePath var="relativePath" basedir="${maven.repo.local}" path="${lib.path}" separator="/" />
-                                <u:file var="checkExist" name="${maven.repo.local}/${relativePath}" />
-                                <j:choose>
-                                    <j:when test="${checkExist.exists()}">
-                                        <j:set var="relativePathCheck" value="${relativePath}X" />
-                                        <j:if test='${relativePathCheck == "X"}'>
-                                            <j:set var="relativePath"
-                                                value="${lib.dependency.groupId}/jars/${lib.dependency.artifactId}-${lib.dependency.version}.jar" />
-                                        </j:if>
-                                        <j:if test="${lib.dependency.type == 'tld'}"><j:set var="outputDir" value="${maven.war.tld.dir}"/></j:if>
-                                        <j:if test="${lib.dependency.type != 'tld'}"><j:set var="outputDir" value="/WEB-INF/lib"/></j:if>
-                                        <dependent-module deploy-path="${outputDir}"
-                                            handle="module:/classpath/var/MAVEN_REPO/${relativePath}">
-                                            <dependency-type>uses</dependency-type>
-                                        </dependent-module>
-                                    </j:when>
-                                    <j:otherwise>
-                                        <dependent-module deploy-path="/WEB-INF/lib" handle="module:/classpath/lib/${lib.path}">
-                                            <dependency-type>uses</dependency-type>
-                                        </dependent-module>
-                                    </j:otherwise>
-                                </j:choose>
-                            </j:if>
-                        </j:otherwise>
-                    </j:choose>
-                </j:forEach>
-            </j:if>
-        </wb-module>
-    </project-modules>
+            <j:otherwise>
+              <!-- make sure it's a webapp library -->
+              <j:if test="${lib.dependency.getProperty('war.bundle')=='true'}">
+                <maven:makeRelativePath var="relativePath" basedir="${maven.repo.local}" path="${lib.path}"
+                  separator="/" />
+                <u:file var="checkExist" name="${maven.repo.local}/${relativePath}" />
+                <j:choose>
+                  <j:when test="${checkExist.exists()}">
+                    <j:set var="relativePathCheck" value="${relativePath}X" />
+                    <j:if test='${relativePathCheck == "X"}'>
+                      <j:set var="relativePath"
+                        value="${lib.dependency.groupId}/jars/${lib.dependency.artifactId}-${lib.dependency.version}.jar" />
+                    </j:if>
+                    <j:if test="${lib.dependency.type == 'tld'}">
+                      <j:set var="outputDir" value="${maven.war.tld.dir}" />
+                    </j:if>
+                    <j:if test="${lib.dependency.type != 'tld'}">
+                      <j:set var="outputDir" value="/WEB-INF/lib" />
+                    </j:if>
+                    <dependent-module deploy-path="${outputDir}"
+                      handle="module:/classpath/var/MAVEN_REPO/${relativePath}">
+                      <dependency-type>uses</dependency-type>
+                    </dependent-module>
+                  </j:when>
+                  <j:otherwise>
+                    <dependent-module deploy-path="/WEB-INF/lib" handle="module:/classpath/lib/${lib.path}">
+                      <dependency-type>uses</dependency-type>
+                    </dependent-module>
+                  </j:otherwise>
+                </j:choose>
+              </j:if>
+            </j:otherwise>
+          </j:choose>
+        </j:forEach>
+      </j:if>
+    </wb-module>
+  </project-modules>
 </j:whitespace>
\ No newline at end of file
diff --git a/eclipse/src/plugin-resources/templates/wtp/1.5/org.eclipse.wst.common.project.facet.core.xml.jelly b/eclipse/src/plugin-resources/templates/wtp/1.5/org.eclipse.wst.common.project.facet.core.xml.jelly
index ba4ff634..04cc062d 100644
--- a/eclipse/src/plugin-resources/templates/wtp/1.5/org.eclipse.wst.common.project.facet.core.xml.jelly
+++ b/eclipse/src/plugin-resources/templates/wtp/1.5/org.eclipse.wst.common.project.facet.core.xml.jelly
@@ -1,35 +1,67 @@
 <?xml version="1.0" encoding="ISO-8859-1"?>
 <!--
-    /*
-    * Copyright 2001-2006 The Apache Software Foundation.
-    *
-    * Licensed under the Apache License, Version 2.0 (the "License");
-    * you may not use this file except in compliance with the License.
-    * You may obtain a copy of the License at
-    *
-    *      http://www.apache.org/licenses/LICENSE-2.0
-    *
-    * Unless required by applicable law or agreed to in writing, software
-    * distributed under the License is distributed on an "AS IS" BASIS,
-    * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-    * See the License for the specific language governing permissions and
-    * limitations under the License.
-    */
+  /*
+  * Copyright 2001-2006 The Apache Software Foundation.
+  *
+  * Licensed under the Apache License, Version 2.0 (the "License");
+  * you may not use this file except in compliance with the License.
+  * You may obtain a copy of the License at
+  *
+  *      http://www.apache.org/licenses/LICENSE-2.0
+  *
+  * Unless required by applicable law or agreed to in writing, software
+  * distributed under the License is distributed on an "AS IS" BASIS,
+  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  * See the License for the specific language governing permissions and
+  * limitations under the License.
+  */
 -->
-<j:whitespace xmlns:j="jelly:core" xmlns:maven="jelly:maven" xmlns:u="jelly:util" xmlns:eclipse="eclipse" xmlns="dummy" trim="true">
+<j:whitespace xmlns:j="jelly:core" xmlns:maven="jelly:maven" xmlns:u="jelly:util" xmlns:eclipse="eclipse" xmlns="dummy"
+  trim="true">
   <faceted-project>
-    <fixed facet="jst.java"/>
-    <installed facet="jst.java" version="5.0"/>
+    <fixed facet="jst.java" />
+    <j:set var="javaversion" value="${maven.compile.target}" />
+    <j:if test="javaversion==null">
+      <j:set var="javaversion" value="${maven.compile.source}" />
+    </j:if>
+    <j:choose>
+      <j:when test="${javaversion.startsWith('1.5')}">
+        <j:set var="javaversion" value="5.0" />
+      </j:when>
+      <j:when test="${javaversion.startsWith('1.6')}">
+        <j:set var="javaversion" value="6.0" />
+      </j:when>
+    </j:choose>
+    <installed facet="jst.java" version="${javaversion}" />
     <j:set var="mmpt" value="${maven.multiproject.type}" trim="true" />
     <j:choose>
       <j:when test="${mmpt == 'war'}">
-        <fixed facet="jst.web"/>
-        <installed facet="jst.web" version="2.3"/>
+        <fixed facet="jst.web" />
+        <!-- try to detect servlet dependency, maven.eclipse.servletapilist can hold different names -->
+        <j:set var="moduleversion" value="x" />
+        <u:tokenize var="servletapilist" delim=",">${maven.eclipse.servletapilist}</u:tokenize>
+        <j:forEach var="servletapi" items="${servletapilist}" trim="true">
+          <j:if test="${pom.getDependency(servletapi) != null}">
+            <j:set var="moduleversion" value="${pom.getDependency(servletapi).getVersion()}" />
+          </j:if>
+        </j:forEach>
+        <!-- defaults to 2.2 -->
+        <j:choose>
+          <j:when test="${moduleversion.startsWith('2.4')}">
+            <installed facet="jst.web" version="2.4" />
+          </j:when>
+          <j:when test="${moduleversion.startsWith('2.3')}">
+            <installed facet="jst.web" version="2.3" />
+          </j:when>
+          <j:otherwise>
+            <installed facet="jst.web" version="2.2" />
+          </j:otherwise>
+        </j:choose>
       </j:when>
       <j:otherwise>
-        <fixed facet="jst.utility"/>
-        <installed facet="jst.utility" version="1.0"/>
+        <fixed facet="jst.utility" />
+        <installed facet="jst.utility" version="1.0" />
       </j:otherwise>
-    </j:choose>                  
+    </j:choose>
   </faceted-project>
 </j:whitespace>
\ No newline at end of file
diff --git a/eclipse/src/plugin-test/wtptest/project.properties b/eclipse/src/plugin-test/wtptest/project.properties
index 96b0d4d3..6da990ce 100644
--- a/eclipse/src/plugin-test/wtptest/project.properties
+++ b/eclipse/src/plugin-test/wtptest/project.properties
@@ -1,6 +1,3 @@
 maven.eclipse.resources.addtoclasspath=true
 maven.multiproject.type=war
-
-maven.eclipse.projectnatures=org.eclipse.jdt.core.javanature,org.eclipse.wst.common.modulecore.ModuleCoreNature
-maven.eclipse.buildcommands=org.eclipse.wst.common.modulecore.ComponentStructuralBuilder,org.eclipse.jdt.core.javabuilder,org.eclipse.wst.common.modulecore.ComponentStructuralBuilderDependencyResolver
-maven.eclipse.conclasspath=org.eclipse.jst.server.core.container/org.eclipse.jst.server.tomcat.runtimeTarget/Apache Tomcat v5.5,org.eclipse.jst.j2ee.internal.web.container/wtptest
+maven.eclipse.wtp.enable=true
\ No newline at end of file
diff --git a/eclipse/xdocs/changes.xml b/eclipse/xdocs/changes.xml
index 1474d0c8..fcb8ed31 100644
--- a/eclipse/xdocs/changes.xml
+++ b/eclipse/xdocs/changes.xml
@@ -23,7 +23,9 @@
     <author email="dion@multitask.com.au">dIon Gillard</author>
   </properties>
   <body>
-    <release version="1.11.1-SNAPSHOT" date="In SVN">
+    <release version="1.12-SNAPSHOT" date="In SVN">
+      <action dev="aheritier" type="add">Project preferences are now generated with the classpath (to define the java version for sources and target).</action>
+      <action dev="aheritier" type="update">WTP settings are now generated if maven.eclipse.wtp.enable=false. No more need to define natures, build commands.</action>
       <action dev="aheritier" type="add" issues="MPECLIPSE-127">Add basic support (jar,war) for WTP 1.5</action>
       <action dev="aheritier" type="fix">WTP configuration : tld dependencies weren't deployed in ${maven.war.tld.dir} but in WEB-INF/lib</action>
       <action dev="aheritier" type="add">New goal eclipse:multiclean</action>
diff --git a/eclipse/xdocs/faq.fml b/eclipse/xdocs/faq.fml
index 2466c92e..cbfc0741 100644
--- a/eclipse/xdocs/faq.fml
+++ b/eclipse/xdocs/faq.fml
@@ -119,4 +119,26 @@ maven.hibernate.output.dir=${maven.build.dir}/generated-sources/schema
       </answer>
     </faq>
   </part>
+  
+  <part id="WTP">
+    <title>WTP support</title>
+    <faq id="version">
+      <question>How can I set the version of WTP ?</question>
+      <answer>
+        <p>To specify the version of WTP configuration files, set the property <a href="properties.html">maven.eclipse.wtp.version</a> .</p>
+      </answer>
+    </faq>
+    <faq id="activation">
+      <question>How can I activate WTP support for my project ?</question>
+      <answer>
+        <p>To activate WTP you have to set the property <a href="properties.html">maven.eclipse.wtp.enable</a> to true.</p>
+      </answer>
+    </faq>
+    <faq id="multiproject">
+      <question>Can I use WTP in a multiproject ?</question>
+      <answer>
+        <p>Yes, the plugin will read the value of the property maven.multiproject.type to generate the settings for libraries and for the web application.</p>
+      </answer>
+    </faq>
+  </part>
 </faqs>
diff --git a/eclipse/xdocs/properties.xml b/eclipse/xdocs/properties.xml
index d5414515..c050501b 100644
--- a/eclipse/xdocs/properties.xml
+++ b/eclipse/xdocs/properties.xml
@@ -252,6 +252,13 @@
             </p>
           </td>
         </tr>
+        <tr>
+          <td>maven.eclipse.wtp.enable</td>
+          <td>Yes</td>
+          <td>
+            <p>Do you want to generate settings for WTP? Default value : false.</p>
+          </td>
+        </tr>
         <tr>
           <td>maven.eclipse.wtp.version</td>
           <td>Yes</td>
@@ -260,6 +267,14 @@
             <p>Default value is <code>1.5</code>.</p>
           </td>
         </tr>
+        <tr>
+          <td>maven.eclipse.servletapilist</td>
+          <td>Yes</td>
+          <td>
+            <p>List of servlet api used to find the servlet version to set in wtp.</p>
+            <p>Default value is <code>javax.servlet:servlet-api,servletapi:servletapi,geronimo-spec:geronimo-spec-servlet</code>.</p>
+          </td>
+        </tr>
       </table>
       <p>
         Note that you will need to defined a