Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 348334 - OCL.initialize is inadequate
Summary: OCL.initialize is inadequate
Status: CLOSED WONTFIX
Alias: None
Product: OCL
Classification: Modeling
Component: Core (show other bugs)
Version: 3.1.0   Edit
Hardware: PC Windows Vista
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: OCL Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-05 12:50 EDT by Ed Willink CLA
Modified: 2013-05-20 11:37 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 Ed Willink CLA 2011-06-05 12:50:57 EDT 
The Pivot variant is totally inadequate registering only *.ecore.

The Ecore variant ignores delegates.
Comment 1 Axel Uhl CLA 2011-06-05 15:50:21 EDT 
Ed,

what exactly do you mean with "ignores delegates?"
Comment 2 Ed Willink CLA 2011-06-05 16:33:55 EDT 
(In reply to comment #1)
> Ed,
> 
> what exactly do you mean with "ignores delegates?"

The delegates are registered via extension points. This is not replicated in the initialize(). From DelegatesTest at least

String oclDelegateURI = OCLDelegateDomain.OCL_DELEGATE_URI;
EOperation.Internal.InvocationDelegate.Factory.Registry.INSTANCE.put
  (oclDelegateURI, new OCLInvocationDelegateFactory.Global());
EStructuralFeature.Internal.SettingDelegate.Factory.Registry.INSTANCE.put
  (oclDelegateURI, new OCLSettingDelegateFactory.Global());
EValidator.ValidationDelegate.Registry.INSTANCE.put
  (oclDelegateURI, new OCLValidationDelegateFactory.Global());
QueryDelegate.Factory.Registry.INSTANCE.put
  (oclDelegateURI, new OCLQueryDelegateFactory.Global());
Comment 3 Axel Uhl CLA 2011-06-06 02:40:08 EDT 
I see. Probably a post-Helios change since this clearly changes initialize()'s behavior.
Comment 4 Ed Willink CLA 2011-06-06 03:31:16 EDT 
(In reply to comment #3)
> I see. Probably a post-Helios change since this clearly changes initialize()'s
> behavior.

Absolutely. (post-Indigo).
Comment 5 Ed Willink CLA 2011-11-08 13:38:47 EST 
There are now OCLDelegateDomain.initialize routines to initialize delegates. With code generation rendering all bar Query delegates obsolete, there seems little point enforcing unnecessary initialization.
Comment 6 Ed Willink CLA 2013-05-20 11:37:31 EDT 
CLOSED after a year in the RESOLVED state.