From 6595fc96c171d7989f7bac04583e27f0110e20a4 Mon Sep 17 00:00:00 2001
From: Andrea Acampora <andrea.acampora@studio.unibo.it>
Date: Tue, 7 Mar 2023 16:10:52 +0100
Subject: [PATCH] refactor: return value depends on both db and dt operations

---
 .../application/controller/HealthProfessionalController.kt   | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/main/kotlin/application/controller/HealthProfessionalController.kt b/src/main/kotlin/application/controller/HealthProfessionalController.kt
index d9228765..2977fcaf 100644
--- 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)