You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@unomi.apache.org by jk...@apache.org on 2021/02/24 14:41:36 UTC

[unomi] branch master updated: in updatePropertiesAction: fix override strategy when updating profile properties (#251)

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 ecbae7e  in updatePropertiesAction: fix override strategy when updating profile properties (#251)
ecbae7e is described below

commit ecbae7edc7ede13f0aaa421cea4ad25905678203
Author: giladw <gw...@yotpo.com>
AuthorDate: Wed Feb 24 16:41:28 2021 +0200

    in updatePropertiesAction: fix override strategy when updating profile properties (#251)
---
 .../apache/unomi/plugins/baseplugin/actions/UpdatePropertiesAction.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/baseplugin/src/main/java/org/apache/unomi/plugins/baseplugin/actions/UpdatePropertiesAction.java b/plugins/baseplugin/src/main/java/org/apache/unomi/plugins/baseplugin/actions/UpdatePropertiesAction.java
index ef175d9..385918e 100644
--- a/plugins/baseplugin/src/main/java/org/apache/unomi/plugins/baseplugin/actions/UpdatePropertiesAction.java
+++ b/plugins/baseplugin/src/main/java/org/apache/unomi/plugins/baseplugin/actions/UpdatePropertiesAction.java
@@ -125,7 +125,7 @@ public class UpdatePropertiesAction implements ActionExecutor {
                 }
             }
             if (propType != null) {
-                isProfileOrPersonaUpdated |= PropertyHelper.setProperty(target, prop, PropertyHelper.getValueByTypeId(propsMap.get(prop), propType.getValueTypeId()), "alwaysSet");
+                isProfileOrPersonaUpdated |= PropertyHelper.setProperty(target, prop, PropertyHelper.getValueByTypeId(propsMap.get(prop), propType.getValueTypeId()), strategy);
             } else {
                 isProfileOrPersonaUpdated |= PropertyHelper.setProperty(target, prop, propsMap.get(prop), strategy);
             }