Hi Manuel, thanks for the patch. I am currently working on a couple of other issues with the file retrieval and added your patch to the corresponding bug [1].
Best, Caspar [1] http://bugs.mitk.org/show_bug.cgi?id=19446 -----Ursprüngliche Nachricht----- Von: Manuel Sánchez García [mailto:manuel.sanchez.gar...@outlook.com] Gesendet: Dienstag, 9. Februar 2016 14:50 An: mitk-users@lists.sourceforge.net Betreff: [mitk-users] Bug fix for QmitkStoreSCPLauncher (patch) Hi all, This post provides a trivial one-line patch to the DICOM import plugin. It was crashing on me when retrieving files from a PACS. The problem seems to come from QmitkStoreSCPLauncher. The class tries to read the list of transferred files from stdout, but storescp sends messages to stderr. The path just asks Qt to merge both channels. Regards Manuel diff --git a/Plugins/org.mitk.gui.qt.dicom/src/internal/QmitkStoreSCPLauncher.cpp b/Plugins/org.mitk.gui.qt.dicom/src/internal/QmitkStoreSCPLauncher.cpp index b33e5c6..7ed5453 100644 --- a/Plugins/org.mitk.gui.qt.dicom/src/internal/QmitkStoreSCPLauncher.cpp +++ b/Plugins/org.mitk.gui.qt.dicom/src/internal/QmitkStoreSCPLauncher.c +++ pp @@ -31,6 +31,7 @@ See LICENSE.txt or http://www.mitk.org for details. QmitkStoreSCPLauncher::QmitkStoreSCPLauncher(QmitkStoreSCPLauncherBuilder* builder) : m_StoreSCP(new QProcess()) { + m_StoreSCP->setProcessChannelMode(QProcess::MergedChannels); connect( m_StoreSCP, SIGNAL(error(QProcess::ProcessError)),this, SLOT(OnProcessError(QProcess::ProcessError))); connect( m_StoreSCP, SIGNAL(stateChanged(QProcess::ProcessState)),this, SLOT(OnStateChanged(QProcess::ProcessState))); connect( m_StoreSCP, SIGNAL(readyReadStandardOutput()),this, SLOT(OnReadyProcessOutput())); ------------------------------------------------------------------------------ Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140 _______________________________________________ mitk-users mailing list mitk-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mitk-users ------------------------------------------------------------------------------ Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140 _______________________________________________ mitk-users mailing list mitk-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mitk-users
