[jbosstools-issues] [JBoss JIRA] (JBIDE-14325) Remove UI dependencies from o.j.t.common.core

Rob Stryker (JIRA) jira-events at lists.jboss.org
Fri Apr 26 13:00:54 EDT 2013


Rob Stryker created JBIDE-14325:
-----------------------------------

             Summary: Remove UI dependencies from o.j.t.common.core
                 Key: JBIDE-14325
                 URL: https://issues.jboss.org/browse/JBIDE-14325
             Project: Tools (JBoss Tools)
          Issue Type: Bug
          Components: common/jst/core
    Affects Versions: 4.1.0.Alpha2
            Reporter: Rob Stryker
            Assignee: Rob Stryker
            Priority: Critical
             Fix For: 4.1.0.Beta1


One class is polluting o.j.t.common.core with a guaranteed UI class reference. The class in question is org.jboss.tools.common.text.TextProposal. This class should not be in a core plugin at all. 

This class USED to live in org.jboss.tools.common, but was mistakenly moved to org.jboss.tools.common.core. The package name was NOT changed, so consumers of this class almost 100% did not change their dependencies or depend on teh new LOCATION at all. 

Moving this class *BACK* to where it was before, while technically an API change, is almost assuredly very very low risk. There is a danger that *NEW* consumers have appeared, and that risk should be considered, but not exaggerated. 

One other class depends on two jeetools packages which are very large and have a sprawling dependency list. I can not say with certainty whether any UI is pulled in via this reference, but, i feel it is safest to remove it if it can be done in a safe manner. 

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


More information about the jbosstools-issues mailing list