16 Mar
2012
16 Mar
'12
12:43 p.m.
I was wondering why camillo did the changes but this is good change since now we can do union on collection and not only set.
Stef
More precisely the crashes are due to the changes in the Collection>>union:
The original definition was: union: aCollection ^ self asSet addAll: aCollection; yourself
The new definition is: union: aCollection | set | set := self asSet addAll: aCollection; yourself. ^ self class withAll: set asArray
This breaks the assumptions behind FMMultivalueLink because although FMMultivalueLink is a subclass of Collection, it is not supposed to be created with a "withAll:" constructor.