From d54f99d84fcd2bf908f1f0494472bdde8a5f8c2a Mon Sep 17 00:00:00 2001
From: Piotr Gawron <piotr.gawron@uni.lu>
Date: Fri, 7 Apr 2017 17:14:01 +0200
Subject: [PATCH] old fashion methods removed

---
 frontend-js/src/main/js/gui/UserPanel.js      | 76 ++++++-------------
 frontend-js/src/test/js/gui/UserPanel-test.js | 13 ++++
 2 files changed, 37 insertions(+), 52 deletions(-)

diff --git a/frontend-js/src/main/js/gui/UserPanel.js b/frontend-js/src/main/js/gui/UserPanel.js
index 8b97b12beb..ff1dd150d9 100644
--- a/frontend-js/src/main/js/gui/UserPanel.js
+++ b/frontend-js/src/main/js/gui/UserPanel.js
@@ -25,19 +25,21 @@ function UserPanel(params) {
     }
   });
   
-  self.getLoginButton().onclick = function(){
+  var loginButton = self.getControlElement(PanelControlElementType.USER_TAB_LOGIN_BUTTON);
+  var logoutButton = self.getControlElement(PanelControlElementType.USER_TAB_LOGOUT_BUTTON);
+  loginButton.onclick = function(){
     var login = self.getLoginText().value;
     var password = self.getPasswordText().value;
     
-    ServerConnector.login(login, password).then(function(){
+    return ServerConnector.login(login, password).then(function(){
       window.location.reload(false);
     }).catch(function(problem){
       GuiConnector.alert(problem);
     });
   };
   
-  self.getLogoutButton().onclick = function(){
-    ServerConnector.logout().then(function(){
+  logoutButton.onclick = function(){
+    return ServerConnector.logout().then(function(){
       window.location.reload(false);
     }).catch(function(problem){
       GuiConnector.alert(problem);
@@ -130,63 +132,33 @@ UserPanel.prototype._createUserDataTab = function() {
   this.setControlElement(PanelControlElementType.USER_TAB_LOGOUT_BUTTON, logoutButton);  
 };
 
-
-UserPanel.prototype.getUserLoginTab = function() {
-  return this.getElementByName(this.getElement(), "userLoginTab");
-};
-
-UserPanel.prototype.getPasswordText= function() {
-  return this.getElementByName(this.getElement(), "passwordText");
-};
-UserPanel.prototype.getLoginText = function() {
-  return this.getElementByName(this.getElement(), "loginText");
-};
-
-UserPanel.prototype.getUserDataTab = function() {
-  return this.getElementByName(this.getElement(), "userDataTab");
-};
-
-UserPanel.prototype.getLoginValue = function() {
-  return this.getElementByName(this.getElement(), "loginValue");
-};
-
-UserPanel.prototype.getLogoutButton = function() {
-  return this.getElementByName(this.getElement(), "logoutButton");
-};
-
-UserPanel.prototype.getLoginButton = function() {
-  return this.getElementByName(this.getElement(), "loginButton");
-};
-
-UserPanel.prototype.getNameValue = function() {
-  return this.getElementByName(this.getElement(), "nameValue");
-};
-
-UserPanel.prototype.getSurnameValue = function() {
-  return this.getElementByName(this.getElement(), "surnameValue");
-};
-
-UserPanel.prototype.getEmailValue = function() {
-  return this.getElementByName(this.getElement(), "emailValue");
-};
-
 UserPanel.prototype.showLoginPage = function() {
   var self = this;
-  self.getUserLoginTab().style.display = "block";
+  var userLoginTabDiv = self.getControlElement(PanelControlElementType.USER_TAB_LOGIN_DIV);
+  var userDataTabDiv = self.getControlElement(PanelControlElementType.USER_TAB_USER_DIV);
 
-  self.getUserDataTab().style.display = "none";
+  userLoginTabDiv.style.display = "block";
+
+  userDataTabDiv.style.display = "none";
 };
 
 UserPanel.prototype.showUserProfilePage = function(user) {
   var self = this;
-  self.getUserLoginTab().style.display = "none";
+  var userLoginTabDiv = self.getControlElement(PanelControlElementType.USER_TAB_LOGIN_DIV);
+  var userDataTabDiv = self.getControlElement(PanelControlElementType.USER_TAB_USER_DIV);
+  var loginText = self.getControlElement(PanelControlElementType.USER_TAB_LOGIN_TEXT);
+  var nameText = self.getControlElement(PanelControlElementType.USER_TAB_NAME_TEXT);
+  var surnameText = self.getControlElement(PanelControlElementType.USER_TAB_SURNAME_TEXT);
+  var emailText = self.getControlElement(PanelControlElementType.USER_TAB_EMAIL_TEXT);
+
+  userLoginTabDiv.style.display = "none";
 
-  self.getUserDataTab().style.display = "block";
+  userDataTabDiv.style.display = "block";
   
-  self.getLoginValue().innerHTML = user.getLogin(); 
-  self.getNameValue().innerHTML = user.getName(); 
-  self.getSurnameValue().innerHTML = user.getSurname(); 
-  self.getEmailValue().innerHTML = user.getEmail(); 
+  loginText.innerHTML = user.getLogin(); 
+  nameText.innerHTML = user.getName(); 
+  surnameText.innerHTML = user.getSurname(); 
+  emailText.innerHTML = user.getEmail(); 
 };
 
 module.exports = UserPanel;
diff --git a/frontend-js/src/test/js/gui/UserPanel-test.js b/frontend-js/src/test/js/gui/UserPanel-test.js
index b9b99bc8f7..a50463e2bb 100644
--- a/frontend-js/src/test/js/gui/UserPanel-test.js
+++ b/frontend-js/src/test/js/gui/UserPanel-test.js
@@ -42,4 +42,17 @@ describe('UserPanel', function() {
     var user = helper.createUser();
     panel.showUserProfilePage(user);
   });
+  
+  it('show login page', function() {
+    var div = helper.createUserTab();
+
+    var map = helper.createCustomMap();
+
+    var panel = new UserPanel({
+      element : div,
+      customMap : map
+    });
+    
+    panel.showLoginPage();
+  });
 });
-- 
GitLab