-
Notifications
You must be signed in to change notification settings - Fork 2
Open
Labels
Area: MappingRelates to one of the `ITopicMappingService` interfaces or implementations.Relates to one of the `ITopicMappingService` interfaces or implementations.Area: RepositoriesRelates to the `ITopicRepository` interface or one of its implementations.Relates to the `ITopicRepository` interface or one of its implementations.Priority: 2Severity 2: MajorStatus 0: DiscussionNeeds further evaluation of requirements and prioritization.Needs further evaluation of requirements and prioritization.Type: ImprovementImproves the functionality or interface of an existing feature.Improves the functionality or interface of an existing feature.
Milestone
Description
In Section 4.3 of The Framework Design Guidelines, the authors make a compelling argument for using base classes over interfaces for cases where polymorphism or multi inheritance aren’t needed. This applies effectively to e.g., ITopicRepository
(which already has a base class) and possibly the TopicMappingService
s.
If so, should we consider changing this for OnTopic 6.x? This would be a major breaking change, but it would also afford us a more stable foundation to build off of in the future.
Metadata
Metadata
Assignees
Labels
Area: MappingRelates to one of the `ITopicMappingService` interfaces or implementations.Relates to one of the `ITopicMappingService` interfaces or implementations.Area: RepositoriesRelates to the `ITopicRepository` interface or one of its implementations.Relates to the `ITopicRepository` interface or one of its implementations.Priority: 2Severity 2: MajorStatus 0: DiscussionNeeds further evaluation of requirements and prioritization.Needs further evaluation of requirements and prioritization.Type: ImprovementImproves the functionality or interface of an existing feature.Improves the functionality or interface of an existing feature.