[jbosstools-issues] [JBoss JIRA] (JBIDE-10392) Cannot target AS 7.1 with the new projects

[Rob Stryker (Updated) (JIRA)]

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

Rob Stryker updated JBIDE-10392:
--------------------------------

             Assignee: Snjezana Peco  (was: Rob Stryker)
    Affects Version/s: 3.3.0.M5
          Component/s: project-examples


I did some digging and found that none of the fixes in project examples add the facet runtime as a primary runtime object to the project. In my environment, after importing the hello-world example via JBoss Central, the only "fix" that is != null is WTPRuntimeFix. This fix, however, seems heavily targeted to ESB fixes, and not to generic projects such as hello-world. 

What I'm noting is that on line 94, the project has no primary runtime set yet. Therefore, it slips into the correcting of the esb section, where it does nothing. 

I'm attaching a patch which attempts to fix this logic. It isolates the code to check for whether esb is required. Then it ensures that the proper facet-runtime is added regardless of whether one exists on the project already. 

The patch is not 100%, though, because I commented out two lines, calling fixNonFacetedEsbProject.  I commented these lines out because by this point in the code, we have already guaranteed the project *IS* faceted. I am confident this code is correct, but I admit I do not know all the usecases and so cannot guarantee it. A test on your part would prove very helpful :) 
                
> Cannot target AS 7.1 with the new projects
> ------------------------------------------
>
>                 Key: JBIDE-10392
>                 URL: https://issues.jboss.org/browse/JBIDE-10392
>             Project: Tools (JBoss Tools)
>          Issue Type: Bug
>          Components: JBossAS/Servers, project-examples
>    Affects Versions: 3.3.0.M5
>            Reporter: Max Rydahl Andersen
>            Assignee: Snjezana Peco
>            Priority: Critical
>             Fix For: 3.3.0.M5
>
>
> install AS 7.1
> Run examples - it tells me it requries AS 7.0...ignore that.
> Target runtime is now blank even though I have AS7.1 installed.
> That should not happen.

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