Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 573836 - [oclinecore] Revisit multiplicity default
Summary: [oclinecore] Revisit multiplicity default
Status: RESOLVED FIXED
Alias: None
Product: OCL
Classification: Modeling
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: OCL Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-05-28 09:30 EDT by Ed Willink CLA
Modified: 2021-08-29 06:30 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ed Willink CLA 2021-05-28 09:30:09 EDT
With may-be-invalid analysis the null-by-default multiplicity is a real pain.

The OCLinEcore serialization culd be changed to default to [1], just like UML, but unlike Ecore, in text witjput affecting Ecore.
Comment 1 Ed Willink CLA 2021-06-02 00:14:48 EDT
The OCLinEcore tutorial states that OCLinEcore follows UML rather than Ecore default multiplicity.
Comment 2 Ed Willink CLA 2021-07-06 02:13:01 EDT
(In reply to Ed Willink from comment #0)
> With may-be-invalid analysis the null-by-default multiplicity is a real pain.

For *.ecore a change is not really a problem since the *.ecore setting is definitive; it's just a matter of which polarity is implicit in textual form.

For Complete OCL (or OCLinEcore) source the default seems to guarantee may-be-invalid. Really tedious for Complete OCL tests.
Comment 3 Ed Willink CLA 2021-07-06 02:45:12 EDT
testLoad_Bug402767_oclinecore written in 2013 tests for the original 'UML' behaviour.

https://bugs.eclipse.org/bugs/show_bug.cgi?id=464927#c5 motivates the change for maximum compatibility in 2015.

Since [1..1] is potentially breaking we will need a preference.
Comment 4 Ed Willink CLA 2021-07-12 11:42:37 EDT
Preference with changed default pushed to master for M1,
Comment 5 Eclipse Genie CLA 2021-08-29 06:30:32 EDT
New Gerrit change created: https://git.eclipse.org/r/c/ocl/org.eclipse.ocl/+/184577