Issue 992 in moose-technology: FMNullMultivalueLink does not support union: (FMNullMultivalueLink cannot have variable sized instances)

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 992 by [hidden email]: 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

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Comment #1 on issue 992 by [hidden email]: 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

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Comment #2 on issue 992 by [hidden email]: 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.

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev

Updates:
        Status: Fixed
        Labels: Milestone-5.0 Component-Fame

Comment #3 on issue 992 by [hidden email]: 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.

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev