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 2019/01/04 13:31:17 UTC
incubator-unomi git commit: UNOMI-217: improve namings
Repository: incubator-unomi
Updated Branches:
refs/heads/master d3b1e950d -> 26ce3c174
UNOMI-217: improve namings
Project: http://git-wip-us.apache.org/repos/asf/incubator-unomi/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-unomi/commit/26ce3c17
Tree: http://git-wip-us.apache.org/repos/asf/incubator-unomi/tree/26ce3c17
Diff: http://git-wip-us.apache.org/repos/asf/incubator-unomi/diff/26ce3c17
Branch: refs/heads/master
Commit: 26ce3c1746632ff2180f490e1f2b32b0e588af94
Parents: d3b1e95
Author: Kevan <ke...@jahia.com>
Authored: Fri Jan 4 14:31:11 2019 +0100
Committer: Kevan <ke...@jahia.com>
Committed: Fri Jan 4 14:31:11 2019 +0100
----------------------------------------------------------------------
.../services/services/ProfileServiceImpl.java | 34 ++++++++++----------
1 file changed, 17 insertions(+), 17 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-unomi/blob/26ce3c17/services/src/main/java/org/apache/unomi/services/services/ProfileServiceImpl.java
----------------------------------------------------------------------
diff --git a/services/src/main/java/org/apache/unomi/services/services/ProfileServiceImpl.java b/services/src/main/java/org/apache/unomi/services/services/ProfileServiceImpl.java
index 1e79c68..9f023e3 100644
--- a/services/src/main/java/org/apache/unomi/services/services/ProfileServiceImpl.java
+++ b/services/src/main/java/org/apache/unomi/services/services/ProfileServiceImpl.java
@@ -1016,38 +1016,38 @@ public class ProfileServiceImpl implements ProfileService, SynchronousBundleList
return changed;
}
- private boolean mergeSystemProperties(Map<String, Object> targetProperties, Map<String, Object> toBeMergeProperties) {
+ private boolean mergeSystemProperties(Map<String, Object> targetProperties, Map<String, Object> sourceProperties) {
boolean changed = false;
- for (Map.Entry<String, Object> toBeMergeProperty : toBeMergeProperties.entrySet()) {
- if (toBeMergeProperty.getValue() != null) {
- if (!targetProperties.containsKey(toBeMergeProperty.getKey())) {
- targetProperties.put(toBeMergeProperty.getKey(), toBeMergeProperty.getValue());
+ for (Map.Entry<String, Object> sourceProperty : sourceProperties.entrySet()) {
+ if (sourceProperty.getValue() != null) {
+ if (!targetProperties.containsKey(sourceProperty.getKey())) {
+ targetProperties.put(sourceProperty.getKey(), sourceProperty.getValue());
changed = true;
} else {
- Object targetProperty = targetProperties.get(toBeMergeProperty.getKey());
+ Object targetProperty = targetProperties.get(sourceProperty.getKey());
- if (targetProperty instanceof Map && toBeMergeProperty.getValue() instanceof Map) {
+ if (targetProperty instanceof Map && sourceProperty.getValue() instanceof Map) {
// merge Maps like "goals", "campaigns"
@SuppressWarnings("unchecked")
- Map<String, Object> mapToBeMergeProp = (Map<String, Object>) toBeMergeProperty.getValue();
+ Map<String, Object> mapSourceProp = (Map<String, Object>) sourceProperty.getValue();
@SuppressWarnings("unchecked")
Map<String, Object> mapTargetProp = (Map<String, Object>) targetProperty;
- for (Map.Entry<String, ?> mapToBeMergeEntry : mapToBeMergeProp.entrySet()) {
- if (!mapTargetProp.containsKey(mapToBeMergeEntry.getKey())) {
- mapTargetProp.put(mapToBeMergeEntry.getKey(), mapToBeMergeEntry.getValue());
+ for (Map.Entry<String, ?> mapSourceEntry : mapSourceProp.entrySet()) {
+ if (!mapTargetProp.containsKey(mapSourceEntry.getKey())) {
+ mapTargetProp.put(mapSourceEntry.getKey(), mapSourceEntry.getValue());
changed = true;
}
}
- } else if (targetProperty instanceof Collection && toBeMergeProperty.getValue() instanceof Collection) {
+ } else if (targetProperty instanceof Collection && sourceProperty.getValue() instanceof Collection) {
// merge Collections like "lists"
- Collection collectionToBeMerge = (Collection) toBeMergeProperty.getValue();
- Collection collectionTarget = (Collection) targetProperty;
+ Collection sourceCollection = (Collection) sourceProperty.getValue();
+ Collection targetCollection = (Collection) targetProperty;
- for (Object itemToBeMerge : collectionToBeMerge) {
- if (!collectionTarget.contains(itemToBeMerge)) {
+ for (Object sourceItem : sourceCollection) {
+ if (!targetCollection.contains(sourceItem)) {
try {
- collectionTarget.add(itemToBeMerge);
+ targetCollection.add(sourceItem);
changed = true;
} catch (Exception e) {
// may be Collection type issue