Fix goofy initialization order requirements for EffectChains by making them send all their existing state right after being added to the engine.

1 files changed, 7 insertions, 4 deletions

diff --git a/src/effects/effectchainmanager.cpp b/src/effects/effectchainmanager.cpp
index 07476705b7..2ff4e09543 100644
--- a/src/effects/effectchainmanager.cpp
+++ b/src/effects/effectchainmanager.cpp
@@ -15,11 +15,14 @@ EffectChainManager::~EffectChainManager() {
 void EffectChainManager::addEffectChain(EffectChainPointer pEffectChain) {
     if (pEffectChain) {
         m_effectChains.append(pEffectChain);
-        EffectsRequest* pRequest = new EffectsRequest();
-        pRequest->type = EffectsRequest::ADD_EFFECT_CHAIN;
-        pRequest->AddEffectChain.pChain = pEffectChain->getEngineEffectChain();
-        m_pEffectsManager->writeRequest(pRequest);
+        pEffectChain->addToEngine();
+    }
+}
 
+void EffectChainManager::removeEffectChain(EffectChainPointer pEffectChain) {
+    if (pEffectChain) {
+        m_effectChains.removeAll(pEffectChain);
+        pEffectChain->removeFromEngine();
     }
 }