From 35f7b7db2f6f5b11ea5a8dd0b2220106171d0853 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jacob=20St=C3=B8ren?= Date: Wed, 2 Oct 2013 16:08:32 +0200 Subject: [PATCH] Added some comments on getting rid of QModelIndex based commands --- .../ProjectDataModel/RimUiTreeModelPdm.cpp | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/ApplicationCode/ProjectDataModel/RimUiTreeModelPdm.cpp b/ApplicationCode/ProjectDataModel/RimUiTreeModelPdm.cpp index c30250b659..68ec961b7c 100644 --- a/ApplicationCode/ProjectDataModel/RimUiTreeModelPdm.cpp +++ b/ApplicationCode/ProjectDataModel/RimUiTreeModelPdm.cpp @@ -147,11 +147,13 @@ bool RimUiTreeModelPdm::deletePropertyFilter(const QModelIndex& itemIndex) bool wasSomeFilterActive = propertyFilterCollection->hasActiveFilters(); // Remove Ui items pointing at the pdm object to delete - removeRows_special(itemIndex.row(), 1, itemIndex.parent()); + removeRows_special(itemIndex.row(), 1, itemIndex.parent()); // To be deleted propertyFilterCollection->remove(propertyFilter); delete propertyFilter; + // updateUiSubTree(propertyFilterCollection); // To be enabled + if (wasFilterActive) { propertyFilterCollection->reservoirView()->scheduleGeometryRegen(RivReservoirViewPartMgr::PROPERTY_FILTERED); @@ -187,11 +189,13 @@ bool RimUiTreeModelPdm::deleteRangeFilter(const QModelIndex& itemIndex) bool wasSomeFilterActive = rangeFilterCollection->hasActiveFilters(); // Remove Ui items pointing at the pdm object to delete - removeRows_special(itemIndex.row(), 1, itemIndex.parent()); + removeRows_special(itemIndex.row(), 1, itemIndex.parent()); // To be deleted rangeFilterCollection->remove(rangeFilter); delete rangeFilter; + // updateUiSubTree(rangeFilterCollection); // To be enabled + if (wasFilterActive) { rangeFilterCollection->reservoirView()->scheduleGeometryRegen(RivReservoirViewPartMgr::PROPERTY_FILTERED); @@ -221,11 +225,12 @@ bool RimUiTreeModelPdm::deleteReservoirView(const QModelIndex& itemIndex) CVF_ASSERT(reservoirView); // Remove Ui items pointing at the pdm object to delete - removeRows_special(itemIndex.row(), 1, itemIndex.parent()); + removeRows_special(itemIndex.row(), 1, itemIndex.parent()); // To be deleted reservoirView->eclipseCase()->removeReservoirView(reservoirView); delete reservoirView; + // updateUiSubTree(reservoirView->eclipseCase()); // To be enabled clearClipboard(); return true;