Changeset 323:5f6c6c0fb14e for kraken-dom/src/main/java/org/krakenapps/dom/api/impl/AdminApiImpl.java

Timestamp:

09/11/11 17:21:42 (8 months ago)

Author:

delmitz

Branch:

default

Convert:

svn:7c3792e6-d75b-4784-96a6-b298f655ee64/trunk@2757

Message:

• fixed NPE.
• added getAdminByLoginName.

File:

• kraken-dom/src/main/java/org/krakenapps/dom/api/impl/AdminApiImpl.java (modified) (7 diffs)

kraken-dom/src/main/java/org/krakenapps/dom/api/impl/AdminApiImpl.java

@@ -98 +98 @@
                                                         }
                                                 } else if (loggedIn.size() >= maxSessions) {
-                                                        Session peek = loggedIn.peek().session;
-                                                        throw new MaxSessionException(peek.getAdminId(), peek);
+                                                        LoggedInAdmin peek = loggedIn.peek();
+                                                        throw new MaxSessionException(peek.loginName, peek.session);
                                                 }
                                         }
@@ -109 +109 @@
                         for (LoginCallback callback : callbacks)
                                 callback.onLoginSuccess(admin, session);
-                        loggedIn.add(new LoggedInAdmin(admin.getRole().getLevel(), new Date(), session));
+                        loggedIn.add(new LoggedInAdmin(admin.getRole().getLevel(), new Date(), session, admin.getUser()
+                                        .getLoginName()));
                         return admin;
                 } else {
@@ -123 +124 @@
                 private Date loginTime;
                 private Session session;
+                private String loginName;
 
                 private LoggedInAdmin(Session session) {
@@ -128 +130 @@
                 }
 
-                private LoggedInAdmin(int level, Date loginTime, Session session) {
+                private LoggedInAdmin(int level, Date loginTime, Session session, String loginName) {
                         this.level = level;
                         this.loginTime = loginTime;
                         this.session = session;
+                        this.loginName = loginName;
                 }
 
@@ -218 +221 @@
         private Admin getAdmin(String nick, Session session) {
                 try {
-                        EntityManager em = entityManagerService.getEntityManager();
-                        Admin admin = (Admin) em.createQuery("SELECT a FROM Admin a LEFT JOIN a.user u WHERE u.loginName = ?")
-                                        .setParameter(1, nick).getSingleResult();
+                        Admin admin = getAdminByLoginName(nick);
                         if (!admin.isEnabled()) {
                                 Date failed = admin.getLastLoginFailedDateTime();
@@ -232 +233 @@
                                         updateLoginFailures(admin, true);
                         }
-
                         admin.setLastLoginDateTime(new Date());
+
+                        EntityManager em = entityManagerService.getEntityManager();
                         em.merge(admin);
                         return admin;
@@ -277 +279 @@
                         return null;
 
+                return admin;
+        }
+
+        @Transactional
+        @Override
+        public Admin getAdminByLoginName(String loginName) {
+                EntityManager em = entityManagerService.getEntityManager();
+                Admin admin = (Admin) em.createQuery("SELECT a FROM Admin a LEFT JOIN a.user u WHERE u.loginName = ?")
+                                .setParameter(1, loginName).getSingleResult();
                 return admin;
         }