← Back to team overview

oship-dev team mailing list archive

[Bug 606383] Re: seemingly unnecessary assignments in the initializer of class Cluster

 

** Project changed: oship => oshippy

-- 
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/606383

Title:
  seemingly unnecessary assignments in the initializer of class Cluster

Status in Open Source Health Information Platform - Python:
  Fix Released

Bug description:
   The following assignments in the initializer of class Cluster seems unnecessary since they are done in the initializer of the superclass, aren't they?
         self.uid = uid
          self.atnodeid = archetypeNodeId
          self.name = name
          self.atdetails = archetypeDetails
          self.fdraudit = feederAudit
          self.parent = parent

  It seems that only self.items should be retained.

  I think we should be consistent in the call to the superclass initializer: 
  Compare the call    super(Cluster, self).__init__(uid, archetypeNodeId, name, archetypeDetails, feederAudit, links, parent)
  in class Cluster 
  with the call  Locatable.__init__(self, uid, archetypeNodeId, name, archetypeDetails, feederAudit, links)
  in class Item.





References