oship-dev team mailing list archive
-
oship-dev team
-
Mailing list archive
-
Message #01229
[Bug 610533] [NEW] some issues in DvOrdinal
Public bug reported:
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.
** Affects: oship
Importance: High
Status: New
** Changed in: oship
Importance: Undecided => High
** Changed in: oship
Milestone: None => 1.5
--
some issues in DvOrdinal
https://bugs.launchpad.net/bugs/610533
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): 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.
Follow ups
References