Author: jim.ma Date: 2013-06-14 03:48:59 -0400 (Fri, 14 Jun 2013) New Revision: 17672 Modified: container/jboss72/branches/jbossws-jboss720/server-integration/src/main/java/org/jboss/as/webservices/deployers/EndpointServiceDeploymentAspect.java container/jboss72/branches/jbossws-jboss720/server-integration/src/main/java/org/jboss/as/webservices/service/EndpointService.java Log: [JBWS-3649]:Invoke LifecycleHandler in EndpointService Modified: container/jboss72/branches/jbossws-jboss720/server-integration/src/main/java/org/jboss/as/webservices/deployers/EndpointServiceDeploymentAspect.java =================================================================== --- container/jboss72/branches/jbossws-jboss720/server-integration/src/main/java/org/jboss/as/webservices/deployers/EndpointServiceDeploymentAspect.java 2013-06-14 07:35:08 UTC (rev 17671) +++ container/jboss72/branches/jbossws-jboss720/server-integration/src/main/java/org/jboss/as/webservices/deployers/EndpointServiceDeploymentAspect.java 2013-06-14 07:48:59 UTC (rev 17672) @@ -28,6 +28,7 @@ import org.jboss.as.webservices.service.EndpointService; import org.jboss.msc.service.ServiceTarget; import org.jboss.ws.common.deployment.EndpointLifecycleDeploymentAspect; +import org.jboss.ws.common.integration.AbstractDeploymentAspect; import org.jboss.wsf.spi.deployment.Deployment; import org.jboss.wsf.spi.deployment.Endpoint; import org.jboss.wsf.spi.deployment.LifecycleHandler; @@ -37,8 +38,9 @@ * * @author alessio.soldano(a)jboss.com * @author <a href="mailto:ropalka@redhat.com">Richard Opalka</a> + * @author <a href="mailto:ema@redhat.com">Jim Ma</a> */ -public final class EndpointServiceDeploymentAspect extends EndpointLifecycleDeploymentAspect implements Cloneable { +public final class EndpointServiceDeploymentAspect extends AbstractDeploymentAspect implements Cloneable { private boolean stopServices = false; @@ -48,7 +50,6 @@ final DeploymentUnit unit = getRequiredAttachment(dep, DeploymentUnit.class); for (final Endpoint ep : dep.getService().getEndpoints()) { EndpointService.install(target, ep, unit); - getLifecycleHandler(ep, true).start(ep); } } @@ -57,9 +58,6 @@ if (stopServices) { final DeploymentUnit unit = getRequiredAttachment(dep, DeploymentUnit.class); for (final Endpoint ep : dep.getService().getEndpoints()) { - LifecycleHandler lifecycleHandler = getLifecycleHandler(ep, false); - if (lifecycleHandler != null) - lifecycleHandler.stop(ep); EndpointService.uninstall(ep, unit); } } else { Modified: container/jboss72/branches/jbossws-jboss720/server-integration/src/main/java/org/jboss/as/webservices/service/EndpointService.java =================================================================== --- container/jboss72/branches/jbossws-jboss720/server-integration/src/main/java/org/jboss/as/webservices/service/EndpointService.java 2013-06-14 07:35:08 UTC (rev 17671) +++ container/jboss72/branches/jbossws-jboss720/server-integration/src/main/java/org/jboss/as/webservices/service/EndpointService.java 2013-06-14 07:48:59 UTC (rev 17672) @@ -106,11 +106,13 @@ registerRecordProcessor(processor, endpoint); } registerEndpoint(endpoint); + endpoint.getLifecycleHandler().start(endpoint); } @Override public void stop(final StopContext context) { ROOT_LOGGER.stopping(name); + endpoint.getLifecycleHandler().stop(endpoint); endpoint.setSecurityDomainContext(null); if (hasWebservicesMD(endpoint)) { pclWebAppControllerValue.getValue().decrementUsers();