WTP default server configuration only used for Websphere (and not for JBoss and 
others)
---------------------------------------------------------------------------------------

                 Key: MECLIPSE-454
                 URL: http://jira.codehaus.org/browse/MECLIPSE-454
             Project: Maven 2.x Eclipse Plugin
          Issue Type: Bug
          Components: WTP support
    Affects Versions: 2.5.1
         Environment: Windows XP SP3, JDK 1.5.0_14, Eclipse 3.3.2, Maven 2.0.8
            Reporter: M.-Leander Reimer
            Priority: Minor
         Attachments: EclipseWtpFacetsWriter.java

The <wtpdefaultserver> configuration setting is currently only used for 
Websphere servers. But shouldn't setting this to some other server also lead to 
a <runtime> definition in the Eclipse facets XML file 
(org.eclipse.wst.common.project.facet.core.xml)?

For example:
<wtpdefaultserver>JBoss v4.2</wtpdefaultserver> 
should produce
<runtime name="JBoss v4.2"/>

The minimal changes I made to correct this are in 
EclipseWtpFacetsWriter.writeModuleTypeFacetCore( XMLWriter writer, String 
packaging ) line 136 ...
I have simply put the relevant 3 lines out of the Websphere specific if 
statement.


            if (config.getWorkspaceConfiguration().getDefaultDeployServerName() 
!= null) {
                writer.startElement( "runtime" );
                writer.addAttribute( "name", 
config.getWorkspaceConfiguration().getDefaultDeployServerName() );
                writer.endElement(); // runtime
            }
            
            if ( this.config.getWorkspaceConfiguration().getWebsphereVersion() 
!= null )
            {
                writeFacetInstalledElement( writer, 
FACET_COM_IBM_WEBSPHERE_EXTENDED_EAR,
                                            
this.config.getWorkspaceConfiguration().getWebsphereVersion() ); // installed
                writeFacetInstalledElement( writer, 
FACET_COM_IBM_WEBSPHERE_COEXISTENCE_EAR,
                                            
this.config.getWorkspaceConfiguration().getWebsphereVersion() ); // installed
            }

Best regards,
Leander


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to