Don't use a static for the threadboundary

Otherwise we risk calling into already destroyed models.

2 files changed, 2 insertions, 2 deletions

diff --git a/common/modelresult.cpp b/common/modelresult.cpp
index 9f3e662..3778d4d 100644
--- a/common/modelresult.cpp
+++ b/common/modelresult.cpp
@@ -24,7 +24,6 @@
 
 #include "domain/folder.h"
 #include "log.h"
-#include "threadboundary.h"
 
 #undef DEBUG_AREA
 #define DEBUG_AREA "client.modelresult"
@@ -253,7 +252,6 @@ void ModelResult<T, Ptr>::setFetcher(const std::function<void(const Ptr &parent)
 template <class T, class Ptr>
 void ModelResult<T, Ptr>::setEmitter(const typename Sink::ResultEmitter<Ptr>::Ptr &emitter)
 {
-    static async::ThreadBoundary threadBoundary;
     setFetcher([this](const Ptr &parent) { mEmitter->fetch(parent); });
 
     emitter->onAdded([this](const Ptr &value) {
diff --git a/common/modelresult.h b/common/modelresult.h
index 64431da..7924e2d 100644
--- a/common/modelresult.h
+++ b/common/modelresult.h
@@ -27,6 +27,7 @@
 #include <functional>
 #include "query.h"
 #include "resultprovider.h"
+#include "threadboundary.h"
 
 template <class T, class Ptr>
 class ModelResult : public QAbstractItemModel
@@ -76,4 +77,5 @@ private:
     Sink::Query mQuery;
     std::function<void(const Ptr &)> loadEntities;
     typename Sink::ResultEmitter<Ptr>::Ptr mEmitter;
+    async::ThreadBoundary threadBoundary;
 };