Browse Source

BUG: stop after unable to create dir.

An interim test of continuing on to the next series after
being unable to create the export directory had been left in,
this commit halts the export after the first.
Added a suggestion to repair the database if a source file
is not found.

Slicer Issue #3163
Nicole Aucoin 9 years ago
parent
commit
86837e6a68
1 changed files with 4 additions and 4 deletions
  1. 4 4
      Libs/DICOM/Widgets/ctkDICOMBrowser.cpp

+ 4 - 4
Libs/DICOM/Widgets/ctkDICOMBrowser.cpp

@@ -967,13 +967,13 @@ void ctkDICOMBrowser::exportSelectedSeries(QString dirPath, QStringList uids)
         {
         {
         QString errorString =
         QString errorString =
           QString("Unable to create export destination directory:\n\n")
           QString("Unable to create export destination directory:\n\n")
-          + destinationDir;
+          + destinationDir
+          + QString("\n\nHalting export.");
         ctkMessageBox createDirectoryErrorMessageBox;
         ctkMessageBox createDirectoryErrorMessageBox;
         createDirectoryErrorMessageBox.setText(errorString);
         createDirectoryErrorMessageBox.setText(errorString);
         createDirectoryErrorMessageBox.setIcon(QMessageBox::Warning);
         createDirectoryErrorMessageBox.setIcon(QMessageBox::Warning);
         createDirectoryErrorMessageBox.exec();
         createDirectoryErrorMessageBox.exec();
-        // go on the the next series if present
-        continue;
+        return;
         }
         }
       }
       }
 
 
@@ -1012,7 +1012,7 @@ void ctkDICOMBrowser::exportSelectedSeries(QString dirPath, QStringList uids)
         d->ExportProgress->setValue(numFiles);
         d->ExportProgress->setValue(numFiles);
         QString errorString = QString("Export source file not found:\n\n")
         QString errorString = QString("Export source file not found:\n\n")
           + filePath
           + filePath
-          + QString("\n\nHalting export.");
+          + QString("\n\nHalting export.\n\nError may be fixed via Repair.");
         ctkMessageBox copyErrorMessageBox;
         ctkMessageBox copyErrorMessageBox;
         copyErrorMessageBox.setText(errorString);
         copyErrorMessageBox.setText(errorString);
         copyErrorMessageBox.setIcon(QMessageBox::Warning);
         copyErrorMessageBox.setIcon(QMessageBox::Warning);