From c538477ffe4461f803c5e6562547a16d3a9092d5 Mon Sep 17 00:00:00 2001 From: Don Allingham Date: Thu, 7 Apr 2005 00:51:26 +0000 Subject: [PATCH] * src/FamilyView.py: keep track of local person during update * src/PedView.py: update database pointer when database changed, update active person * src/gramps_main.py: keep track of active-changed svn: r4316 --- gramps2/ChangeLog | 6 +++ gramps2/src/FamilyView.py | 32 ++++++++------- gramps2/src/PedView.py | 81 ++++++++++++++++++++++---------------- gramps2/src/PeopleView.py | 10 +++-- gramps2/src/gramps_main.py | 15 ++++--- 5 files changed, 88 insertions(+), 56 deletions(-) diff --git a/gramps2/ChangeLog b/gramps2/ChangeLog index 9c8d89399..3135e6d14 100644 --- a/gramps2/ChangeLog +++ b/gramps2/ChangeLog @@ -1,3 +1,9 @@ +2005-04-06 Don Allingham + * src/FamilyView.py: keep track of local person during update + * src/PedView.py: update database pointer when database changed, update + active person + * src/gramps_main.py: keep track of active-changed + 2005-04-06 Martin Hawlisch * src/gramps.glade: People View: Add label for filter text input field. * src/GenericFilter.py (ProbablyAlive): Add check for input parameter, diff --git a/gramps2/src/FamilyView.py b/gramps2/src/FamilyView.py index 03e559f60..c5c8069ea 100644 --- a/gramps2/src/FamilyView.py +++ b/gramps2/src/FamilyView.py @@ -852,34 +852,38 @@ class FamilyView: if self.person == None: return - if self.selected_spouse.get_handle() == self.family.get_father_handle(): - self.family.set_father_handle(None) + cur_person = self.person + cur_spouse = self.selected_spouse + cur_family = self.family + + if cur_spouse.get_handle() == cur_family.get_father_handle(): + cur_family.set_father_handle(None) else: - self.family.set_mother_handle(None) + cur_family.set_mother_handle(None) trans = self.parent.db.transaction_begin() - if self.selected_spouse: - self.selected_spouse.remove_family_handle(self.family.get_handle()) - self.parent.db.commit_person(self.selected_spouse,trans) + if cur_spouse: + cur_spouse.remove_family_handle(cur_family.get_handle()) + self.parent.db.commit_person(cur_spouse,trans) - self.parent.db.commit_family(self.family,trans) + self.parent.db.commit_family(cur_family,trans) - if len(self.family.get_child_handle_list()) == 0: - mother_id = self.family.get_mother_handle() - father_id = self.family.get_father_handle() + if len(cur_family.get_child_handle_list()) == 0: + mother_id = cur_family.get_mother_handle() + father_id = cur_family.get_father_handle() for handle in [father_id, mother_id]: if handle: p = self.parent.db.get_person_from_handle(handle) - p.remove_family_handle(self.family.get_handle()) + p.remove_family_handle(cur_family.get_handle()) self.parent.db.commit_person(p,trans) - if len(self.person.get_family_handle_list()) > 0: - handle = self.person.get_family_handle_list()[0] + if len(cur_person.get_family_handle_list()) > 0: + handle = cur_person.get_family_handle_list()[0] family = self.parent.db.find_family_from_handle(handle,trans) - person_id = self.person.get_handle() + person_id = cur_person.get_handle() self.person = self.parent.db.get_person_from_handle(person_id) n = self.person.get_primary_name().get_regular_name() self.parent.db.transaction_commit(trans,_("Remove Spouse (%s)") % n) diff --git a/gramps2/src/PedView.py b/gramps2/src/PedView.py index 08cf99d1b..d74213364 100644 --- a/gramps2/src/PedView.py +++ b/gramps2/src/PedView.py @@ -165,15 +165,15 @@ class PedigreeView: def __init__(self,parent,canvas,update,status_bar,lp): self.parent = parent + self.relcalc = Relationship.RelationshipCalculator(self.parent.db) self.parent.connect('database-changed',self.change_db) + self.parent.connect('active-changed',self.active_changed) self.canvas = canvas self.canvas_items = [] self.boxes = [] self.root = self.canvas.root() self.active_person = None - r = Relationship.RelationshipCalculator(self.parent.db) - self.distance = r.get_relationship_distance self.x1 = 0 self.x2 = 0 self.y1 = 0 @@ -183,17 +183,31 @@ class PedigreeView: self.load_person = lp self.anchor = None self.canvas.parent.connect('button-press-event',self.on_canvas_press) + self.change_db(self.parent.db) + self.distance = self.relcalc.get_relationship_distance def change_db(self,db): # Reconnect signals + self.db = db db.connect('person-add', self.person_updated_cb) db.connect('person-update', self.person_updated_cb) db.connect('person-delete', self.person_updated_cb) - db.connect('person-rebuild', self.person_updated_cb) + db.connect('person-rebuild', self.person_rebuild) + self.relcalc.set_db(db) + self.active_person = None - def person_updated_cb(self,handle_list=None): - # Redraw view on changes of persons + def person_updated_cb(self,handle_list): self.load_canvas(self.active_person) + + def person_rebuild(self): + self.load_canvas(self.active_person) + + def active_changed(self,handle): + if handle: + self.active_person = self.db.get_person_from_handle(handle) + self.load_canvas(self.active_person) + else: + self.load_canvas(None) def clear(self): for i in self.canvas_items: @@ -231,7 +245,7 @@ class PedigreeView: for t in lst: if t: - boxtext = build_detail_string(self.parent.db,t[0]).encode("UTF-8") + boxtext = build_detail_string(self.db,t[0]).encode("UTF-8") for line in boxtext.split("\n"): try: a.set_text(line,len(line)) @@ -266,7 +280,7 @@ class PedigreeView: self.canvas_items.append(self.anchor_txt) for family_handle in self.active_person.get_family_handle_list(): - family = self.parent.db.get_family_from_handle(family_handle) + family = self.db.get_family_from_handle(family_handle) if len(family.get_child_handle_list()) > 0: button,arrow = self.make_arrow_button(gtk.ARROW_LEFT, self.on_show_child_menu) @@ -321,11 +335,10 @@ class PedigreeView: p[1]) p = lst[i] box = DispBox(self.root,style,xpts[i],ypts[i],w,h,p[0], - self.parent.db, + self.db, self.parent.change_active_person, self.load_person, self.build_full_nav_menu) self.boxes.append(box) - self.parent.change_active_person(person) def make_arrow_button(self,direction,function): """Make a button containing an arrow with the attached callback""" @@ -373,18 +386,18 @@ class PedigreeView: # button. The menu consists of the children of the current root # person of the tree. Attach a child to each menu item. - childlist = find_children(self.parent.db,self.active_person) + childlist = find_children(self.db,self.active_person) if len(childlist) == 1: - child = self.parent.db.get_person_from_handle(childlist[0]) + child = self.db.get_person_from_handle(childlist[0]) if child: - self.load_canvas(child) + self.parent.change_active_person(child) elif len(childlist) > 1: myMenu = gtk.Menu() for child_handle in childlist: - child = self.parent.db.get_person_from_handle(child_handle) + child = self.db.get_person_from_handle(child_handle) cname = NameDisplay.displayer.display(child) menuitem = gtk.MenuItem(None) - if find_children(self.parent.db,child): + if find_children(self.db,child): label = gtk.Label('%s' % cname) else: label = gtk.Label(cname) @@ -404,9 +417,9 @@ class PedigreeView: attached with menu item.""" person_handle = obj.get_data(_PERSON) - person = self.parent.db.get_person_from_handle(person_handle) + person = self.db.get_person_from_handle(person_handle) if person: - self.load_canvas(person) + self.parent.change_active_person(person) return 1 def add_parent_button(self,parent,x,y,h): @@ -430,10 +443,10 @@ class PedigreeView: attached to the button and change the root person to that person, redrawing the view.""" person_handle = obj.get_data(_PERSON) - person = self.parent.db.get_person_from_handle(person_handle) + person = self.db.get_person_from_handle(person_handle) if self.active_person: self.active_person = person - self.load_canvas(person) + self.parent.change_active_person(person) def draw_canvas_line(self,x1,y1,x2,y2,h,w,data,style,ls): """Draw an two segment line between the x,y point pairs. Attach @@ -466,12 +479,14 @@ class PedigreeView: """Catch X events over a line and respond to the ones we care about""" person_handle = obj.get_data(_PERSON) - person = self.parent.db.get_person_from_handle(person_handle) + if not person_handle: + return + person = self.db.get_person_from_handle(person_handle) style = self.canvas.get_style() if event.type == gtk.gdk._2BUTTON_PRESS: if event.button == 1 and event.type == gtk.gdk._2BUTTON_PRESS: - self.load_canvas(person) + self.parent.change_active_person(person) elif event.type == gtk.gdk.ENTER_NOTIFY: obj.set(fill_color_gdk=style.bg[gtk.STATE_SELECTED], width_pixels=4) @@ -498,15 +513,15 @@ class PedigreeView: mrel = m != RelLib.Person.CHILD_REL_BIRTH frel = f != RelLib.Person.CHILD_REL_BIRTH - family = self.parent.db.get_family_from_handle(family_handle) + family = self.db.get_family_from_handle(family_handle) if family != None: father_handle = family.get_father_handle() if father_handle != None: - father = self.parent.db.get_person_from_handle(father_handle) + father = self.db.get_person_from_handle(father_handle) self.find_tree(father,(2*index)+1,depth+1,lst,frel) mother_handle = family.get_mother_handle() if mother_handle != None: - mother = self.parent.db.get_person_from_handle(mother_handle) + mother = self.db.get_person_from_handle(mother_handle) self.find_tree(mother,(2*index)+2,depth+1,lst,mrel) def on_canvas1_event(self,obj,event): @@ -582,12 +597,12 @@ class PedigreeView: fam_list = person.get_family_handle_list() no_spouses = 1 for fam_id in fam_list: - family = self.parent.db.get_family_from_handle(fam_id) + family = self.db.get_family_from_handle(fam_id) if family.get_father_handle() == person.get_handle(): sp_id = family.get_mother_handle() else: sp_id = family.get_father_handle() - spouse = self.parent.db.get_person_from_handle(sp_id) + spouse = self.db.get_person_from_handle(sp_id) if not spouse: continue @@ -613,12 +628,12 @@ class PedigreeView: pfam_list = person.get_parent_family_handle_list() no_siblings = 1 for (f,mrel,frel) in pfam_list: - fam = self.parent.db.get_family_from_handle(f) + fam = self.db.get_family_from_handle(f) sib_list = fam.get_child_handle_list() for sib_id in sib_list: if sib_id == person.get_handle(): continue - sib = self.parent.db.get_person_from_handle(sib_id) + sib = self.db.get_person_from_handle(sib_id) if not sib: continue @@ -641,9 +656,9 @@ class PedigreeView: # Go over children and build their menu item = gtk.MenuItem(_("Children")) no_children = 1 - childlist = find_children(self.parent.db,person) + childlist = find_children(self.db,person) for child_handle in childlist: - child = self.parent.db.get_person_from_handle(child_handle) + child = self.db.get_person_from_handle(child_handle) if not child: continue @@ -652,7 +667,7 @@ class PedigreeView: item.set_submenu(gtk.Menu()) child_menu = item.get_submenu() - if find_children(self.parent.db,child): + if find_children(self.db,child): label = gtk.Label('%s' % NameDisplay.displayer.display(child)) else: label = gtk.Label(NameDisplay.displayer.display(child)) @@ -675,9 +690,9 @@ class PedigreeView: # Go over parents and build their menu item = gtk.MenuItem(_("Parents")) no_parents = 1 - par_list = find_parents(self.parent.db,person) + par_list = find_parents(self.db,person) for par_id in par_list: - par = self.parent.db.get_person_from_handle(par_id) + par = self.db.get_person_from_handle(par_id) if not par: continue @@ -686,7 +701,7 @@ class PedigreeView: item.set_submenu(gtk.Menu()) par_menu = item.get_submenu() - if find_parents(self.parent.db,par): + if find_parents(self.db,par): label = gtk.Label('%s' % NameDisplay.displayer.display(par)) else: label = gtk.Label(NameDisplay.displayer.display(par)) diff --git a/gramps2/src/PeopleView.py b/gramps2/src/PeopleView.py index 06712b7b5..efe84a795 100644 --- a/gramps2/src/PeopleView.py +++ b/gramps2/src/PeopleView.py @@ -281,17 +281,21 @@ class PeopleView: sel_sensitivity = 1 else: sel_sensitivity = 0 + merge_sensitivity = len(mlist) == 2 entries = [ (gtk.STOCK_GO_BACK,self.parent.back_clicked,back_sensitivity), (gtk.STOCK_GO_FORWARD,self.parent.fwd_clicked,fwd_sensitivity), - #FIXME: revert to stock item when German gtk translation is fixed - #(gtk.STOCK_HOME,self.parent.on_home_clicked,1), - (_("Home"),self.parent.on_home_clicked,1), + (gtk.STOCK_HOME,self.parent.on_home_clicked,1), (_("Add Bookmark"),self.parent.on_add_bookmark_activate,sel_sensitivity), (None,None,0), (gtk.STOCK_ADD, self.parent.add_button_clicked,1), (gtk.STOCK_REMOVE, self.parent.remove_button_clicked,sel_sensitivity), (_("Edit"), self.parent.edit_button_clicked,sel_sensitivity), + #(None,None,0), + #(_("Compare and Merge"), self.parent.on_merge_activate, + # merge_sensitivity), + #(_("Fast Merge"), self.parent.on_fast_merge_activate, + # merge_sensitivity), ] menu = gtk.Menu() diff --git a/gramps2/src/gramps_main.py b/gramps2/src/gramps_main.py index 10795b8b2..4817cb669 100755 --- a/gramps2/src/gramps_main.py +++ b/gramps2/src/gramps_main.py @@ -110,6 +110,7 @@ class Gramps(GrampsDBCallback.GrampsDBCallback): __signals__ = { 'database-changed' : (GrampsDbBase.GrampsDbBase,), + 'active-changed' : (str,), } def __init__(self,args): @@ -1451,6 +1452,7 @@ class Gramps(GrampsDBCallback.GrampsDBCallback): self.set_buttons(0) self.active_person = None self.modify_statusbar() + self.emit('active-changed',(None,)) elif (self.active_person == None or person.get_handle() != self.active_person.get_handle()): self.active_person = self.db.get_person_from_handle(person.get_handle()) @@ -1478,9 +1480,11 @@ class Gramps(GrampsDBCallback.GrampsDBCallback): else: self.backbtn.set_sensitive(0) self.back.set_sensitive(0) + self.emit('active-changed',(self.active_person.get_handle(),)) else: self.active_person = self.db.get_person_from_handle(person.get_handle()) self.set_buttons(1) + self.emit('active-changed',(self.active_person.get_handle(),)) def modify_statusbar(self): @@ -1497,13 +1501,13 @@ class Gramps(GrampsDBCallback.GrampsDBCallback): def display_relationship(self): default_person = self.db.get_default_person() - if not default_person: + active = self.active_person + if default_person == None or active == None: return u'' try: pname = NameDisplay.displayer.display(default_person) (name,plist) = self.relationship.get_relationship( - default_person, - self.active_person) + default_person,active) if name: if plist == None: @@ -1671,12 +1675,11 @@ class Gramps(GrampsDBCallback.GrampsDBCallback): GrampsKeys.save_last_file(name) self.gtop.get_widget("filter").set_text("") - self.emit("database-changed", (self.db,)) - self.relationship = self.RelClass(self.db) + self.emit("database-changed", (self.db,)) self.change_active_person(self.find_initial_person()) - self.goto_active_person() # TODO: This should emit a signal so other views can update itself + self.goto_active_person() if callback: callback(_('Setup complete'))