Author: fbricon Date: 2011-07-20 15:04:43 -0400 (Wed, 20 Jul 2011) New Revision: 33077 Modified: trunk/maven/plugins/org.jboss.tools.maven.jsf/src/org/jboss/tools/maven/jsf/configurators/JSFProjectConfigurator.java Log: Do not install the JSF Facet if the Web Facet is missing, or you'll face terrible consequences Modified: trunk/maven/plugins/org.jboss.tools.maven.jsf/src/org/jboss/tools/maven/jsf/configurators/JSFProjectConfigurator.java =================================================================== --- trunk/maven/plugins/org.jboss.tools.maven.jsf/src/org/jboss/tools/maven/jsf/configurators/JSFProjectConfigurator.java 2011-07-20 17:36:18 UTC (rev 33076) +++ trunk/maven/plugins/org.jboss.tools.maven.jsf/src/org/jboss/tools/maven/jsf/configurators/JSFProjectConfigurator.java 2011-07-20 19:04:43 UTC (rev 33077) @@ -143,8 +143,9 @@ if (!fproj.hasProjectFacet(dynamicWebFacet)) { Activator.log(Messages.JSFProjectConfigurator_The_project_does_not_contain_the_Web_Module_facet); - } - installJSFFacet(fproj, jsfVersion, mavenProject, monitor); + } else { + installJSFFacet(fproj, jsfVersion, mavenProject, monitor); + } installM2Facet(fproj, monitor); }