From 88e5406249fd7746dc0b2a05eb0176d296a77e1a Mon Sep 17 00:00:00 2001
From: Martin Hawlisch <martin.hawlisch@gmx.de>
Date: Mon, 11 Apr 2005 10:11:04 +0000
Subject: [PATCH] * src/plugins/Check.py: Fix events, that reference non
 existing places

svn: r4342
---
 gramps2/ChangeLog            |  2 ++
 gramps2/src/plugins/Check.py | 21 ++++++++++++++++++++-
 2 files changed, 22 insertions(+), 1 deletion(-)

diff --git a/gramps2/ChangeLog b/gramps2/ChangeLog
index a55aa4e9e..d9c7f89a9 100644
--- a/gramps2/ChangeLog
+++ b/gramps2/ChangeLog
@@ -1,6 +1,8 @@
 2005-04-11  Martin Hawlisch  <Martin.Hawlisch@gmx.de>
 	* src/PedView.py (load_canvas): Catch bug of not existing family
 
+	* src/plugins/Check.py: Fix events, that reference non existing places
+
 2005-04-10  Alex Roitman  <shura@gramps-project.org>
 	* Release: Version 1.1.95 "When danger reared its ugly head,
 	he bravely turned his tail and fled" released.
diff --git a/gramps2/src/plugins/Check.py b/gramps2/src/plugins/Check.py
index 4572649d8..a6866f70e 100644
--- a/gramps2/src/plugins/Check.py
+++ b/gramps2/src/plugins/Check.py
@@ -66,6 +66,7 @@ def runTool(database,active_person,callback,parent=None):
         checker.check_parent_relationships()
         checker.cleanup_empty_families(0)
         checker.check_events()
+        checker.check_place_references()
         database.transaction_commit(trans, _("Check Integrity"))
         database.enable_signals()
         database.request_rebuild()
@@ -98,6 +99,7 @@ class CheckIntegrity:
         self.invalid_events = []
         self.invalid_birth_events = []
         self.invalid_death_events = []
+        self.invalid_place_references = []
 
     def check_for_broken_family_links(self):
         self.broken_links = []
@@ -380,6 +382,18 @@ class CheckIntegrity:
                         self.db.commit_person(person,self.trans)
                         self.invalid_events.append(key)
 
+    def check_place_references(self):
+        for key in self.db.get_event_handles():
+            event = self.db.get_event_from_handle(key)
+            place_handle = event.get_place_handle()
+            if place_handle:
+                place = self.db.get_place_from_handle(place_handle)
+                if not place:
+                    # The referenced place does not exist in the database
+                    event.set_place_handle("")
+                    self.db.commit_event(event,self.trans)
+                    self.invalid_place_references.append(key)
+
     def build_report(self,cl=0):
         bad_photos = len(self.bad_photo)
         replaced_photos = len(self.replaced_photo)
@@ -393,8 +407,9 @@ class CheckIntegrity:
         birth_invalid = len(self.invalid_birth_events)
         death_invalid = len(self.invalid_death_events)
         person = birth_invalid + death_invalid
+        place_references = len(self.invalid_place_references)
 
-        errors = blink + efam + photos + rel + person + event_invalid
+        errors = blink + efam + photos + rel + person + event_invalid + place_references
         
         if errors == 0:
             if cl:
@@ -477,6 +492,10 @@ class CheckIntegrity:
             self.text.write(_("1 invalid death event name was fixed\n"))
         elif death_invalid > 1:
             self.text.write(_("%d invalid death event names were fixed\n") % death_invalid)
+        if place_references == 1:
+            self.text.write(_("1 place was referenced but not found\n"))
+        elif place_references > 1:
+            self.text.write(_("%d places were referenced, but not found\n") % place_references)
 
         return errors