[jbosstools-issues] [JBoss JIRA] (JBDS-2131) remove duplicate code from discovery plugins

[Nick Boldt (JIRA)]

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

Nick Boldt updated JBDS-2131:
-----------------------------

        Fix Version/s: 5.0.0.CR1
          Description: 
Snjezana, can you take a look at this?

Discovery plugins are here, for JBT/JBDS Core:

https://svn.jboss.org/repos/jbosstools/trunk/central/plugins/org.jboss.tools.central.discovery/

https://svn.jboss.org/repos/devstudio/trunk/product/plugins/com.jboss.jbds.central.discovery/

and JBT/JBDS SOA Tooling:

https://svn.jboss.org/repos/jbosstools/trunk/build/aggregate/soa-site/plugins/org.jboss.tools.central.discovery.soa-tooling/

https://svn.jboss.org/repos/devstudio/trunk/product-soa/plugins/com.jboss.jbds.central.discovery.soa-tooling/

On 05/10/2012 12:22 PM, Max Rydahl Andersen wrote:
> 
>> That's the plugin that provides Central discovery for SOA-P and BRMS connectors.
>>
>> Not sure if the code's needed. AFAIK we just need the plugin.xml file.
> 
> Afaik there should be *zero* java code in the discovery plugin, both for JBDS and SOA connectors.
> 
> /max
> 
>>
>> N
>>
>> On 05/10/2012 08:02 AM, Max Rydahl Andersen wrote:
>>> ..and why does it have java code in it copied from central ?!
>>>
>>> ./src/org/jboss/tools/central/configurators/DefaultJBossCentralDiscoveryConfigurator.java
>>> ./src/org/jboss/tools/central/configurators/IJBossCentralDiscoveryConfigurator.java
>>> ./src/org/jboss/tools/central/discovery/JBossCentralDiscoveryActivator.java
>>>
>>> /max

    Affects Version/s: 5.0.0.Beta3

    
> remove duplicate code from discovery plugins
> --------------------------------------------
>
>                 Key: JBDS-2131
>                 URL: https://issues.jboss.org/browse/JBDS-2131
>             Project: Developer Studio (JBoss Developer Studio)
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: central
>    Affects Versions: 5.0.0.Beta3
>            Reporter: Nick Boldt
>            Assignee: Snjezana Peco
>             Fix For: 5.0.0.CR1
>
>
> Snjezana, can you take a look at this?
> Discovery plugins are here, for JBT/JBDS Core:
> https://svn.jboss.org/repos/jbosstools/trunk/central/plugins/org.jboss.tools.central.discovery/
> https://svn.jboss.org/repos/devstudio/trunk/product/plugins/com.jboss.jbds.central.discovery/
> and JBT/JBDS SOA Tooling:
> https://svn.jboss.org/repos/jbosstools/trunk/build/aggregate/soa-site/plugins/org.jboss.tools.central.discovery.soa-tooling/
> https://svn.jboss.org/repos/devstudio/trunk/product-soa/plugins/com.jboss.jbds.central.discovery.soa-tooling/
> On 05/10/2012 12:22 PM, Max Rydahl Andersen wrote:
> > 
> >> That's the plugin that provides Central discovery for SOA-P and BRMS connectors.
> >>
> >> Not sure if the code's needed. AFAIK we just need the plugin.xml file.
> > 
> > Afaik there should be *zero* java code in the discovery plugin, both for JBDS and SOA connectors.
> > 
> > /max
> > 
> >>
> >> N
> >>
> >> On 05/10/2012 08:02 AM, Max Rydahl Andersen wrote:
> >>> ..and why does it have java code in it copied from central ?!
> >>>
> >>> ./src/org/jboss/tools/central/configurators/DefaultJBossCentralDiscoveryConfigurator.java
> >>> ./src/org/jboss/tools/central/configurators/IJBossCentralDiscoveryConfigurator.java
> >>> ./src/org/jboss/tools/central/discovery/JBossCentralDiscoveryActivator.java
> >>>
> >>> /max

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