Merge pull request #250 from ReagentX/feat/cs/fix-188

Make `person_centric_id` optional

README.md

@@ -18,7 +18,7 @@ Documentation for the library is located [here](imessage-database/README.md).
 
 ### Supported Features
 
-This crate supports every iMessage feature as of macOS 14.3.1 (23D60) and iOS 17.3.1 (21D61):
+This crate supports every iMessage feature as of macOS 14.4.1 (23E224) and iOS 17.4.1 (21E236):
 
 - Multi-part messages
 - Replies/Threads

imessage-database/src/tables/handle.rs

@@ -144,18 +144,20 @@ impl Diagnostic for Handle {
             "FROM handle ",
             "WHERE person_centric_id NOT NULL"
         );
-        let mut rows = db.prepare(query).map_err(TableError::Messages)?;
-        let count_dupes: Option<i32> = rows
-            .query_row([], |r| r.get(0))
-            .map_err(TableError::Handle)?;
 
-        done_processing();
+        if let Ok(mut rows) = db.prepare(query).map_err(TableError::Handle) {
+            let count_dupes: Option<i32> = rows
+                .query_row([], |r| r.get(0))
+                .map_err(TableError::Handle)?;
 
-        if let Some(dupes) = count_dupes {
-            if dupes > 0 {
-                println!("\rContacts with more than one ID: {dupes}");
+            if let Some(dupes) = count_dupes {
+                if dupes > 0 {
+                    println!("\rContacts with more than one ID: {dupes}");
+                }
             }
         }
+
+        done_processing();
         Ok(())
     }
 }