From 46f515448dd77907ccb24855c0316f63e9812536 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marko=20Mili=C4=87?= <marko@tesobe.com>
Date: Tue, 13 May 2025 09:58:13 +0200
Subject: [PATCH] feature/Tweak endpoint addConsentUser v4.0.0

---
 obp-api/src/main/scala/code/api/v4_0_0/APIMethods400.scala | 4 +++-
 obp-api/src/main/scala/code/api/v5_1_0/APIMethods510.scala | 2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/obp-api/src/main/scala/code/api/v4_0_0/APIMethods400.scala b/obp-api/src/main/scala/code/api/v4_0_0/APIMethods400.scala
index b7bf2973a..01ea36639 100644
--- a/obp-api/src/main/scala/code/api/v4_0_0/APIMethods400.scala
+++ b/obp-api/src/main/scala/code/api/v4_0_0/APIMethods400.scala
@@ -8081,7 +8081,9 @@ trait APIMethods400 extends MdcLoggable {
             consent <- Future(Consents.consentProvider.vend.getConsentByConsentId(consentId)) map {
               i => connectorEmptyResponse(i, callContext)
             }
-            _ <- Helper.booleanToFuture(ConsentUserAlreadyAdded, cc=callContext) { consent.userId != null }
+            _ <- Helper.booleanToFuture(ConsentUserAlreadyAdded, cc = cc.callContext) {
+              Option(consent.userId).forall(_.isBlank) // checks whether userId is not populated
+            }
             consent <- Future(Consents.consentProvider.vend.updateConsentUser(consentId, user)) map {
               i => connectorEmptyResponse(i, callContext)
             }
diff --git a/obp-api/src/main/scala/code/api/v5_1_0/APIMethods510.scala b/obp-api/src/main/scala/code/api/v5_1_0/APIMethods510.scala
index a7885cfba..37c91d520 100644
--- a/obp-api/src/main/scala/code/api/v5_1_0/APIMethods510.scala
+++ b/obp-api/src/main/scala/code/api/v5_1_0/APIMethods510.scala
@@ -1439,7 +1439,7 @@ trait APIMethods510 {
       nameOf(updateConsentUserIdByConsentId),
       "PUT",
       "/management/banks/BANK_ID/consents/CONSENT_ID/created-by-user",
-      "Update Consent Created by User by CONSENT_ID",
+      "Update Created by User of Consent by CONSENT_ID",
       s"""
          |
          |This endpoint is used to Update the User bound to a consent.