Cyclone-Team/gramps
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Make DbWriteBase inherit from DbReadBase to increase pylint score

Browse Source 
svn: r16913
This commit is contained in:
Michiel Nauta 2011-03-24 21:38:04 +00:00
parent d79b17c10d
commit caec97cb7f
1 changed files with 11 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/gen/db/base.py
View File

@ -1056,7 +1056,8 @@ class DbReadBase(object):
""" """
raise NotImplementedError raise NotImplementedError
def set_prefixes(self, person, media, family, source, place, event, repository, note): def set_prefixes(self, person, media, family, source, place, event,
repository, note):
""" """
Set the prefixes for the gramps ids for all gramps objects Set the prefixes for the gramps ids for all gramps objects
""" """
@ -1132,7 +1133,7 @@ class DbReadBase(object):
""" """
raise NotImplementedError raise NotImplementedError
class DbWriteBase(object): class DbWriteBase(DbReadBase):
""" """
GRAMPS database object. This object is a base class for all GRAMPS database object. This object is a base class for all
database interfaces. All methods raise NotImplementedError database interfaces. All methods raise NotImplementedError
@ -1146,6 +1147,7 @@ class DbWriteBase(object):
A new DbWriteBase class should never be directly created. Only classes A new DbWriteBase class should never be directly created. Only classes
derived from this class should be created. derived from this class should be created.
""" """
DbReadBase.__init__(self)
def add_event(self, event, transaction, set_gid=True): def add_event(self, event, transaction, set_gid=True):
""" """
@ -1597,7 +1599,8 @@ class DbWriteBase(object):
# loop through the family list # loop through the family list
for family_handle in person.get_family_handle_list(): for family_handle in person.get_family_handle_list():
if not family_handle: continue if not family_handle:
continue
family = self.get_family_from_handle(family_handle) family = self.get_family_from_handle(family_handle)
@ -1606,7 +1609,8 @@ class DbWriteBase(object):
else: else:
family.set_mother_handle(None) family.set_mother_handle(None)
if not family.get_father_handle() and not family.get_mother_handle() and \ if not family.get_father_handle() and \
not family.get_mother_handle() and \
not family.get_child_ref_list(): not family.get_child_ref_list():
self.remove_family_relationships(family_handle, trans) self.remove_family_relationships(family_handle, trans)
else: else:
@ -1630,9 +1634,9 @@ class DbWriteBase(object):
self.find_backlink_handles(handle,['Person'])] self.find_backlink_handles(handle,['Person'])]
for phandle in person_list: for phandle in person_list:
p = self.get_person_from_handle(phandle) prsn = self.get_person_from_handle(phandle)
p.remove_handle_references('Person', [handle]) prsn.remove_handle_references('Person', [handle])
self.commit_person(p, trans) self.commit_person(prsn, trans)
self.remove_person(handle, trans) self.remove_person(handle, trans)
def remove_family_relationships(self, family_handle, trans=None): def remove_family_relationships(self, family_handle, trans=None):