[gatein-issues] [JBoss JIRA] (GTNPORTAL-3280) Remove only what was added in JavascriptConfigDeployer and GateInSkinConfigDeployer

[Peter Palaga (JIRA)]

     [ https://issues.jboss.org/browse/GTNPORTAL-3280?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Peter Palaga updated GTNPORTAL-3280:
------------------------------------

           Status: Resolved  (was: Pull Request Sent)
    Fix Version/s: 3.7.0.Final
       Resolution: Done

    
> Remove only what was added in JavascriptConfigDeployer and GateInSkinConfigDeployer
> -----------------------------------------------------------------------------------
>
>                 Key: GTNPORTAL-3280
>                 URL: https://issues.jboss.org/browse/GTNPORTAL-3280
>             Project: GateIn Portal
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>            Reporter: Peter Palaga
>            Assignee: Peter Palaga
>             Fix For: 3.7.0.Final
>
>
> Calling {{org.exoplatform.portal.resource.MainResourceResolver.removeServletContext(ServletContext)}} (via {{org.exoplatform.portal.resource.AbstractResourceService.unregisterServletContext(WebApp)}}) from {{org.exoplatform.web.application.javascript.JavascriptConfigDeployer.remove(WebApp)}} results in several unnecessary warnings during server shutdown and portal container reload. The cause is that only contexts having {{/WEB-INF/gatein-resources.xml}} are registered, but one tries to unregister all contexts, no matter if they contain {{/WEB-INF/gatein-resources.xml}} or not. The fix should add the "has resources.xml" condition also on removal in {{org.exoplatform.web.application.javascript.JavascriptConfigDeployer.remove(WebApp)}}
> The same holds for {{GateInSkinConfigRemoval}}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira