Merge branch 'pauseFix' into 'master'

BUG: Viewers shouldn't be pausable by scene controller

See merge request iMSTK/iMSTK!532

Source/Common/imstkModule.h

@@ -64,9 +64,10 @@ public:
     bool getInit() const { return m_init; }
 
     ///
-    /// \brief Get whether the module is currently paused
+    /// \brief Set/Get whether the module is currently paused
     ///
     bool getPaused() const { return m_paused; }
+    void setPaused(const bool paused) { m_paused = paused; }
 
     ExecutionType getExecutionType() const { return m_executionType; }
     void setExecutionType(ExecutionType type) { m_executionType = type; }

Source/SimulationManager/imstkKeyboardSceneControl.cpp

@@ -74,16 +74,12 @@ KeyboardSceneControl::OnKeyPress(const char key)
         // To ensure consistency toggle/invert based of m_sceneManager
         const bool paused = sceneManager->getPaused();
 
-        // Resume or pause all modules
+        // Resume or pause all modules, expect viewers
         for (auto module : driver->getModules())
         {
-            if (paused)
+            if (std::dynamic_pointer_cast<Viewer>(module) == nullptr)
             {
-                module->resume();
-            }
-            else
-            {
-                module->pause();
+                module->setPaused(!paused);
             }
         }
     }