Useless class "TimePeriodOfDay"

Submitted by Joerg Freudenstein on Tuesday, 3 June, 2014 - 16:47
Issue ID
151
Component
UML General
Category
Feature request
Priority
Normal
Assigned
Status
Fixed
Source
Website
Description

In the validty model, the class "TimePeriodOfDay" is an aggregation to the "Period" class.
The only possibility is to specialise this class by "TimePeriodOfDay", there are no other links or attributes.

I'm wondering, if this class cannot be removed to introduce a direct aggregation between "Period" and "TimePeriodByHour".

Found Version
{"changeLogs":[{"date":1528373460379,"componentOLD":"- Select a value -","component":"UML General","categoryOLD":"- Select a value -","category":"Feature request","priorityOLD":"- None -","priority":"Normal","assignedOLD":"","assigned":"iancornwell (42)","statusOLD":"- None -","status":"Fixed"},{"date":1537268798274}]}

Posted by iancornwell on November 24, 2023 Permalink

Agree it is currently useless, but it is the right name for the role of TimePeriodByHour, so the fix should probably be to move the attributes into TimePeriodOfDay, make that concrete, and remove the ByHour class.

Posted by iancornwell on November 24, 2022 Permalink

Done in line with comment #1 above as agreed by TG. This meant that only one class was left in package TimePeriodOfDay, so moved the class to Validity and deleted the package TimePeriodOfDay.