Maybe fix clang-tidy

niknetniko · Dec 3, 2024 · 15a961f · 15a961f
1 parent 4a31618
commit 15a961f
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/person_detail/person_family_tab.cpp b/src/person_detail/person_family_tab.cpp
@@ -100,15 +100,19 @@ void PersonFamilyTab::onShowPedigreeChart() const {
 }
 
 void PersonFamilyTab::onParentClicked(const QModelIndex& index) const {
+    if (!index.isValid()) {
+        return;
+    }
     assert(index.model() == parentsTreeView->model());
     auto personId = index.model()->index(index.row(), DisplayParentModel::PERSON_ID, index.parent()).data();
-    qDebug() << "Parent double clicked results in" << personId;
     openOrSelectPerson(personId.toLongLong());
 }
 
 void PersonFamilyTab::onPartnerOrChildClicked(const QModelIndex& index) const {
+    if (!index.isValid()) {
+        return;
+    }
     assert(index.model() == partnerAndDescendantTreeView->model());
     auto personId = index.model()->index(index.row(), FamilyDisplayModel::PERSON_ID, index.parent()).data();
-    qDebug() << "Partner or child double clicked results in" << personId;
     openOrSelectPerson(personId.toLongLong());
 }