Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 348334

Summary:	OCL.initialize is inadequate
Product:	[Modeling] OCL	Reporter:	Ed Willink <ed>
Component:	Core	Assignee:	OCL Inbox <mdt-ocl-inbox>
Status:	CLOSED WONTFIX	QA Contact:
Severity:	normal
Priority:	P3	CC:	eclipse
Version:	3.1.0
Target Milestone:	---
Hardware:	PC
OS:	Windows Vista
Whiteboard:

Description Ed Willink

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

2011-06-05 15:50:21 EDT

Ed,

what exactly do you mean with "ignores delegates?"

Comment 2 Ed Willink

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

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

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

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

2013-05-20 11:37:31 EDT

CLOSED after a year in the RESOLVED state.