From 8c1daa577125c65beb073e584c44a6a3994db005 Mon Sep 17 00:00:00 2001
From: Nikolay <46225163+vigdorov@users.noreply.github.com>
Date: Sat, 1 Aug 2020 17:32:59 +0300
Subject: [PATCH] =?UTF-8?q?HM-79.=20=D0=94=D0=BE=D0=B1=D0=B0=D0=B2=D0=BB?=
 =?UTF-8?q?=D0=B5=D0=BD=D0=B0=20=D1=80=D0=B0=D0=B1=D0=BE=D1=82=D0=B0=20?=
 =?UTF-8?q?=D1=84=D0=BE=D1=80=D0=BC=D1=8B=20=D1=81=20=D1=80=D1=83=D1=87?=
 =?UTF-8?q?=D0=BA=D0=BE=D0=B9,=20=D0=B2=D0=B0=D0=BB=D0=B8=D0=B4=D0=B0?=
 =?UTF-8?q?=D1=86=D0=B8=D1=8F=20=D1=84=D0=BE=D1=80=D0=BC=D1=8B.=20HM-88.?=
 =?UTF-8?q?=20=D0=94=D0=BE=D0=B1=D0=B0=D0=B2=D0=BB=D0=B5=D0=BD=D1=8B=20?=
 =?UTF-8?q?=D0=BF=D1=83=D0=BD=D0=BA=D1=82=D1=8B=20=D0=BC=D0=B5=D0=BD=D1=8E?=
 =?UTF-8?q?=20=D1=81=20js.=20HM-76.=20=D0=94=D0=BE=D0=B1=D0=B0=D0=B2=D0=BB?=
 =?UTF-8?q?=D0=B5=D0=BD=D0=B0=20=D0=BF=D0=BE=D0=B4=D1=81=D0=B2=D0=B5=D1=82?=
 =?UTF-8?q?=D0=BA=D0=B0=20=D1=8D=D0=BB=D0=B5=D0=BC=D0=B5=D0=BD=D1=82=D0=BE?=
 =?UTF-8?q?=D0=B2=20=D0=BC=D0=B5=D0=BD=D1=8E=20(#40)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/api/LocalStorageAPI.js                    |  6 +-
 src/api/TokenAPI.js                           |  3 +-
 src/app.html                                  | 41 +++------
 src/app.js                                    |  9 +-
 src/components/form-control/FormControl.js    | 17 ++++
 src/components/login-form/LoginForm.js        | 73 +++++++++++++++
 src/components/login-form/index.js            |  3 +
 .../{loginPage => login-page}/LoginPage.css   |  8 +-
 src/components/login-page/LoginPage.js        | 29 ++++++
 src/components/login-page/index.js            |  3 +
 src/components/loginPage/LoginPage.js         | 38 --------
 .../MainMenu.css}                             |  4 +
 src/components/main-menu/MainMenu.js          | 89 +++++++++++++++++++
 src/components/main-menu/index.js             |  3 +
 .../NavButtonComponent.js                     | 33 -------
 .../navigation-buttons-component/constants.js | 18 ----
 .../navigation-buttons-component/index.js     |  3 -
 .../RouterPagesContainer.js                   |  9 +-
 18 files changed, 252 insertions(+), 137 deletions(-)
 create mode 100644 src/components/login-form/LoginForm.js
 create mode 100644 src/components/login-form/index.js
 rename src/components/{loginPage => login-page}/LoginPage.css (88%)
 create mode 100644 src/components/login-page/LoginPage.js
 create mode 100644 src/components/login-page/index.js
 delete mode 100644 src/components/loginPage/LoginPage.js
 rename src/components/{navigation-buttons-component/NavButtonComponent.css => main-menu/MainMenu.css} (88%)
 create mode 100644 src/components/main-menu/MainMenu.js
 create mode 100644 src/components/main-menu/index.js
 delete mode 100644 src/components/navigation-buttons-component/NavButtonComponent.js
 delete mode 100644 src/components/navigation-buttons-component/constants.js
 delete mode 100644 src/components/navigation-buttons-component/index.js

diff --git a/src/api/LocalStorageAPI.js b/src/api/LocalStorageAPI.js
index 31e6a58..0ba320d 100644
--- a/src/api/LocalStorageAPI.js
+++ b/src/api/LocalStorageAPI.js
@@ -16,7 +16,7 @@ class LocalStorageAPI {
      * Возвращает распарсенный объект из Local Storage по ключу из конструктора
      */
     request () {
-        const value = localStorage.getItem(this.key) || '{}';
+        const value = this.api.getItem(this.key) || '{}';
         return JSON.parse(value);
     }
 
@@ -25,14 +25,14 @@ class LocalStorageAPI {
      * @param {Object} value - значение в Local Storage
      */
     createOrUpdate (value) {
-        localStorage.setItem(this.key, JSON.stringify(value));
+        this.api.setItem(this.key, JSON.stringify(value));
     }
 
     /**
      * Очищает значение Local Storage по ключу из конструктора
      */
     remove () {
-        localStorage.removeItem(this.key);
+        this.api.removeItem(this.key);
     }
 }
 
diff --git a/src/api/TokenAPI.js b/src/api/TokenAPI.js
index c328844..0e17230 100644
--- a/src/api/TokenAPI.js
+++ b/src/api/TokenAPI.js
@@ -1,4 +1,5 @@
 import LocalStorageAPI from './LocalStorageAPI';
+import {LOCAL_STORAGE_TYPE} from './consts';
 
 const API_NAME = 'storageServiceUITokenApi';
 
@@ -16,7 +17,7 @@ const API_NAME = 'storageServiceUITokenApi';
 class TokenApi {
     constructor () {
         this.localApi = new LocalStorageAPI(API_NAME);
-        this.sessionApi = new LocalStorageAPI(API_NAME);
+        this.sessionApi = new LocalStorageAPI(API_NAME, LOCAL_STORAGE_TYPE.SESSION);
     }
 
     /**
diff --git a/src/app.html b/src/app.html
index 6448a37..3dccd63 100644
--- a/src/app.html
+++ b/src/app.html
@@ -21,17 +21,7 @@
                     <span class="navbar-toggler-icon"></span>
                 </button>
                 <div class="collapse navbar-collapse" id="navbarNavDropdown">
-                    <ul class="navbar-nav">
-                        <li class="nav-item">
-                            <a class="nav-link active" aria-current="page" href="/">Главная</a>
-                        </li>
-                        <li class="nav-item">
-                            <a class="nav-link" href="/api">Список хранилищ</a>
-                        </li>
-                        <li class="nav-item">
-                            <a class="nav-link" href="/logs">Журнал</a>
-                        </li>
-                    </ul>
+                    <ul class="navbar-nav"></ul>
                 </div>
         </nav>
     </template>
@@ -83,24 +73,17 @@
     <template id="login-page">
         <div class="Login__page">
             <div class="Login__page-container btn-prymary">
-                <div class="Login__logo-box"></div>
+                <div class="Login__logo-box">
+                    <img class="Login__logo" alt="logo">
+                </div>
                 <h3 class="Login__title">Storage service v0.01</h3>
-                <form class="Login__form">
-                    <div class="form-group Login__input">
-                        <label for="login  btn-primary">Логин пользователя:</label>
-                        <input type="text" class="form-control" id="login" aria-describedby="loginError">
-                        <small id="loginError" class="form-text text-muted"></small>
-                      </div>
-                      <div class="form-group Login__input">
-                        <label for="password">Пароль пользователя:</label>
-                        <input type="password" class="form-control" id="password" aria-describedby="passwordError">
-                        <small id="passwordError" class="form-text text-muted"></small>
-                      </div>
-                      <div class="form-group form-check Login__check">
+                <form class="Login__form needs-validation" novalidate>
+                    <div class="Login__inputContainer"></div>
+                    <div class="form-group form-check Login__check">
                         <input type="checkbox" class="form-check-input" id="check">
                         <label class="form-check-label" for="check">Оставаться в системе</label>
-                      </div>
-                      <button type="submit" class="btn btn-primary Login__submit">Войти</button>
+                    </div>
+                    <button type="submit" class="btn btn-primary Login__submit">Войти</button>
                 </form>
             </div>
         </div>
@@ -108,7 +91,7 @@
 
     <!-- Шаблон контейнера страниц-->
     <template id="page-container">
-        <div class="PageContainer"></div>
+        <div class="PageContainer h-100"></div>
     </template>
     <!-- Шаблон кнопки-->
     <template id="button">
@@ -117,7 +100,7 @@
 
     <!-- Шаблон страниц -->
     <template id="page">
-        <div class="Page"></div>
+        <div class="Page h-100"></div>
     </template>
 
     <!-- Шаблон Modal -->
@@ -332,7 +315,7 @@
     <template id="form-control">
         <div class="mb-3">
             <label class="form-label"></label>
-            <div class="form-text"></div>
+            <div class="form-text invalid-feedback"></div>
         </div>
     </template>
 
diff --git a/src/app.js b/src/app.js
index b397b5c..cacde27 100644
--- a/src/app.js
+++ b/src/app.js
@@ -5,16 +5,15 @@ import 'bootstrap';
 import './services/AdminConfigsService';
 import ApiPage from './components/api-page';
 import MainPage from './components/main-page';
-import navMenuButtons from './components/navigation-buttons-component/NavButtonComponent';
-import {NAV_MENU} from './components/navigation-buttons-component/constants';
+import mainMenu from './components/main-menu/MainMenu';
 import routeService from './services/RouteService';
 import RouterPagesContainer from './components/router-pages-container/index';
 import LogsPage from './components/logs-page/index';
-import LoginPage from './components/loginPage/LoginPage';
+import LoginPage from './components/login-page';
 
-navMenuButtons.render(NAV_MENU);
+mainMenu.render();
 
-const routerPagesContainer = new RouterPagesContainer();
+const routerPagesContainer = new RouterPagesContainer(mainMenu);
 
 /**
  * Добавление страниц в Роутинг выполняется на странице app.js
diff --git a/src/components/form-control/FormControl.js b/src/components/form-control/FormControl.js
index f43d38b..ed68ecc 100644
--- a/src/components/form-control/FormControl.js
+++ b/src/components/form-control/FormControl.js
@@ -10,6 +10,7 @@ class FormControl extends Component {
         placeholder = '',
         initValue = '',
         className = '',
+        required = false,
     } = {}) {
         super('#form-control', parentNode);
 
@@ -19,6 +20,10 @@ class FormControl extends Component {
             tagName: this.getInputTagName(type),
             options: {
                 className: `form-control ${className}`,
+            },
+            args: {
+                type: type === FORM_TYPES.PASSWORD ? 'password' : 'text',
+                ...(required ? {required: ''} : {}),
             }
         });
         this.input.placeholder = placeholder;
@@ -28,6 +33,10 @@ class FormControl extends Component {
         this.label.insertAdjacentElement('afterend', this.input);
         this.label.textContent = label;
         this.label.setAttribute('for', id);
+
+        this.addEventListener(this.input, 'focus', this.clearError);
+        this.addEventListener(this.input, 'click', this.clearError);
+        this.addEventListener(this.input, 'keydown', this.clearError);
     }
 
     disabled = (value) => {
@@ -42,6 +51,14 @@ class FormControl extends Component {
         this.input.value = value;
     }
 
+    setError = (errorMessage) => {
+        this.errorText.textContent = errorMessage;
+    }
+
+    clearError = () => {
+        this.errorText.textContent = '';
+    }
+
     getInputTagName (type) {
         switch (type) {
             case FORM_TYPES.TEXT:
diff --git a/src/components/login-form/LoginForm.js b/src/components/login-form/LoginForm.js
new file mode 100644
index 0000000..5b91c81
--- /dev/null
+++ b/src/components/login-form/LoginForm.js
@@ -0,0 +1,73 @@
+import Component from '../component';
+import Image from '../../img/logo.svg';
+import FormControl from '../form-control';
+import {FORM_TYPES} from '../../consts';
+
+class LoginForm extends Component {
+    constructor (parentNode) {
+        super('#login-page', parentNode);
+
+        this.logoBox = this.mainNode.querySelector('.Login__logo-box');
+        this.form = this.mainNode.querySelector('.Login__form');
+        this.inputContainer = this.mainNode.querySelector('.Login__inputContainer');
+        this.submitButton = this.mainNode.querySelector('.Login__submit');
+        this.checkboxSystem = this.mainNode.querySelector('.form-check-input');
+        this.logoImage = this.mainNode.querySelector('.Login__logo');
+        this.logoImage.src = Image;
+
+        this.loginControl = new FormControl(this.inputContainer, {
+            label: 'Логин:',
+            id: 'login-form-user-login',
+            placeholder: 'Введите логин',
+            required: true,
+        });
+        this.passwordControl = new FormControl(this.inputContainer, {
+            label: 'Пароль:',
+            id: 'login-form-user-password',
+            placeholder: 'Введите пароль',
+            type: FORM_TYPES.PASSWORD,
+            required: true,
+        });
+
+        this.addEventListener(this.form, 'submit', this.submit);
+    }
+
+    disabled = (value) => {
+        const elements = [
+            this.loginControl.input,
+            this.passwordControl.input,
+            this.submitButton,
+            this.checkboxSystem,
+        ];
+
+        elements.forEach((element) => {
+            element.disabled = value;
+        });
+    }
+
+    validateInputs = () => {
+        this.form.classList.add('was-validated');
+        const login = this.loginControl.getValue();
+        const password = this.passwordControl.getValue();
+
+        if (!login) {
+            this.loginControl.setError('Заполните логин');
+        }
+        if (!password) {
+            this.passwordControl.setError('Заполните пароль');
+        }
+        return this.form.checkValidity();
+    }
+
+    submit = (event) => {
+        event.preventDefault();
+        if (this.validateInputs()) {
+            this.next('submit', {
+                login: this.loginControl.getValue(),
+                password: this.passwordControl.getValue(),
+            });
+        }
+    }
+}
+
+export default LoginForm;
diff --git a/src/components/login-form/index.js b/src/components/login-form/index.js
new file mode 100644
index 0000000..e9607b6
--- /dev/null
+++ b/src/components/login-form/index.js
@@ -0,0 +1,3 @@
+import LoginForm from './LoginForm';
+
+export default LoginForm;
diff --git a/src/components/loginPage/LoginPage.css b/src/components/login-page/LoginPage.css
similarity index 88%
rename from src/components/loginPage/LoginPage.css
rename to src/components/login-page/LoginPage.css
index 9cba99c..b735833 100644
--- a/src/components/loginPage/LoginPage.css
+++ b/src/components/login-page/LoginPage.css
@@ -1,8 +1,4 @@
-.Login__page {
-    position: fixed;
-    top: 0;
-    left: 0;
-    width: 100%;
+.Login__page { 
     height: 100%;
     background-color: white;
     display: flex;
@@ -40,4 +36,4 @@
 .Login__logo-box {
     display: flex;
     align-items: center;
-}
\ No newline at end of file
+}
diff --git a/src/components/login-page/LoginPage.js b/src/components/login-page/LoginPage.js
new file mode 100644
index 0000000..bb2d38a
--- /dev/null
+++ b/src/components/login-page/LoginPage.js
@@ -0,0 +1,29 @@
+import Component from '../component/index';
+import './LoginPage.css';
+import LoginForm from '../login-form';
+import authServiceApi from '../../api/AuthServiceAPI';
+import routeService from '../../services/RouteService';
+
+class LoginPage extends Component {
+    constructor (mainNodeSelector, parentNode) {
+        super(mainNodeSelector, parentNode);
+
+        this.form = new LoginForm(this.mainNode);
+
+        this.addSubscribe(this.form, 'submit', ({login, password}) => {
+            this.form.disabled(true);
+            authServiceApi.auth(login, password)
+                .then(() => {
+                    this.form.disabled(false);
+                    routeService.goTo('/');
+                })
+                .catch((e) => {
+                    // TODO: Времено используется alert, потом прикрутим систему нотификаций
+                    // eslint-disable-next-line no-alert
+                    alert(e?.response?.data?.message || 'Неизвестная ошибка');
+                    this.form.disabled(false);
+                });
+        });
+    }
+}
+export default LoginPage;
diff --git a/src/components/login-page/index.js b/src/components/login-page/index.js
new file mode 100644
index 0000000..6c9595d
--- /dev/null
+++ b/src/components/login-page/index.js
@@ -0,0 +1,3 @@
+import LoginPage from './LoginPage';
+
+export default LoginPage;
diff --git a/src/components/loginPage/LoginPage.js b/src/components/loginPage/LoginPage.js
deleted file mode 100644
index 2b40861..0000000
--- a/src/components/loginPage/LoginPage.js
+++ /dev/null
@@ -1,38 +0,0 @@
-import Component from '../component/index';
-import './LoginPage.css';
-import {createElement} from '../../utils/elementUtils';
-import Image from '../../img/logo.svg';
-
-class LoginPage extends Component {
-    constructor () {
-        super('#login-page', document.body);
-
-        this.logoBox = this.mainNode.querySelector('.Login__logo-box');
-        this.form = this.mainNode.querySelector('.Login__form');
-        this.submitBtn = this.mainNode.querySelector('.Login__submit');
-        this.logoImg = createElement({
-            tagName: 'img',
-            parentNode: this.logoBox,
-            options: {
-                className: 'Login__logo',
-            },
-            args: {
-                src: Image,
-                alt: 'logo'
-            }
-        });
-
-        this.addEventListener(this.form, 'submit', (evt) => {
-            this.send(evt);
-        });
-        this.addEventListener(this.submitBtn, 'click', (evt) => {
-            this.send(evt);
-        });
-
-    }
-
-    send = (evt) => {
-        evt.preventDefault();
-    }
-}
-export default LoginPage;
diff --git a/src/components/navigation-buttons-component/NavButtonComponent.css b/src/components/main-menu/MainMenu.css
similarity index 88%
rename from src/components/navigation-buttons-component/NavButtonComponent.css
rename to src/components/main-menu/MainMenu.css
index b7fd956..a86367d 100644
--- a/src/components/navigation-buttons-component/NavButtonComponent.css
+++ b/src/components/main-menu/MainMenu.css
@@ -7,6 +7,10 @@
     margin: 5px 10px;
 }
 
+.nav-item {
+    cursor: pointer;
+}
+
 @media (max-width: 900px){
     .Buttons__container {
         flex-direction: column;
diff --git a/src/components/main-menu/MainMenu.js b/src/components/main-menu/MainMenu.js
new file mode 100644
index 0000000..a11c5f9
--- /dev/null
+++ b/src/components/main-menu/MainMenu.js
@@ -0,0 +1,89 @@
+import Component from '../component/index';
+import routeService from '../../services/RouteService';
+import Image from '../../img/logo.svg';
+
+import './MainMenu.css';
+import {createElement} from '../../utils/elementUtils';
+import {EVENTS} from '../../consts';
+
+export const NAV_MENU = [
+    {
+        title: 'Главная',
+        url: '/'
+    },
+    {
+        title: 'Список хранилищ',
+        url: '/api'
+    },
+    {
+        title: 'Журнал',
+        url: '/logs'
+    },
+];
+
+class MainMenu extends Component {
+    menuItems = [];
+
+    constructor () {
+        super('#main-menu', document.body);
+
+        this.buttonsContainer = this.mainNode.querySelector('.navbar-nav');
+        this.logoImg = document.createElement('img');
+        this.logoImg.src = Image;
+        this.logoImg.alt = 'logo';
+        this.logoImg.className = 'Logo mr-2';
+        this.logoBox = this.mainNode.querySelector('.Logo__box');
+        this.logoBox.appendChild(this.logoImg);
+
+        this.addSubscribe(routeService, EVENTS.ROUTE_CHANGE, (route) => {
+            this.menuItems.forEach(({url, link}) => {
+                if (route.url === url) {
+                    link.classList.add('active');
+                } else {
+                    link.classList.remove('active');
+                }
+            });
+        });
+    }
+
+    render = () => {
+        this.menuItems = NAV_MENU.map(({url, title}) => {
+            const li = createElement({
+                tagName: 'li',
+                parentNode: this.buttonsContainer,
+                options: {
+                    className: 'nav-item',
+                },
+            });
+            const link = createElement({
+                tagName: 'a',
+                parentNode: li,
+                options: {
+                    className: 'nav-link',
+                    textContent: title,
+                },
+            });
+            this.addEventListener(li, 'click', () => {
+                routeService.goTo(url);
+            });
+
+            return {url, link};
+        });
+    }
+
+    isHide = () => {
+        return this.mainNode.parentNode;
+    }
+
+    hideMenu = () => {
+        this.mainNode.remove();
+    }
+
+    showMenu = () => {
+        document.body.prepend(this.mainNode);
+    }
+}
+
+const navMenuButtons = new MainMenu();
+
+export default navMenuButtons;
diff --git a/src/components/main-menu/index.js b/src/components/main-menu/index.js
new file mode 100644
index 0000000..59c85a0
--- /dev/null
+++ b/src/components/main-menu/index.js
@@ -0,0 +1,3 @@
+import MainMenu from './MainMenu';
+
+export default MainMenu;
diff --git a/src/components/navigation-buttons-component/NavButtonComponent.js b/src/components/navigation-buttons-component/NavButtonComponent.js
deleted file mode 100644
index 6aef50d..0000000
--- a/src/components/navigation-buttons-component/NavButtonComponent.js
+++ /dev/null
@@ -1,33 +0,0 @@
-import Component from '../component/index';
-import routeService from '../../services/RouteService';
-import ButtonComponent from '../button-component/ButtonComponent';
-import Image from '../../img/logo.svg';
-
-import './NavButtonComponent.css';
-
-class NavButtonComponent extends Component {
-    constructor () {
-        super('#main-menu', document.body);
-
-        this.buttonsContainer = this.mainNode.querySelector('.Buttons__container');
-        this.logoImg = document.createElement('img');
-        this.logoImg.src = Image;
-        this.logoImg.alt = 'logo';
-        this.logoImg.className = 'Logo mr-2';
-        this.logoBox = this.mainNode.querySelector('.Logo__box');
-        this.logoBox.appendChild(this.logoImg);
-    }
-
-    render = (menu) => {
-        menu.forEach((element) => {
-            this.button = new ButtonComponent(this.buttonsContainer, element.title, 'btn btn-outline-primary NavButton');
-            this.button.subscribe('click', () => {
-                routeService.goTo(element.url);
-            });
-        });
-    }
-}
-
-const navMenuButtons = new NavButtonComponent();
-
-export default navMenuButtons;
diff --git a/src/components/navigation-buttons-component/constants.js b/src/components/navigation-buttons-component/constants.js
deleted file mode 100644
index 565c577..0000000
--- a/src/components/navigation-buttons-component/constants.js
+++ /dev/null
@@ -1,18 +0,0 @@
-export const NAV_MENU = [
-    {
-        title: 'Главная',
-        url: '/'
-    },
-    {
-        title: 'Список хранилищ',
-        url: '/api'
-    },
-    {
-        title: 'Журнал',
-        url: '/logs'
-    },
-    {
-        title: 'Контакты',
-        url: '/'
-    },
-];
diff --git a/src/components/navigation-buttons-component/index.js b/src/components/navigation-buttons-component/index.js
deleted file mode 100644
index 6092c4a..0000000
--- a/src/components/navigation-buttons-component/index.js
+++ /dev/null
@@ -1,3 +0,0 @@
-import NavButtonComponent from './NavButtonComponent';
-
-export default NavButtonComponent;
diff --git a/src/components/router-pages-container/RouterPagesContainer.js b/src/components/router-pages-container/RouterPagesContainer.js
index a46328c..65efbeb 100644
--- a/src/components/router-pages-container/RouterPagesContainer.js
+++ b/src/components/router-pages-container/RouterPagesContainer.js
@@ -33,7 +33,7 @@ class RouterPagesContainer extends Component {
      */
     url;
 
-    constructor () {
+    constructor (mainMenu) {
         super('#page-container', document.body);
 
         this.addSubscribe(routeService, EVENTS.ROUTE_CHANGE, ({url}) => {
@@ -42,6 +42,13 @@ class RouterPagesContainer extends Component {
                 return route.url === url;
             }) || {};
 
+            // Показывает или прячет меню в зависимости от роут
+            if (['/login'].includes(url)) {
+                mainMenu.hideMenu();
+            } else {
+                mainMenu.showMenu();
+            }
+
             // Рендерит новую страницу, если url изменился
             if (url !== this.currentUrl) {
                 // Удаляет предыдущую страницу