[jbosstools-issues] [JBoss JIRA] (JBIDE-13309) New Restful Webservice wizard shouldn't change web.xml by default

[Brian Fitzpatrick (JIRA)]

    [ https://issues.jboss.org/browse/JBIDE-13309?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12947572#comment-12947572 ] 

Brian Fitzpatrick commented on JBIDE-13309:
-------------------------------------------

[~xcoulon] Can you review the changes if you get a chance? I know it looks like a lot of changes, but it's really only a few lines... 

1) JBossRSGenerateWizard.java - the default for "updateWebXML" went from true to false.
2) JBossRSGenerateWizardPage.java - I added "name.setEnabled(updateWebXML.getSelection());" at line 142 so the name box would be initially set to the correct state.
3) RSServiceSampleCreationCommand.java - createRESTApplicationClass has changed to reduce the code as suggested by Pete Muir in this JIRA.

The rest of the changes github is seeing are due to the Git tooling in Eclipse doing a wholesale file change for some reason - but the rest is the same. 
                
> New Restful Webservice wizard shouldn't change web.xml by default
> -----------------------------------------------------------------
>
>                 Key: JBIDE-13309
>                 URL: https://issues.jboss.org/browse/JBIDE-13309
>             Project: Tools (JBoss Tools)
>          Issue Type: Bug
>          Components: webservices
>    Affects Versions: 4.0.0.Final
>            Reporter: Pete Muir
>            Assignee: Brian Fitzpatrick
>             Fix For: 4.2.x
>
>
> We recommend the class-style activation of JAX-RS applications
> {code}
> /**
>  * A class extending {@link Application} and annotated with @ApplicationPath is the Java EE 6
>  * "no XML" approach to activating JAX-RS.
>  * 
>  * <p>
>  * Resources are served relative to the servlet path specified in the {@link ApplicationPath}
>  * annotation.
>  * </p>
>  */
> @ApplicationPath("/rest")
> public class JaxRsActivator extends Application {
>    /* class body intentionally left blank */
> }
> {code}
> JBDS should offer this as the default approach.

--
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