Don't hide rows of properties not within the model (#1507) (#1518)

(cherry picked from commit 25e5b83)

Signed-off-by: Alejandro Hernandez Cordero <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>

Author: mergify[bot]

rviz_common/src/rviz_common/properties/property_tree_widget.cpp

@@ -117,9 +117,13 @@ void PropertyTreeWidget::setModel(PropertyTreeModel * model)
 void PropertyTreeWidget::propertyHiddenChanged(const Property * property)
 {
   if (model_) {
-    setRowHidden(
-      property->rowNumberInParent(), model_->parentIndex(property),
-      property->getHidden() );
+    const auto & parent_index = model_->parentIndex(property);
+    if (parent_index.isValid()) {
+      setRowHidden(property->rowNumberInParent(), parent_index, property->getHidden());
+    } else {
+      printf("Trying to hide property '%s' that is not part of the model.\n",
+        qPrintable(property->getName()));
+    }
   }
 }