Yair Zaslavsky has uploaded a new change for review.

Change subject: core: Fixing DbUserCacheManager when to refresh users
......................................................................

core: Fixing DbUserCacheManager when to refresh users

a. firstName of dbUser should be compared & set with firstName of DirectoryUser

Change-Id: Icec0ffa48671e4fed2b631a60db9dd226ad6d347
Bug-Url: https://bugzilla.redhat.com/1081204
Signed-off-by: Yair Zaslavsky <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DbUserCacheManager.java
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/32/26132/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DbUserCacheManager.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DbUserCacheManager.java
index bea4289..cae50ed 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DbUserCacheManager.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DbUserCacheManager.java
@@ -186,8 +186,8 @@
 
         // Compare the attributes of the database user with those of the 
directory, copy those that changed and update
         // the flag that indicates that the database needs to be updated:
-        if (!StringUtils.equals(dbUser.getFirstName(), 
directoryUser.getName())) {
-            dbUser.setFirstName(directoryUser.getName());
+        if (!StringUtils.equals(dbUser.getFirstName(), 
directoryUser.getFirstName())) {
+            dbUser.setFirstName(directoryUser.getFirstaName());
             update = true;
         }
         if (!StringUtils.equals(dbUser.getLastName(), 
directoryUser.getLastName())) {


-- 
To view, visit http://gerrit.ovirt.org/26132
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icec0ffa48671e4fed2b631a60db9dd226ad6d347
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to