[forge-issues] [JBoss JIRA] (FORGE-530) Group Faces Servlet url-patterns into a single servlet-mapping

[George Gastaldi (JIRA)]

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

George Gastaldi updated FORGE-530:
----------------------------------

              Status: Pull Request Sent  (was: Coding In Progress)
    Git Pull Request: https://github.com/forge/core/pull/132

    
> Group Faces Servlet url-patterns into a single servlet-mapping
> --------------------------------------------------------------
>
>                 Key: FORGE-530
>                 URL: https://issues.jboss.org/browse/FORGE-530
>             Project: Forge
>          Issue Type: Enhancement
>            Reporter: George Gastaldi
>            Assignee: George Gastaldi
>            Priority: Trivial
>
> When running faces setup, the following entry is generated on web.xml (if asked so):
> {code:xml}
>   <servlet-mapping>
>     <servlet-name>Faces Servlet</servlet-name>
>     <url-pattern>*.xhtml</url-pattern>
>   </servlet-mapping>
>   <servlet-mapping>
>     <servlet-name>Faces Servlet</servlet-name>
>     <url-pattern>/faces/*</url-pattern>
>   </servlet-mapping>
> {code}
> Since Servlet 2.5, several url-patterns may be grouped into a single servlet-mapping, so that the web.xml becomes smaller:
> {code:xml}
>   <servlet-mapping>
>     <servlet-name>Faces Servlet</servlet-name>
>     <url-pattern>*.xhtml</url-pattern>
>     <url-pattern>/faces/*</url-pattern>
>   </servlet-mapping>
> {code}
> The method _org.jboss.forge.spec.javaee.jsf.ServletMappingHelper.createMappingIfNotExists(Node, Node, String)_ should be reviewed to accomplish that.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira