Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 335652 - [index] move AbstractAllContainersState to org.eclipse.xtext
Summary: [index] move AbstractAllContainersState to org.eclipse.xtext
Status: NEW
Alias: None
Product: TMF
Classification: Modeling
Component: Xtext Backlog (show other bugs)
Version: 2.0.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 326219
Blocks:
  Show dependency tree
 
Reported: 2011-01-28 01:52 EST by Knut Wannheden CLA
Modified: 2012-11-20 09:36 EST (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Knut Wannheden CLA 2011-01-28 01:52:33 EST 
The class AbstractAllContainersState currently resides in the plug-in org.eclipse.xtext.ui. I think it would make sense to move it to the plug-in org.eclipse.xtext since it doesn't contain any UI code nor depends on the Eclipse UI. It only depends on the IWorkspace. This thus also applies to WorkspaceProjectsState and WorkspaceProjectsStateHelper.

At the same time the AbstractAllContainersState should get the IWorkspace injected instead of getting it from the static method ResourcesPlugin#getWorkspace().