summaryrefslogtreecommitdiffstats
path: root/common
diff options
context:
space:
mode:
authorChristian Mollekopf <chrigi_1@fastmail.fm>2018-06-01 08:59:47 +0200
committerChristian Mollekopf <chrigi_1@fastmail.fm>2018-06-01 08:59:47 +0200
commit814dfdaa1bdefe01c9eb37de52997c198a77f22c (patch)
tree08740ce4e5d61fbb171fbd626fe8d5850640edf9 /common
parent2daac34a28aa8a1623f6d347e2a81474b65caf8a (diff)
downloadsink-814dfdaa1bdefe01c9eb37de52997c198a77f22c.tar.gz
sink-814dfdaa1bdefe01c9eb37de52997c198a77f22c.zip
Avoid trying to remove an item that we don't have in the model.
This triggered the assert in createIndexFromId.
Diffstat (limited to 'common')
-rw-r--r--common/modelresult.cpp3
1 files changed, 3 insertions, 0 deletions
diff --git a/common/modelresult.cpp b/common/modelresult.cpp
index 495eca0..57f5ce8 100644
--- a/common/modelresult.cpp
+++ b/common/modelresult.cpp
@@ -340,6 +340,9 @@ template <class T, class Ptr>
340void ModelResult<T, Ptr>::remove(const Ptr &value) 340void ModelResult<T, Ptr>::remove(const Ptr &value)
341{ 341{
342 auto childId = qHash(*value); 342 auto childId = qHash(*value);
343 if (!mEntities.contains(childId)) {
344 return;
345 }
343 //The removed entity will have no properties, but we at least need the parent property. 346 //The removed entity will have no properties, but we at least need the parent property.
344 auto actualEntity = mEntities.value(childId); 347 auto actualEntity = mEntities.value(childId);
345 auto id = parentId(actualEntity); 348 auto id = parentId(actualEntity);