← Back to team overview

oship-dev team mailing list archive

[Bug 608831] Re: some issues in class Event

 

On Fri, 2010-07-30 at 13:42 +0000, Sergio Miranda Freire wrote:
Parent may be defined as an attribute or a method according to the specifications. In a previous talk with Diego, we decided that we would implement it as a method in OSHIP. 
> Then we can use the Zope machinery to return for method parent the value of the __parent__ attribute, and set in the class the __parent__ attribute to the correct value at or after the initialization of an object, depending on the context.
> 
So in this decision then; *all* instances where the 'parent' attribute is addressed it must be removed. Also the parent() and name() methods need to be defined in Pathable. 

I will file bug reports for the mlhim and openehr models.  I will fix
the mlhim model.

-- 
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.





References