From b26e487f3c65808d349c7a53c0ea2a3981b61bac Mon Sep 17 00:00:00 2001
From: Anton Tananaev <anton.tananaev@gmail.com>
Date: Wed, 20 Feb 2013 23:58:19 +1300
Subject: Add user preferences

---
 .../traccar/web/server/model/DataServiceImpl.java  | 27 +++++++++++-----------
 1 file changed, 14 insertions(+), 13 deletions(-)

(limited to 'src/org/traccar/web/server')

diff --git a/src/org/traccar/web/server/model/DataServiceImpl.java b/src/org/traccar/web/server/model/DataServiceImpl.java
index 1f6ff60..c2d9f72 100644
--- a/src/org/traccar/web/server/model/DataServiceImpl.java
+++ b/src/org/traccar/web/server/model/DataServiceImpl.java
@@ -102,8 +102,8 @@ public class DataServiceImpl extends RemoteServiceServlet implements DataService
     }
 
     @Override
-    public boolean authenticated() {
-        return (getUser() != null);
+    public User authenticated() {
+        return getUser();
     }
 
     @Override
@@ -152,19 +152,20 @@ public class DataServiceImpl extends RemoteServiceServlet implements DataService
         if (currentUser.getAdmin() || (currentUser.getId() == user.getId() && !user.getAdmin())) {
             EntityManager entityManager = entityManagerFactory.createEntityManager();
             try {
-                // TODO: better solution?
-                if (currentUser.getId() == user.getId()) {
-                    currentUser.setLogin(user.getLogin());
-                    currentUser.setPassword(user.getPassword());
-                    currentUser.setUserSettings(user.getUserSettings());
-                    user = currentUser;
-                } else {
-                    // TODO: handle other users
-                }
-
                 entityManager.getTransaction().begin();
                 try {
-                    entityManager.merge(user);
+                    // TODO: better solution?
+                    if (currentUser.getId() == user.getId()) {
+                        entityManager.merge(currentUser);
+                        currentUser.setLogin(user.getLogin());
+                        currentUser.setPassword(user.getPassword());
+                        currentUser.setUserSettings(user.getUserSettings());
+                        entityManager.merge(currentUser);
+                        user = currentUser;
+                    } else {
+                        // TODO: handle other users
+                    }
+
                     entityManager.getTransaction().commit();
                     setUser(user);
                     return user;
-- 
cgit v1.2.3