• Jean-Paul Smets's avatar
    The implementation of "Accessor overloading" was not done properly. First of... · 846ba5e6
    Jean-Paul Smets authored
    The implementation of "Accessor overloading" was not done properly. First of all, there is no need to overload public setters, because the Alias.Reindex is able to use the private ones. So overloading the private ones is enough. Second, it is not appriopriate to reimplement the way data is set or accessed on objects. Instead of setting title with self.title = value, one should call self._baseSetTitle(value) which does much more, including casting data types and potentially storing values externally.
    
    git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@22573 20353a03-c40f-0410-a6d1-a30d3c3de9de
    846ba5e6
Person.py 9.03 KB