← Back to team overview

oship-dev team mailing list archive

[Bug 603694] [NEW] Method hasExtension in UidBasedId seems to be not correctly implemented.

 

Public bug reported:

As it is now, _hasExtension receives true in the initialization of
org.openehr.rm.support.identification and eventually only changes its
value if method extension is called. This way when _hasExtension is
called without a call to extension before, it may return the wrong
value.

My suggestion is to remove the line self._hasExtension = True in the
initializer and implement the method hasExtension according to the
suggestion in bug 603688.

** Affects: oship
     Importance: High
         Status: New

** Changed in: oship
   Importance: Undecided => High

** Changed in: oship
    Milestone: None => 1.5

-- 
Method hasExtension in UidBasedId seems to be not correctly implemented.
https://bugs.launchpad.net/bugs/603694
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:
As it is now, _hasExtension receives true in the initialization of org.openehr.rm.support.identification and eventually only changes its value if method extension is called. This way when _hasExtension is called without a call to extension before, it may return the wrong value.

My suggestion is to remove the line self._hasExtension = True in the initializer and implement the method hasExtension according to the suggestion in bug 603688.





Follow ups

References