-
Notifications
You must be signed in to change notification settings - Fork 2
MemberDispatcher: Support Nullable Types #43
Copy link
Copy link
Closed
Labels
Area: MappingRelates to one of the `ITopicMappingService` interfaces or implementations.Relates to one of the `ITopicMappingService` interfaces or implementations.Priority: 3Severity 3: BlockingStatus 5: CompleteTask is considered complete, and ready for deployment.Task is considered complete, and ready for deployment.Type: ImprovementImproves the functionality or interface of an existing feature.Improves the functionality or interface of an existing feature.
Milestone
Metadata
Metadata
Assignees
Labels
Area: MappingRelates to one of the `ITopicMappingService` interfaces or implementations.Relates to one of the `ITopicMappingService` interfaces or implementations.Priority: 3Severity 3: BlockingStatus 5: CompleteTask is considered complete, and ready for deployment.Task is considered complete, and ready for deployment.Type: ImprovementImproves the functionality or interface of an existing feature.Improves the functionality or interface of an existing feature.
Currently, strings can be set to
null, but other potentially nullable types cannot—including references, as well as value types such asint?orbool?. In this case, the call toSetPropertyValue()will be ignored entirely, and any preexisting value will be retained. This could result in unexpected, if not buggy, behavior. To remedy this, ideally theMemberDispatchershould be able to detect nullable types and, if a type is nullable, then permit the null value. If the type is not nullable, we should consider falling back todefault.