The changes in D879 fix a problem with nodes being hidden in certain cases, but the underlying design should be rethought.
From an API-perspective, it would make sense for QmitkSynchronizedWidgetConnector::SetSelectAll() to already change the data selection. Instead, it is only indirectly connected to updating the selection. Cleaning this up could offer a better solution than what was done in D879.