refactor: return value depends on both db and dt operations

SmartOperatingBlock · Mar 7, 2023 · 6595fc9 · 6595fc9
1 parent 067f157
commit 6595fc9
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/main/kotlin/application/controller/HealthProfessionalController.kt b/src/main/kotlin/application/controller/HealthProfessionalController.kt
@@ -24,13 +24,12 @@ class HealthProfessionalController(
 
     override fun createHealthProfessional(healthProfessional: HealthProfessional): HealthProfessional? =
         dtManager.createHealthProfessional(healthProfessional).let {
-            dbManager.createHealthProfessional(healthProfessional)
+            if (it) dbManager.createHealthProfessional(healthProfessional) else null
         }
 
     override fun deleteHealthProfessional(healthProfessionalId: String): Boolean =
-        dtManager.deleteHealthProfessional(healthProfessionalId).let {
+        dtManager.deleteHealthProfessional(healthProfessionalId) &&
             dbManager.deleteHealthProfessional(healthProfessionalId)
-        }
 
     override fun getHealthProfessional(healthProfessionalId: String): HealthProfessional? =
         dbManager.getHealthProfessional(healthProfessionalId)