[unomi] branch master updated: UNOMI-717: fix retro compatibility for context response .getPersonali… (#540)

[jkevan]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 608ded728 UNOMI-717: fix retro compatibility for context response 
.getPersonali… (#540)
608ded728 is described below

commit 608ded72809e4c3a23b6696e07071231116eb3ff
Author: kevan Jahanshahi <jke...@apache.org>
AuthorDate: Tue Nov 15 16:32:55 2022 +0100

    UNOMI-717: fix retro compatibility for context response .getPersonali… 
(#540)
    
    * UNOMI-717: fix retro compatibility for context response 
.getPersonalizations()
    
    * UNOMI-717: fix retro compatibility for context response 
.getPersonalizations()
---
 api/src/main/java/org/apache/unomi/api/ContextResponse.java        | 7 -------
 .../java/org/apache/unomi/rest/endpoints/ContextJsonEndpoint.java  | 2 ++
 2 files changed, 2 insertions(+), 7 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..6da1f3875 100644
--- a/api/src/main/java/org/apache/unomi/api/ContextResponse.java
+++ b/api/src/main/java/org/apache/unomi/api/ContextResponse.java
@@ -203,14 +203,7 @@ public class ContextResponse implements Serializable {
      * @deprecated personalizations results are more complex since 2.1.0 and 
they are now available under: getPersonalizationResults()
      */
     @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());
             }
         }