From 508dc6548b625f2d977ed2610f74361aa0e5df9a Mon Sep 17 00:00:00 2001
From: Paul McPhee <paul.mcphee@digital.justice.gov.uk>
Date: Fri, 2 Aug 2024 17:16:47 +0100
Subject: [PATCH] PI-2391: Refactored

---
 .../delius/service/InterventionService.kt     | 38 +++----------------
 1 file changed, 6 insertions(+), 32 deletions(-)

diff --git a/projects/court-case-and-delius/src/main/kotlin/uk/gov/justice/digital/hmpps/integrations/delius/service/InterventionService.kt b/projects/court-case-and-delius/src/main/kotlin/uk/gov/justice/digital/hmpps/integrations/delius/service/InterventionService.kt
index 1754ad2eb0..2d0e9fb3bd 100644
--- a/projects/court-case-and-delius/src/main/kotlin/uk/gov/justice/digital/hmpps/integrations/delius/service/InterventionService.kt
+++ b/projects/court-case-and-delius/src/main/kotlin/uk/gov/justice/digital/hmpps/integrations/delius/service/InterventionService.kt
@@ -8,7 +8,6 @@ import uk.gov.justice.digital.hmpps.integrations.delius.event.nsi.NsiRepository
 import uk.gov.justice.digital.hmpps.integrations.delius.person.entity.PersonRepository
 import uk.gov.justice.digital.hmpps.integrations.delius.person.entity.getPerson
 import uk.gov.justice.digital.hmpps.integrations.delius.provider.entity.Staff
-import java.util.*
 import uk.gov.justice.digital.hmpps.integrations.delius.event.nsi.Nsi as NsiEntity
 import uk.gov.justice.digital.hmpps.integrations.delius.event.nsi.NsiManager as NsiManagerEntity
 
@@ -18,7 +17,6 @@ class InterventionService(
     private val eventRepository: EventRepository,
     private val nsiRepository: NsiRepository,
 ) {
-
     fun getNsiByCodes(crn: String, convictionId: Long, nsiCodes: List<String>): NsiDetails {
         val person = personRepository.getPerson(crn)
         val event = eventRepository.getByPersonAndEventNumber(person, convictionId)
@@ -76,37 +74,13 @@ fun Staff.toStaffDetails(): StaffDetails = StaffDetails(
     grade?.keyValueOf()
 )
 
-fun NsiEntity.toOutcomeRecall(): Boolean? =
-    Optional.ofNullable(outcome).map {
-        try {
-            OutcomeType.valueOf(it.code)
-        } catch (ex: Exception) {
-            OutcomeType.UNKNOWN
-        }.isOutcomeRecall
-    }.orElse(null)
-
-fun NsiEntity.toRecallRejectedOrWithdrawn(): Boolean? {
-    val status = try {
-        Status.valueOf(nsiStatus.code)
-    } catch (ex: Exception) {
-        Status.UNKNOWN
-    }
+private inline fun <reified T : Enum<T>> String.toEnumOrElse(default: T) =
+    T::class.java.enumConstants.firstOrNull { it.name == this } ?: default
 
-    return if (status.isRejectedOrWithdrawn != null) {
-        if (status.isRejectedOrWithdrawn) {
-            true
-        } else {
-            Optional.ofNullable(outcome).map {
-                try {
-                    OutcomeType.valueOf(it.code)
-                } catch (ex: Exception) {
-                    OutcomeType.UNKNOWN
-                }.isOutcomeRejectedOrWithdrawn
-            }.orElse(false)
-        }
-    } else {
-        null
-    }
+fun NsiEntity.toOutcomeRecall() = outcome?.code?.toEnumOrElse(OutcomeType.UNKNOWN)?.isOutcomeRecall
+
+fun NsiEntity.toRecallRejectedOrWithdrawn() = nsiStatus.code.toEnumOrElse(Status.UNKNOWN).isRejectedOrWithdrawn?.let {
+    it || (outcome?.code?.toEnumOrElse(OutcomeType.UNKNOWN)?.isOutcomeRejectedOrWithdrawn ?: false)
 }
 
 enum class OutcomeType(val code: String, val isOutcomeRejectedOrWithdrawn: Boolean?, val isOutcomeRecall: Boolean?) {