This is an automated email from the ASF dual-hosted git repository.

jkevan pushed a commit to branch fixContextResponsePersoRetroCompatibility
in repository https://gitbox.apache.org/repos/asf/unomi.git

commit b09b0cdd04c6fb6e69b6f2d71d6e9794f4a11b10
Author: Kevan <ke...@jahia.com>
AuthorDate: Tue Nov 15 14:14:11 2022 +0100

    UNOMI-717: fix retro compatibility for context response 
.getPersonalizations()
---
 api/src/main/java/org/apache/unomi/api/ContextResponse.java         | 6 ------
 .../java/org/apache/unomi/rest/endpoints/ContextJsonEndpoint.java   | 2 ++
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/api/src/main/java/org/apache/unomi/api/ContextResponse.java 
b/api/src/main/java/org/apache/unomi/api/ContextResponse.java
index e07662c07..1a8cbdf9e 100644
--- a/api/src/main/java/org/apache/unomi/api/ContextResponse.java
+++ b/api/src/main/java/org/apache/unomi/api/ContextResponse.java
@@ -205,12 +205,6 @@ public class ContextResponse implements Serializable {
     @Deprecated
     @XmlTransient
     public Map<String, List<String>> getPersonalizations() {
-        if (personalizationResults != null) {
-            return personalizationResults.entrySet()
-                    .stream()
-                    .collect(Collectors.toMap(Map.Entry::getKey,
-                            entry -> entry.getValue().getContentIds()));
-        }
         return personalizations;
     }
 
diff --git 
a/rest/src/main/java/org/apache/unomi/rest/endpoints/ContextJsonEndpoint.java 
b/rest/src/main/java/org/apache/unomi/rest/endpoints/ContextJsonEndpoint.java
index c6f98bd37..407eb5f87 100644
--- 
a/rest/src/main/java/org/apache/unomi/rest/endpoints/ContextJsonEndpoint.java
+++ 
b/rest/src/main/java/org/apache/unomi/rest/endpoints/ContextJsonEndpoint.java
@@ -204,10 +204,12 @@ public class ContextJsonEndpoint {
         List<PersonalizationService.PersonalizationRequest> personalizations = 
contextRequest.getPersonalizations();
         if (personalizations != null) {
             data.setPersonalizationResults(new HashMap<>());
+            data.setPersonalizations(new HashMap<>());
             for (PersonalizationService.PersonalizationRequest personalization 
: sanitizePersonalizations(personalizations)) {
                 PersonalizationResult personalizationResult = 
personalizationService.personalizeList(eventsRequestContext.getProfile(), 
eventsRequestContext.getSession(), personalization);
                 
eventsRequestContext.addChanges(personalizationResult.getChangeType());
                 data.getPersonalizationResults().put(personalization.getId(), 
personalizationResult);
+                data.getPersonalizations().put(personalization.getId(), 
personalizationResult.getContentIds());
             }
         }
 

Reply via email to