CVSL-1606 Adding logging when prison staff modify conditions (#1522)

ministryofjustice · Jan 22, 2024 · b547252 · b547252
1 parent 078373d
commit b547252
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/server/routes/conditions.ts b/server/routes/conditions.ts
@@ -107,12 +107,16 @@ export default ({
       const additional = additionalConditions ? conditionsService.formatConditionInputs(req.body) : {}
       const newConditionsObject = conditionsService.createConditionsObjectForLicence(additional, bespoke)
 
-      await licenceService.updateLicenceConditions(
+      const updateOccurred = await licenceService.updateLicenceConditions(
         bookingId,
         res.locals.licence,
         newConditionsObject,
         res.locals.postRelease
       )
+      if (updateOccurred) {
+        const { username, role } = req.user
+        logger.info(`conditions - updated conditions - ${username} - ${role} - ${bookingId}`)
+      }
 
       const newVersion = conditionsServiceFactory.getNewVersion(res.locals.licence)
 
@@ -157,6 +161,8 @@ export default ({
       const { action } = req.params
 
       if (conditionId) {
+        const { username, role } = req.user
+        logger.info(`conditions - delete condition - ${username} - ${role} - ${bookingId}`)
         await licenceService.deleteLicenceCondition(bookingId, res.locals.licence, conditionId)
       }
 

diff --git a/server/services/licenceService.ts b/server/services/licenceService.ts
@@ -167,12 +167,12 @@ export class LicenceService {
       const licenceConditions = { ...existingLicenceConditions, ...newConditionsObject }
 
       if (equals(existingLicenceConditions, licenceConditions)) {
-        return null
+        return false
       }
-
       await this.updateModificationStage(bookingId, existingLicence.stage, { requiresApproval: true })
 
-      return await this.licenceClient.updateSection('licenceConditions', bookingId, licenceConditions, postRelease)
+      await this.licenceClient.updateSection('licenceConditions', bookingId, licenceConditions, postRelease)
+      return true
     } catch (error) {
       logger.error('Error during updateAdditionalConditions', error.stack)
       throw error