ENH: Add removePatient/Series/Study signals to DICOM database

Libs/DICOM/Core/Testing/Cpp/CMakeLists.txt

@@ -29,6 +29,12 @@ set(LIBRARY_NAME ${PROJECT_NAME})
 ctk_add_executable_utf8(${KIT}CppTests ${Tests})
 target_link_libraries(${KIT}CppTests ${LIBRARY_NAME})
 
+find_package(Qt${CTK_QT_VERSION} COMPONENTS Test Widgets REQUIRED)
+target_link_libraries(${KIT}CppTests
+  Qt${CTK_QT_VERSION}::Test
+  Qt${CTK_QT_VERSION}::Widgets
+  )
+
 #
 # Add Tests
 #

Libs/DICOM/Core/Testing/Cpp/ctkDICOMDatabaseTest6.cpp

@@ -21,6 +21,7 @@
 // Qt includes
 #include <QCoreApplication>
 #include <QDir>
+#include <QSignalSpy>
 #include <QTemporaryDir>
 
 // ctkCore includes
@@ -157,6 +158,34 @@ int ctkDICOMDatabaseTest6( int argc, char * argv [] )
     return EXIT_FAILURE;
     }
 
+  {
+    QSignalSpy spySeries(&database, SIGNAL(seriesRemoved(QString)));
+    database.removeSeries(seriesUID);
+    CHECK_INT(spySeries.count(), 1);
+    CHECK_QSTRING(spySeries.at(0).value(0).toString(), seriesUID);
+
+    QStringList seriesUIDs = database.seriesForStudy(studyUID);
+    CHECK_INT(seriesUIDs.count(), 0);
+  }
+  {
+    QSignalSpy spyStudy(&database, SIGNAL(studyRemoved(QString)));
+    database.removeStudy(studyUID);
+    CHECK_INT(spyStudy.count(), 1);
+    CHECK_QSTRING(spyStudy.at(0).value(0).toString(), studyUID);
+
+    QStringList studyUIDs = database.studiesForPatient(patientUID);
+    CHECK_INT(studyUIDs.count(), 0);
+  }
+  {
+    QSignalSpy spyPatient(&database, SIGNAL(patientRemoved(QString)));
+    database.removePatient(patientUID);
+    CHECK_INT(spyPatient.count(), 1);
+    CHECK_QSTRING(spyPatient.at(0).value(0).toString(), patientUID);
+
+    QStringList patientIDs = database.patients();
+    CHECK_INT(patientIDs.count(), 0);
+  }
+
   database.closeDatabase();
 
   std::cerr << "Database is in " << databaseDirectory.path().toStdString() << std::endl;

Libs/DICOM/Core/ctkDICOMDatabase.cpp

@@ -2766,6 +2766,8 @@ bool ctkDICOMDatabase::removeSeries(const QString& seriesInstanceUID, bool clear
 
   d->resetLastInsertedValues();
 
+  emit seriesRemoved(seriesInstanceUID);
+
   return true;
 }
 
@@ -2811,6 +2813,12 @@ bool ctkDICOMDatabase::removeStudy(const QString& studyInstanceUID)
     }
   }
   d->resetLastInsertedValues();
+
+  if(result)
+  {
+    emit studyRemoved(studyInstanceUID);
+  }
+
   return result;
 }
 
@@ -2838,6 +2846,12 @@ bool ctkDICOMDatabase::removePatient(const QString& patientID)
     }
   }
   d->resetLastInsertedValues();
+
+  if(result)
+  {
+    emit patientRemoved(patientID);
+  }
+
   return result;
 }
 

Libs/DICOM/Core/ctkDICOMDatabase.h

@@ -424,6 +424,27 @@ class CTK_DICOM_CORE_EXPORT ctkDICOMDatabase : public QObject
   ///  - instanceUID (unique)
   void instanceAdded(QString);
 
+  /// \brief This signal is emitted after a patient is removed by calling removePatient().
+  ///
+  /// \param patientID (not unique across institutions)
+  ///
+  /// \sa removePatient()
+  void patientRemoved(QString);
+
+  /// \brief This signal is emitted after a study is removed by calling removeStudy().
+  ///
+  /// \param studyInstanceUID (unique)
+  ///
+  /// \sa removeStudy()
+  void studyRemoved(QString);
+
+  /// \brief This signal is emitted after a series is removed by calling removeSeries().
+  ///
+  /// \param seriesInstanceUID (unique)
+  ///
+  /// \sa removeSeries()
+  void seriesRemoved(QString);
+
   /// This signal is emitted when the database has been opened.
   void opened();