Commit a75a9374 authored by Matt Tucker's avatar Matt Tucker Committed by matt

User properties should be keyed on username and not name (JM-261).


git-svn-id: http://svn.igniterealtime.org/svn/repos/messenger/trunk@1263 b35dd754-fafc-0310-a699-88a17e54d16e
parent c21ee1d0
...@@ -255,6 +255,7 @@ public class User implements Cacheable { ...@@ -255,6 +255,7 @@ public class User implements Cacheable {
size += CacheSizes.sizeOfObject(); // overhead of object size += CacheSizes.sizeOfObject(); // overhead of object
size += CacheSizes.sizeOfLong(); // id size += CacheSizes.sizeOfLong(); // id
size += CacheSizes.sizeOfString(username); // username size += CacheSizes.sizeOfString(username); // username
size += CacheSizes.sizeOfString(name); // name
size += CacheSizes.sizeOfString(email); // email size += CacheSizes.sizeOfString(email); // email
size += CacheSizes.sizeOfDate() * 2; // creationDate and modificationDate size += CacheSizes.sizeOfDate() * 2; // creationDate and modificationDate
size += CacheSizes.sizeOfMap(properties); // properties size += CacheSizes.sizeOfMap(properties); // properties
...@@ -365,7 +366,7 @@ public class User implements Cacheable { ...@@ -365,7 +366,7 @@ public class User implements Cacheable {
try { try {
con = DbConnectionManager.getConnection(); con = DbConnectionManager.getConnection();
pstmt = con.prepareStatement(LOAD_PROPERTIES); pstmt = con.prepareStatement(LOAD_PROPERTIES);
pstmt.setString(1, name); pstmt.setString(1, username);
ResultSet rs = pstmt.executeQuery(); ResultSet rs = pstmt.executeQuery();
while (rs.next()) { while (rs.next()) {
properties.put(rs.getString(1), rs.getString(2)); properties.put(rs.getString(1), rs.getString(2));
...@@ -389,7 +390,7 @@ public class User implements Cacheable { ...@@ -389,7 +390,7 @@ public class User implements Cacheable {
try { try {
con = DbConnectionManager.getConnection(); con = DbConnectionManager.getConnection();
pstmt = con.prepareStatement(INSERT_PROPERTY); pstmt = con.prepareStatement(INSERT_PROPERTY);
pstmt.setString(1, name); pstmt.setString(1, username);
pstmt.setString(2, propName); pstmt.setString(2, propName);
pstmt.setString(3, propValue); pstmt.setString(3, propValue);
pstmt.executeUpdate(); pstmt.executeUpdate();
...@@ -413,7 +414,7 @@ public class User implements Cacheable { ...@@ -413,7 +414,7 @@ public class User implements Cacheable {
pstmt = con.prepareStatement(UPDATE_PROPERTY); pstmt = con.prepareStatement(UPDATE_PROPERTY);
pstmt.setString(1, propValue); pstmt.setString(1, propValue);
pstmt.setString(2, propName); pstmt.setString(2, propName);
pstmt.setString(3, name); pstmt.setString(3, username);
pstmt.executeUpdate(); pstmt.executeUpdate();
} }
catch (SQLException e) { catch (SQLException e) {
...@@ -433,7 +434,7 @@ public class User implements Cacheable { ...@@ -433,7 +434,7 @@ public class User implements Cacheable {
try { try {
con = DbConnectionManager.getConnection(); con = DbConnectionManager.getConnection();
pstmt = con.prepareStatement(DELETE_PROPERTY); pstmt = con.prepareStatement(DELETE_PROPERTY);
pstmt.setString(1, name); pstmt.setString(1, username);
pstmt.setString(2, propName); pstmt.setString(2, propName);
pstmt.executeUpdate(); pstmt.executeUpdate();
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment