Skip to content

Commit

Permalink
WFLY-1785 WAR_BUNDLES_JSF_IMPL not working inside EAR
Browse files Browse the repository at this point in the history
  • Loading branch information
ssilvert authored and jaikiran committed Jul 31, 2013
1 parent 101d67a commit 5738dcd
Showing 1 changed file with 2 additions and 2 deletions.
Expand Up @@ -60,18 +60,18 @@ public class JSFDependencyProcessor implements DeploymentUnitProcessor {
@Override
public void deploy(DeploymentPhaseContext phaseContext) throws DeploymentUnitProcessingException {
final DeploymentUnit deploymentUnit = phaseContext.getDeploymentUnit();
final DeploymentUnit topLevelDeployment = deploymentUnit.getParent() == null ? deploymentUnit : deploymentUnit.getParent();

if (!DeploymentTypeMarker.isType(DeploymentType.WAR, deploymentUnit)) {
return;
}
if (JsfVersionMarker.getVersion(deploymentUnit).equals(JsfVersionMarker.WAR_BUNDLES_JSF_IMPL)) {
if (JsfVersionMarker.getVersion(topLevelDeployment).equals(JsfVersionMarker.WAR_BUNDLES_JSF_IMPL)) {
//if JSF is provided by the application we leave it alone
return;
}
//TODO: we should do that same check that is done in com.sun.faces.config.FacesInitializer
//and only add the dependency if JSF is actually needed

final DeploymentUnit topLevelDeployment = deploymentUnit.getParent() == null ? deploymentUnit : deploymentUnit.getParent();
final ModuleSpecification moduleSpecification = deploymentUnit.getAttachment(Attachments.MODULE_SPECIFICATION);
final ModuleLoader moduleLoader = Module.getBootModuleLoader();

Expand Down

0 comments on commit 5738dcd

Please sign in to comment.