[jbosstools-issues] [JBoss JIRA] Resolved: (JBIDE-5867) Move GlobalElVariablesComposite to org.jboss.tools.vpe or another appropriate vpe plugin

[Maxim Areshkau (JIRA)]

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

Maxim Areshkau resolved JBIDE-5867.
-----------------------------------

    Resolution: Done


 class has been moved to org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/preferences/ and import dependency removed from org.jboss.tools.common.el.ui manifest

> Move GlobalElVariablesComposite to org.jboss.tools.vpe or another appropriate vpe plugin
> ----------------------------------------------------------------------------------------
>
>                 Key: JBIDE-5867
>                 URL: https://jira.jboss.org/jira/browse/JBIDE-5867
>             Project: Tools (JBoss Tools)
>          Issue Type: Feature Request
>          Components: Visual Page Editor core
>    Affects Versions: 3.1.0.CR2
>            Reporter: Maxim Areshkau
>            Assignee: Maxim Areshkau
>            Priority: Critical
>             Fix For: 3.2.next
>
>
> Currently, plugin org.jboss.tools.common.el.ui depends on org.jboss.tools.vpe.resref through entry
> Import-Package: org.jboss.tools.vpe.resref.core
> in manifest. That package is only used by class GlobalElVariablesComposite and that class is only used in plugin org.jboss.tools.vpe. So, one vpe plugin is dependent on the other vpe plugin, which is ok, but by some reason a common plugin is inserted into the dependency. That makes our code balancing on the edge of cyclic dependency, though GlobalElVariablesComposite is not needed in common at all. Let's move it to org.jboss.tools.vpe or another appropriate vpe plugin as soon as possible.

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