Status: New Owner: ---- Labels: Type-Defect Priority-Medium
New issue 992 by anne.et...@gmail.com: FMNullMultivalueLink does not support union: (FMNullMultivalueLink cannot have variable sized instances) http://code.google.com/p/moose-technology/issues/detail?id=992
Describe the problem: what do you get? what do you expect? Here is the code that we want to execute to combine in a same collection of a given class its incomingReferences and its superInheritances: (aClass incomingReferences) union: (aClass superInheritances)
If the first set of the union operator is empty, it is a FMNullMultivalueLink. If the second set is not empty, the union: message creates a FMNullMultivalueLink with the size of this second set. However, it is not possible to create a FMNullMultivalueLink with a specified size. We would like that its creates a FMMultivalueLink with the specified size.
Please fill in the labels with the following information: * Type-Defect, Type-Enhancement, Type-Engineering, Type-Review, Type-Other * Component-MooseCore
Comment #1 on issue 992 by anne.et...@gmail.com: FMNullMultivalueLink does not support union: (FMNullMultivalueLink cannot have variable sized instances) http://code.google.com/p/moose-technology/issues/detail?id=992
By adding back FMNullMultiValueLink>>species ^OrderedCollection
it works and all the tests are green. In fact I previously removed this method at the beginning of September. But I found today a case where it is useful so I put it again.
If you agree with this change, the issue can be closed.
Anne
Comment #2 on issue 992 by step...@stack.nl: FMNullMultivalueLink does not support union: (FMNullMultivalueLink cannot have variable sized instances) http://code.google.com/p/moose-technology/issues/detail?id=992
Sounds good. Might be even better with a few tests.
Updates: Status: Fixed Labels: Milestone-5.0 Component-Fame
Comment #3 on issue 992 by tu...@tudorgirba.com: FMNullMultivalueLink does not support union: (FMNullMultivalueLink cannot have variable sized instances) http://code.google.com/p/moose-technology/issues/detail?id=992
Thanks for fixing this.