Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 237067 - [UI] Don't offer legacy packages without legacySupport enabled
Summary: [UI] Don't offer legacy packages without legacySupport enabled
Status: CLOSED INVALID
Alias: None
Product: EMF
Classification: Modeling
Component: cdo.legacy (show other bugs)
Version: 3.0   Edit
Hardware: All All
: P4 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Martin Fluegge CLA
QA Contact: Eike Stepper CLA
URL:
Whiteboard:
Keywords:
Depends on: 247226
Blocks:
  Show dependency tree
 
Reported: 2008-06-13 08:33 EDT by Eike Stepper CLA
Modified: 2012-06-19 07:08 EDT (History)
1 user (show)

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 Eike Stepper CLA 2008-06-13 08:33:26 EDT 
-> RegisterGeneratedPackage sub menu
-> PackageManagerDialog
Comment 1 Eike Stepper CLA 2008-12-01 08:35:34 EST 
Re-assigning to Vik in preparation of his new committer state...
Comment 2 Victor Roldan Betancort CLA 2008-12-10 07:21:12 EST 
Eike,

I've been looking a bit a this bug. However, I can't find any way to determine if a session has legacy support enabled. Could we add something to CDOSession.options() like "isLegacyEnabled()"?

Should I add dependency to:

247226: Transparently support legacy models
https://bugs.eclipse.org/bugs/show_bug.cgi?id=247226
Comment 3 Eike Stepper CLA 2008-12-10 13:51:10 EST 
Yes, I think the CDOSession:isLegacySupportEnabled() method is hidden somewhere in the bug #247226 patch ;-(

I reduced the prio to P4
Comment 4 Eike Stepper CLA 2009-11-01 05:59:29 EST 
Rebasing all unresolved enhancement requests to 3.0
Comment 5 Eike Stepper CLA 2010-04-04 03:06:42 EDT 
Martin, I think this doesn't make much sense since the legacy on/off flag is in the CDOView scope. If you agree then please close this bug as WONTFIX.
Comment 6 Martin Fluegge CLA 2010-04-04 04:42:22 EDT 
Guessing the scope of this bugzilla from the title and the short description this is already done because legacy is switched of by default. But I think that our UI namely the CDOSessionsView should be able to handle legacy. Either by switching it always 'on' or by implementing a button for changing the behavior. We could use this bugzilla for the task or raise a new one. In the latter case I agree with closing this one. What do you think about?
Comment 7 Eike Stepper CLA 2010-04-05 04:10:56 EDT 
My concern is more that EPackages are to be registered against a CDOSession, whereas the new legacy configuration is effective in the scope of CDOViews. I think this bugzilla is just senseless now.