oship-dev team mailing list archive
-
oship-dev team
-
Mailing list archive
-
Message #01765
[Bug 610533] Re: some issues in DvOrdinal
** Project changed: oship => oshippy
** Changed in: oshippy
Milestone: 1.5 => None
--
You received this bug notification because you are a member of OSHIP
Development Team, which is subscribed to Open Source Health Information
Platform.
https://bugs.launchpad.net/bugs/610533
Title:
some issues in DvOrdinal
Status in Open Source Health Information Platform - Python:
New
Bug description:
Is it necessary to test the invariant below in the constructor?
if symbol is None or isinstance(symbol,DvCodedText) == False:
raise ValueError(_("Invalid DvOrdinal.symbol"))
else:
self.symbol=symbol
The invariant below is not enforced:
Limits_valid: limits /= Void and then limits.meaning.is_equal(“limits”)
Reference_range_valid: other_reference_ranges /= Void and then other_reference_ranges.has(limits)
and then the method limits is not correctly implemented: it should
have no arguments and should return a ReferenceRange object
corresponding to an element of otherReferenceRanges attribute whose
meaning is "limits".
The method compare (<) is not implemented:
it should have a DvOrdinal argument and return 1, 0 and -1 according to the comparisons of the corresponding value attributes.
The method isStrictlyComparableTo is not correclty implemented:
In java, they returned true if the symbols came from the same terminology. Maybe this is the maximum we can get without more information.
References