bugfix/Fix term and conditions multiple saving

OpenBankProject · Sep 10, 2024 · f5fa8e9 · f5fa8e9
1 parent 464e3c7
commit f5fa8e9
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/obp-api/src/main/scala/code/model/dataAccess/AuthUser.scala b/obp-api/src/main/scala/code/model/dataAccess/AuthUser.scala
@@ -1045,7 +1045,7 @@ def restoreSomeSessions(): Unit = {
           .map(_.agreementHash).getOrElse(HashUtil.Sha256Hash("not set"))
       val agreementText = getWebUiPropsValue("webui_terms_and_conditions", "not set")
       val hashedAgreementText = HashUtil.Sha256Hash(agreementText)
-      if(hashedAgreementTextOfUser == hashedAgreementText) { // Chech terms and conditions
+      if(hashedAgreementTextOfUser == hashedAgreementText) { // Check terms and conditions
         val hashedAgreementTextOfUser =
           UserAgreementProvider.userAgreementProvider.vend.getUserAgreement(userId, "privacy_conditions")
             .map(_.agreementHash).getOrElse(HashUtil.Sha256Hash("not set"))

diff --git a/obp-api/src/main/scala/code/snippet/TermsAndConditions.scala b/obp-api/src/main/scala/code/snippet/TermsAndConditions.scala
@@ -70,7 +70,9 @@ class TermsAndConditions extends MdcLoggable {
       val agreementText = getWebUiPropsValue("webui_terms_and_conditions", "not set")
       // val hashedAgreementText = HashUtil.Sha256Hash(agreementText)
       UserAgreementProvider.userAgreementProvider.vend.createOrUpdateUserAgreement(
-        AuthUser.currentUser.flatMap(_.user.foreign.map(_.userId)).getOrElse(""), "terms_and_conditions", agreementText)
+        AuthUser.currentUser.flatMap(_.user.foreign.map(_.userId)).getOrElse(""),
+        "terms_and_conditions",
+        agreementText)
       S.redirectTo("/")
     }
   }

diff --git a/obp-api/src/main/scala/code/users/UserAgreement.scala b/obp-api/src/main/scala/code/users/UserAgreement.scala
@@ -82,7 +82,7 @@ object UserAgreement extends UserAgreement with LongKeyedMetaMapper[UserAgreemen
     agreement =>
       tryo {
         val hash = HashUtil.Sha256Hash(agreement.agreementText)
-        agreement.AgreementHash(hash ).save
+        agreement.AgreementHash(hash)
       }
   )