Merge pull request #147 from khfayzullaev/master

Admin should not add an existing username

1 files changed, 5 insertions, 0 deletions

diff --git a/src/org/traccar/web/client/controller/SettingsController.java b/src/org/traccar/web/client/controller/SettingsController.java
index f8fcc72d..44d48f3d 100644
--- a/src/org/traccar/web/client/controller/SettingsController.java
+++ b/src/org/traccar/web/client/controller/SettingsController.java
@@ -33,6 +33,7 @@ import org.traccar.web.shared.model.UserSettings;
 import com.google.gwt.core.client.GWT;
 import com.sencha.gxt.data.shared.ListStore;
 import com.sencha.gxt.widget.core.client.Dialog.PredefinedButton;
+import com.sencha.gxt.widget.core.client.box.AlertMessageBox;
 import com.sencha.gxt.widget.core.client.box.ConfirmMessageBox;
 import com.sencha.gxt.widget.core.client.event.HideEvent;
 
@@ -97,6 +98,10 @@ public class SettingsController implements DeviceView.SettingsHandler {
                                             public void onSuccess(User result) {
                                                 userStore.add(result);
                                             }
+                                            @Override
+                                            public void onFailure(Throwable caught) {
+                                                new AlertMessageBox("Error", "Username is already taken").show();
+                                            }
                                         });
                                     }
                                 }).show();