From 8cb7715fa07a263d3410b9035cc8284ec5c8d7e5 Mon Sep 17 00:00:00 2001 From: Lindsey Dew <1202622+lindseydew@users.noreply.github.com> Date: Wed, 27 Mar 2024 10:09:30 +0000 Subject: [PATCH] Use delete and return nothing --- .../main/scala/coml/gu/sfl/userdeletion/db/SflDynamoDb.scala | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/mobile-save-for-later-user-deletion/src/main/scala/coml/gu/sfl/userdeletion/db/SflDynamoDb.scala b/mobile-save-for-later-user-deletion/src/main/scala/coml/gu/sfl/userdeletion/db/SflDynamoDb.scala index 35e9a9ec..a83ef3fd 100644 --- a/mobile-save-for-later-user-deletion/src/main/scala/coml/gu/sfl/userdeletion/db/SflDynamoDb.scala +++ b/mobile-save-for-later-user-deletion/src/main/scala/coml/gu/sfl/userdeletion/db/SflDynamoDb.scala @@ -5,10 +5,11 @@ import org.scanamo.syntax._ import com.gu.sfl.Logging import com.gu.sfl.persistence.{DynamoSavedArticles, PersistenceConfig} import com.gu.sfl.userdeletion.model.UserDeleteMessage -import org.scanamo.DeleteReturn.OldValue +import org.scanamo.DeleteReturn.Nothing import org.scanamo.generic.auto.genericDerivedFormat import software.amazon.awssdk.services.dynamodb.DynamoDbClient + class SflDynamoDb(persistanceConfig: PersistenceConfig) extends Logging { private val table = Table[DynamoSavedArticles](persistanceConfig.tableName) @@ -16,7 +17,7 @@ class SflDynamoDb(persistanceConfig: PersistenceConfig) extends Logging { private val scanamo = Scanamo(client) def deleteSavedArticleasForUser(user: UserDeleteMessage) = { logger.info(s"Deleting record for user id: ${user.userId}") - val dbResponse = scanamo.exec(table.deleteAndReturn(OldValue)("userId" === user.userId)) + val dbResponse = scanamo.exec(table.deleteAndReturn(Nothing)("userId" === user.userId)) .fold(s"Unable to delete record for user ${user.userId}")((_) => s"Deleted record for ${user.userId}") logger.info(dbResponse) }