ThreadWeaver Job Decorators don't actually hook into an existing job.
Instead, they provide the same interface as the job and just pass through
all calls.
The way the QObjectDecorator is used by KRunner results in no signals actually
getting emitted, making most of the code pointless.
This patch is just a quick hack to show what would happen if the jobDone signal
worked and is mostly intended to present the issue.
jobDone is called now.
| No Linters Available |
| No Unit Test Coverage |
| Buildable 12435 | |
| Build 12453: arc lint + arc unit |
I need this additionally
diff --git a/src/runnermanager.cpp b/src/runnermanager.cpp index f92830d..6ec0c3c 100644 --- a/src/runnermanager.cpp +++ b/src/runnermanager.cpp @@ -345,6 +345,7 @@ public: emit q->matchesChanged(context.matches()); } + teardownRequested = true; checkTearDown(); } @@ -833,6 +834,7 @@ void RunnerManager::reset() } d->context.reset(); + emit queryFinished(); } } // Plasma namespace
With this it works quite reliably for me.