Avoid fetching repeatedly to check for children.

canFetchMore returns true once the fetch is complete to allow the
incremental fetch behaviour. When we check for children we really only
want to fetch once though, otherwise we risk fetching over and over.

1 files changed, 2 insertions, 1 deletions

diff --git a/common/modelresult.cpp b/common/modelresult.cpp
index add84aa..d016978 100644
--- a/common/modelresult.cpp
+++ b/common/modelresult.cpp
@@ -154,7 +154,8 @@ bool ModelResult<T, Ptr>::hasChildren(const QModelIndex &parent) const
         return false;
     }
     //Figure out whether we have children
-    if (canFetchMore(parent)) {
+    const auto id = parent.internalId();
+    if (!mEntityChildrenFetched.contains(id)) {
         //Since we don't retrieve that information as part of the entity,
         //we have to query for the children to see if we have some
         auto p = const_cast<ModelResult<T, Ptr>*>(this);