oship-dev team mailing list archive

[Sergio Miranda Freire <sergio@xxxxxxxxxxxxxxx>]

Public bug reported:

DvParsable initializer is a bit confusing!
Look at its initializer:
    def __init__(self,value,formalism,size,charset,language):
        self.value=value
        size=len(value)
        self.size=size
        self.formalism=formalism
        DvEncapsulated.__init__(self,size,charset,language)

It sets the value of size, then the makes self.size = size and calls
superclass initializer, which will make self.size = size again.

** Affects: oship
     Importance: Low
         Status: New

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

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

-- 
DvParsable initializer is a bit confusing!
https://bugs.launchpad.net/bugs/610162
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:
DvParsable initializer is a bit confusing!
Look at its initializer:
    def __init__(self,value,formalism,size,charset,language):
        self.value=value
        size=len(value)
        self.size=size
        self.formalism=formalism
        DvEncapsulated.__init__(self,size,charset,language)

It sets the value of size, then the makes self.size = size and calls superclass initializer, which will make self.size = size again.