oship-dev team mailing list archive

[Wagner Mezaroba <wagnerfrancisco@xxxxxxxxx>]

I have one question about this bug. How are we going to deal with the
parent attribute? We don't have a parent attribute in the pathable or
locatable classes, so it's not possible just to pass the attribute using
a constructor. Should we set this attribute in each class? Or should we
create this attribute in the Pathable class?

The Event Class Specification defines that we should redefine the parent
attribute to be of History type. So shouldn't we keep the parent
attribute in the Event interface? Just to ensure its type.

-- 
some issues in class Event
https://bugs.launchpad.net/bugs/608831
You received this bug notification because you are a member of OSHIP
Development Team, which is subscribed to Open Source Health Information
Platform.

Status in Open Source Health Information Platform (OSHIP): Opinion

Bug description:
Method offset is not implemented. It should return a DvDuration corresponding to the difference between the value of attribute time and the value of attribute origin (obtained from the parent History).

There are some arguments missing in its initializer (those inherited from Locatable). It is also necessary to call the superclass initializer.

The method parent should be defined in class Pathable, not here. The same could be said about IEvent interface.