Update Snapshots

Add & update translations

See merge request turris/reforis/foris-js!156

.eslintrc.js

@@ -1,6 +1,8 @@
 module.exports = {
-    extends: "eslint-config-reforis",
+    extends: ["eslint-config-reforis", "prettier"],
+    plugins: ["prettier"],
     rules: {
+        "prettier/prettier": ["error"],
         "import/prefer-default-export": "off",
     },
 };

.gitignore

@@ -51,3 +51,4 @@ coverage.xml
 dist/
 foris-*.tgz
 styleguide/
+testUtils

.gitlab-ci.yml

@@ -1,44 +1,44 @@
-image: node:8-alpine
+image: node:10-alpine
 
 stages:
-  - test
+    - test
-  - build
+    - build
-  - publish
+    - publish
 
 before_script:
-  - apk add make
+    - apk add make
-  - npm install
+    - npm install
 
 test:
-  stage: test
+    stage: test
-  script:
+    script:
-    - make test
+        - make test
 
 lint:
-  stage: test
+    stage: test
-  script:
+    script:
-    - make lint
+        - make lint
 
 build:
-  stage: build
+    stage: build
-  script:
+    script:
-    - make pack
+        - make pack
-  artifacts:
+    artifacts:
-    paths:
+        paths:
-      - dist/foris-*.tgz
+            - dist/foris-*.tgz
 
 publish_beta:
-  stage: publish
+    stage: publish
-  only:
+    only:
-    refs:
+        refs:
-      - dev
+            - dev
-  script:
+    script:
-    - make publish-beta
+        - make publish-beta
 
 publish_latest:
-  stage: publish
+    stage: publish
-  only:
+    only:
-    refs:
+        refs:
-      - master
+            - master
-  script:
+    script:
-    - make publish-latest
+        - make publish-latest

.prettierrc

@@ -0,0 +1,11 @@
+{
+    "singleQuote": false,
+    "printWidth": 80,
+    "proseWrap": "always",
+    "tabWidth": 4,
+    "useTabs": false,
+    "trailingComma": "es5",
+    "bracketSpacing": true,
+    "jsxBracketSameLine": false,
+    "semi": true
+}

.weblate

@@ -0,0 +1,3 @@
+[weblate]
+url = https://hosted.weblate.org/api/
+translation = turris/foris-js

Makefile

@@ -1,6 +1,6 @@
 .PHONY: all install-js watch-js build-js collect-files pack publish-beta publish-latest lint test test-js-update-snapshots create-messages update-messages docs docs-watch clean
 
-DEV_PYTHON=python3.7
+DEV_PYTHON=python3
 VENV_NAME?=venv
 VENV_BIN=$(shell pwd)/$(VENV_NAME)/bin
 

README.md

@@ -1,4 +1,5 @@
 # foris-js
+
 Set of utils and common React elements for reForis.
 
 ## Publishing package
@@ -6,13 +7,46 @@ Set of utils and common React elements for reForis.
 ### Beta versions
 
 Each commit to `dev` branch will result in publishing a new version of library
-tagged `beta`. Versions names are based on commit SHA, e.g. 
+tagged `beta`. Versions names are based on commit SHA, e.g.
 `foris@0.1.0-beta.d9073aa4`.
 
 ### Preparing a release
 
 1. Crete a merge request to `dev` branch with version bumped
 2. When merging add `[skip ci]` to commit message to prevent publishing
-unnecessary version
+   unnecessary version
 3. Create a merge request from `dev` to `master` branch
 4. New version should be published automatically
+
+## Manually managed dependencies
+
+Because of `<ForisForm />` component it's required to use exposed
+`ReactRouterDOM` object from `react-router-dom` library. `ReactRouterDOM` is
+exposed by
+[reForis](https://gitlab.labs.nic.cz/turris/reforis/reforis/blob/master/js/webpack.config.js).
+It can be done by following steps:
+
+1. Setting `react-router-dom` as `peerDependencies` and `devDependencies` in
+   `package.json`.
+2. Adding the following rules to `externals` in `webpack.conf.js` of the plugin:
+
+```js
+externals: {
+    ...
+    "react-router-dom": "ReactRouterDOM",
+}
+```
+
+### Docs
+
+Build or watch docs to get more info about library:
+
+```bash
+make docs
+```
+
+or
+
+```bash
+make docs-watch
+```

babel.cfg

@@ -0,0 +1 @@
+[javascript: src/**.js]

babel.config.js

@@ -1,9 +1,4 @@
 module.exports = {
-    presets: [
+    presets: ["@babel/preset-env", "@babel/preset-react"],
-        "@babel/preset-env",
+    plugins: ["@babel/plugin-transform-runtime"],
-        "@babel/preset-react",
-    ],
-    plugins: [
-        "@babel/plugin-transform-runtime",
-    ],
 };

docs/development.md

@@ -0,0 +1,25 @@
+Sooner or later you will face with situation when you want/need to make some
+changes in the library. Then the most important tool for you it's
+[`npm link`](https://docs.npmjs.com/cli/link).
+
+Please, notice that it will not work if you link library just from root of the
+repo. It happens due to location of sources `./src`. You need to pack library
+first `make pack` and then link it from `./dist` directory.
+
+Yeah it's not such comfortable solution for development. But it can fixed by
+writing small script similar as `make pack` but with linking every file and
+directory from `./src` to the some directory and linking then from it. Notice
+that you need to link `package.json` and `package-lock.json` as well.
+
+So step by step:
+
+```bash
+make pack;
+cd dist;
+npm link;
+
+cd $project_dir/js # Navigate to JS directory of the project where you want to link the library
+npm link foris
+```
+
+And that's it ;)

docs/intro.md

@@ -1 +1,6 @@
-Foris JS library is set of componets and utils for Foris JS application and plugins.
+Foris JS library is set of components and utils for Foris JS application and
+plugins.
+
+Please notice that all of these components or utils are used in reForis and
+plugins. If you like to study by example I would recommend to full-text search
+these repos.

jest.config.js

@@ -27,7 +27,5 @@ module.exports = {
     globals: {
         TZ: "utc",
     },
-    transformIgnorePatterns: [
+    transformIgnorePatterns: ["node_modules/(?!(react-datetime)/)"],
-        "node_modules/(?!(react-datetime)/)",
-    ],
 };

package.json

@@ -1,62 +1,70 @@
 {
-  "name": "foris",
+    "name": "foris",
-  "version": "3.1.1",
+    "version": "5.1.11",
-  "description": "Set of components and utils for Foris and its plugins.",
+    "description": "Set of components and utils for Foris and its plugins.",
-  "author": "CZ.NIC, z.s.p.o.",
+    "author": "CZ.NIC, z.s.p.o.",
-  "repository": {
+    "repository": {
-    "type": "git",
+        "type": "git",
-    "url": "https://gitlab.labs.nic.cz/turris/reforis/foris-js.git"
+        "url": "https://gitlab.nic.cz/turris/reforis/foris-js.git"
-  },
+    },
-  "keywords": [
+    "keywords": [
-    "foris",
+        "foris",
-    "reforis"
+        "reforis"
-  ],
+    ],
-  "license": "GPL-3.0",
+    "license": "GPL-3.0",
-  "main": "./src/index.js",
+    "main": "./src/index.js",
-  "dependencies": {
+    "dependencies": {
-    "axios": "^0.19.1",
+        "axios": "^0.21.1",
-    "immutability-helper": "3.0.1",
+        "immutability-helper": "3.0.1",
-    "moment": "^2.24.0",
+        "moment": "^2.24.0",
-    "pdfmake": "^0.1.63",
+        "qrcode.react": "^0.9.3",
-    "qrcode.react": "^0.9.3",
+        "react-datetime": "^3.0.4",
-    "react-datetime": "^2.16.3",
+        "react-uid": "^2.2.0"
-    "react-router": "^5.1.2",
+    },
-    "react-uid": "^2.2.0"
+    "peerDependencies": {
-  },
+        "bootstrap": "4.4.1",
-  "peerDependencies": {
+        "prop-types": "15.7.2",
-    "bootstrap": "4.4.1",
+        "react": "16.9.0",
-    "prop-types": "15.7.2",
+        "react-dom": "16.9.0",
-    "react": "16.9.0",
+        "react-router-dom": "^5.1.2"
-    "react-dom": "16.9.0"
+    },
-  },
+    "devDependencies": {
-  "devDependencies": {
+        "@babel/cli": "^7.12.10",
-    "@babel/cli": "^7.7.7",
+        "@babel/core": "^7.9.0",
-    "@babel/core": "^7.7.7",
+        "@babel/plugin-transform-runtime": "^7.9.0",
-    "@babel/plugin-transform-runtime": "^7.7.6",
+        "@babel/preset-env": "^7.9.0",
-    "@babel/preset-env": "^7.7.7",
+        "@babel/preset-react": "^7.9.4",
-    "@babel/preset-react": "^7.7.4",
+        "@fortawesome/fontawesome-free": "^5.13.0",
-    "@fortawesome/fontawesome-free": "^5.12.0",
+        "@testing-library/react": "^8.0.9",
-    "@testing-library/react": "^8.0.9",
+        "babel-loader": "^8.1.0",
-    "babel-loader": "^8.0.6",
+        "babel-polyfill": "^6.26.0",
-    "babel-polyfill": "^6.26.0",
+        "bootstrap": "^4.5.0",
-    "eslint": "^6.8.0",
+        "css-loader": "^3.5.3",
-    "eslint-config-reforis": "^1.0.0",
+        "eslint": "^6.8.0",
-    "jest": "^24.9.0",
+        "eslint-config-prettier": "^6.11.0",
-    "jest-mock-axios": "^3.2.0",
+        "eslint-config-reforis": "^1.0.0",
-    "moment-timezone": "^0.5.27",
+        "eslint-plugin-prettier": "^3.1.4",
-    "prop-types": "15.7.2",
+        "file-loader": "^6.0.0",
-    "react": "16.9.0",
+        "jest": "^25.2.0",
-    "react-dom": "16.9.0",
+        "jest-mock-axios": "^3.2.0",
-    "react-styleguidist": "^10.4.2",
+        "moment-timezone": "^0.5.28",
-    "snapshot-diff": "^0.5.1"
+        "prettier": "2.0.5",
-  },
+        "prop-types": "15.7.2",
-  "scripts": {
+        "react": "16.9.0",
-    "lint": "eslint src",
+        "react-dom": "16.9.0",
-    "lint:fix": "eslint --fix src",
+        "react-router-dom": "^5.1.2",
-    "test": "jest",
+        "react-styleguidist": "^11.1.5",
-    "test:watch": "jest --watch",
+        "snapshot-diff": "^0.7.0",
-    "test:coverage": "jest --coverage --colors",
+        "style-loader": "^1.2.1",
-    "docs": "npx styleguidist build ",
+        "webpack": "^5.15.0"
-    "docs:watch": "styleguidist server"
+    },
-  }
+    "scripts": {
+        "lint": "eslint src",
+        "lint:fix": "eslint --fix src",
+        "test": "jest",
+        "test:watch": "jest --watch",
+        "test:coverage": "jest --coverage --colors",
+        "docs": "npx styleguidist build ",
+        "docs:watch": "styleguidist server"
+    }
 }

src/alertContext/AlertContext.js

@@ -22,9 +22,12 @@ function AlertContextProvider({ children }) {
     const { AlertContext } = window;
     const [alert, setAlert] = useState(null);
 
-    const setAlertWrapper = useCallback((message, type = ALERT_TYPES.DANGER) => {
+    const setAlertWrapper = useCallback(
-        setAlert({ message, type });
+        (message, type = ALERT_TYPES.DANGER) => {
-    }, [setAlert]);
+            setAlert({ message, type });
+        },
+        [setAlert]
+    );
 
     const dismissAlert = useCallback(() => setAlert(null), [setAlert]);
 
@@ -38,7 +41,7 @@ function AlertContextProvider({ children }) {
                 </Portal>
             )}
             <AlertContext.Provider value={[setAlertWrapper, dismissAlert]}>
-                { children }
+                {children}
             </AlertContext.Provider>
         </>
     );

src/alertContext/AlertContext.md

@@ -0,0 +1,5 @@
+It provides alert context to children. `AlertContext` allows using `useAlert` in
+components.
+
+Notice that `<div id="alert-container"/>` should be presented in HTML doc to get
+it work (In reForis it's already done with base Jinja2 templates).

src/alertContext/__tests__/AlertContext.test.js

@@ -6,9 +6,7 @@
  */
 
 import React from "react";
-import {
+import { render, getByText, queryByText, fireEvent } from "customTestRender";
-    render, getByText, queryByText, fireEvent,
-} from "customTestRender";
 
 import { useAlert, AlertContextProvider } from "../AlertContext";
 
@@ -31,7 +29,7 @@ describe("AlertContext", () => {
         const { container } = render(
             <AlertContextProvider>
                 <AlertTest />
-            </AlertContextProvider>,
+            </AlertContextProvider>
         );
         componentContainer = container;
     });

src/api/hooks.js

@@ -5,13 +5,16 @@
  * See /LICENSE for more information.
  */
 
-import {
+import { useCallback, useEffect, useReducer, useState } from "react";
-    useCallback, useEffect, useReducer, useState,
-} from "react";
 
-import { ForisURLs } from "../forisUrls";
+import { ForisURLs } from "../utils/forisUrls";
 import {
-    API_ACTIONS, API_METHODS, API_STATE, getErrorPayload, HEADERS, TIMEOUT,
+    API_ACTIONS,
+    API_METHODS,
+    API_STATE,
+    getErrorPayload,
+    HEADERS,
+    TIMEOUT,
 } from "./utils";
 
 const DATA_METHODS = ["POST", "PATCH", "PUT"];
@@ -23,69 +26,83 @@ function createAPIHook(method) {
             data: null,
         });
 
-        const sendRequest = useCallback(async ({ data, suffix } = {}) => {
+        const sendRequest = useCallback(
-            const headers = { ...HEADERS };
+            async ({ data, suffix } = {}) => {
-            if (contentType) {
+                const headers = { ...HEADERS };
-                headers["Content-Type"] = contentType;
+                if (contentType) {
-            }
+                    headers["Content-Type"] = contentType;
-
-            dispatch({ type: API_ACTIONS.INIT });
-            try {
-                // Prepare request
-                const request = API_METHODS[method];
-                const config = {
-                    timeout: TIMEOUT,
-                    headers,
-                };
-                const url = suffix ? `${urlRoot}/${suffix}` : urlRoot;
-
-                // Make request
-                let result;
-                if (DATA_METHODS.includes(method)) {
-                    result = await request(url, data, config);
-                } else {
-                    result = await request(url, config);
                 }
 
-                // Process request result
+                dispatch({ type: API_ACTIONS.INIT });
-                dispatch({
+                try {
-                    type: API_ACTIONS.SUCCESS,
+                    // Prepare request
-                    payload: result.data,
+                    const request = API_METHODS[method];
-                });
+                    const config = {
-            } catch (error) {
+                        timeout: TIMEOUT,
-                dispatch({
+                        headers,
-                    type: API_ACTIONS.FAILURE,
+                    };
-                    status: error.response && error.response.status,
+                    const url = suffix ? `${urlRoot}/${suffix}` : urlRoot;
-                    payload: getErrorPayload(error),
+
-                });
+                    // Make request
-            }
+                    let result;
-        }, [urlRoot, contentType]);
+                    if (DATA_METHODS.includes(method)) {
+                        result = await request(url, data, config);
+                    } else {
+                        result = await request(url, config);
+                    }
+
+                    // Process request result
+                    dispatch({
+                        type: API_ACTIONS.SUCCESS,
+                        payload: result.data,
+                    });
+                } catch (error) {
+                    const errorPayload = getErrorPayload(error);
+                    dispatch({
+                        type: API_ACTIONS.FAILURE,
+                        status: error.response && error.response.status,
+                        payload: errorPayload,
+                    });
+                }
+            },
+            [urlRoot, contentType]
+        );
         return [state, sendRequest];
     };
 }
 
 function APIReducer(state, action) {
     switch (action.type) {
-    case API_ACTIONS.INIT:
+        case API_ACTIONS.INIT:
-        return {
+            return {
-            ...state,
+                ...state,
-            state: API_STATE.SENDING,
+                state: API_STATE.SENDING,
-        };
+            };
-    case API_ACTIONS.SUCCESS:
+        case API_ACTIONS.SUCCESS:
-        return {
+            return {
-            state: API_STATE.SUCCESS,
+                state: API_STATE.SUCCESS,
-            data: action.payload,
+                data: action.payload,
-        };
+            };
-    case API_ACTIONS.FAILURE:
+        case API_ACTIONS.FAILURE:
-        if (action.status === 403) {
+            if (action.status === 403) {
-            window.location.assign(ForisURLs.login);
+                window.location.assign(ForisURLs.login);
-        }
+            }
-        return {
+
-            state: API_STATE.ERROR,
+            // Not an API error - should be rethrown.
-            data: action.payload,
+            if (
-        };
+                action.payload &&
-    default:
+                action.payload.stack &&
-        throw new Error();
+                action.payload.message
+            ) {
+                throw action.payload;
+            }
+
+            return {
+                state: API_STATE.ERROR,
+                data: action.payload,
+            };
+        default:
+            throw new Error();
     }
 }
 
@@ -95,11 +112,10 @@ const useAPIPatch = createAPIHook("PATCH");
 const useAPIPut = createAPIHook("PUT");
 const useAPIDelete = createAPIHook("DELETE");
 
-export {
+export { useAPIGet, useAPIPost, useAPIPatch, useAPIPut, useAPIDelete };
-    useAPIGet, useAPIPost, useAPIPatch, useAPIPut, useAPIDelete,
-};
 
-export function useAPIPolling(endpoint, delay = 1000, until) { // delay ms
+export function useAPIPolling(endpoint, delay = 1000, until) {
+    // delay ms
     const [state, setState] = useState({ state: API_STATE.INIT });
     const [getResponse, get] = useAPIGet(endpoint);
 

src/api/utils.js

@@ -13,7 +13,7 @@ export const HEADERS = {
     "X-CSRFToken": getCookie("_csrf_token"),
 };
 
-export const TIMEOUT = 5000;
+export const TIMEOUT = 30500;
 
 export const API_ACTIONS = {
     INIT: 1,
@@ -43,8 +43,10 @@ function getCookie(name) {
         for (let i = 0; i < cookies.length; i++) {
             const cookie = cookies[i].trim();
             // Does this cookie string begin with the name we want?
-            if (cookie.substring(0, name.length + 1) === (`${name}=`)) {
+            if (cookie.substring(0, name.length + 1) === `${name}=`) {
-                cookieValue = decodeURIComponent(cookie.substring(name.length + 1));
+                cookieValue = decodeURIComponent(
+                    cookie.substring(name.length + 1)
+                );
                 break;
             }
         }
@@ -65,9 +67,8 @@ export function getErrorPayload(error) {
     if (error.request) {
         return _("No response received.");
     }
-    /* eslint no-console: "off" */
+    // Return original error because it's not directly related to API request/response.
-    console.error(error);
+    return error;
-    return _("An unknown error occurred. Check the console for more info.");
 }
 
 export function getJSONErrorMessage(error) {

src/bootstrap/Alert.js

@@ -35,12 +35,20 @@ Alert.defaultProps = {
     type: ALERT_TYPES.DANGER,
 };
 
-export function Alert({
+export function Alert({ type, onDismiss, children }) {
-    type, onDismiss, children,
-}) {
     return (
-        <div className={`alert alert-dismissible alert-${type}`}>
+        <div
-            {onDismiss ? <button type="button" className="close" onClick={onDismiss}>&times;</button> : false}
+            className={`alert ${
+                onDismiss ? "alert-dismissible" : ""
+            } alert-${type}`}
+        >
+            {onDismiss ? (
+                <button type="button" className="close" onClick={onDismiss}>
+                    &times;
+                </button>
+            ) : (
+                false
+            )}
             {children}
         </div>
     );

src/bootstrap/Alert.md

@@ -1,20 +1,21 @@
 Bootstrap alert component.
-```jsx
-import {useState} from 'react';
 
-function AlertExample(){
+```jsx
+import { useState } from "react";
+
+function AlertExample() {
     const [alert, setAlert] = useState(true);
     if (alert)
-        return <Alert 
+        return (
-            type='warning' 
+            <Alert type="warning" onDismiss={() => setAlert(false)}>
-            message='Some warning out there!' 
+                Some warning out there!
-            onDismiss={()=>setAlert(false)}
+            </Alert>
-        />;
+        );
-    return <button 
+    return (
-        className='btn btn-secondary' 
+        <button className="btn btn-secondary" onClick={() => setAlert(true)}>
-        onClick={()=>setAlert(true)}
+            Show alert again
-    >Show alert again</button>;
+        </button>
-};
+    );
-<AlertExample/>
+}
-
+<AlertExample />;
 ```

src/bootstrap/Button.js

@@ -25,22 +25,29 @@ Button.propTypes = {
 };
 
 export function Button({
-    className, loading, forisFormSize, children, ...props
+    className,
+    loading,
+    forisFormSize,
+    children,
+    ...props
 }) {
     let buttonClass = className ? `btn ${className}` : "btn btn-primary ";
     if (forisFormSize) {
-        buttonClass = `${buttonClass} col-sm-12 col-lg-3`;
+        buttonClass = `${buttonClass} col-sm-12 col-md-3 col-lg-2`;
     }
 
-    const span = loading
+    const span = loading ? (
-        ? <span className="spinner-border spinner-border-sm" role="status" aria-hidden="true" /> : null;
+        <span
+            className="spinner-border spinner-border-sm"
+            role="status"
+            aria-hidden="true"
+        />
+    ) : null;
 
     return (
         <button type="button" className={buttonClass} {...props}>
             {span}
-            {" "}
             {span ? " " : null}
-            {" "}
             {children}
         </button>
     );

src/bootstrap/Button.md

@@ -11,5 +11,7 @@ Can be used without parameters:
 Using loading spinner:
 
 ```jsx
-<Button loading disabled>Loading...</Button>
+<Button loading disabled>
+    Loading...
+</Button>
 ```

src/bootstrap/CheckBox.js

@@ -22,9 +22,7 @@ CheckBox.defaultProps = {
     disabled: false,
 };
 
-export function CheckBox({
+export function CheckBox({ label, helpText, disabled, ...props }) {
-    label, helpText, disabled, ...props
-}) {
     const uid = useUID();
     return (
         <div className="form-group">
@@ -34,12 +32,15 @@ export function CheckBox({
                     type="checkbox"
                     id={uid}
                     disabled={disabled}
-
                     {...props}
                 />
                 <label className="custom-control-label" htmlFor={uid}>
                     {label}
-                    {helpText && <small className="form-text text-muted">{helpText}</small>}
+                    {helpText && (
+                        <small className="form-text text-muted">
+                            {helpText}
+                        </small>
+                    )}
                 </label>
             </div>
         </div>

src/bootstrap/Checkbox.md

@@ -1,16 +1,17 @@
-Checkbox with label Bootstrap component with predefined sizes and structure for using in foris forms.
+Checkbox with label Bootstrap component with predefined sizes and structure for
-
+using in foris forms.
-All additional `props` are passed to the `<input type="checkbox">` HTML component.
 
+All additional `props` are passed to the `<input type="checkbox">` HTML
+component.
 
 ```js
-import {useState} from 'react';
+import { useState } from "react";
 const [value, setValue] = useState(false);
 
 <CheckBox
     value={value}
-    label="Some label" 
+    label="Some label"
     helpText="Read the small text!"
-    onChange={event =>setValue(event.target.value)}
+    onChange={(event) => setValue(event.target.value)}
-/>
+/>;
 ```

src/bootstrap/DataTimeInput.js

@@ -7,7 +7,7 @@
 
 import React from "react";
 import PropTypes from "prop-types";
-import Datetime from "react-datetime/DateTime";
+import Datetime from "react-datetime";
 import moment from "moment/moment";
 import "react-datetime/css/react-datetime.css";
 import "./DataTimeInput.css";
@@ -38,14 +38,17 @@ const DEFAULT_DATE_FORMAT = "YYYY-MM-DD";
 const DEFAULT_TIME_FORMAT = "HH:mm:ss";
 
 export function DataTimeInput({
-    value, onChange, isValidDate, dateFormat, timeFormat, children, ...props
+    value,
+    onChange,
+    isValidDate,
+    dateFormat,
+    timeFormat,
+    children,
+    ...props
 }) {
     function renderInput(datetimeProps) {
         return (
-            <Input
+            <Input {...props} {...datetimeProps}>
-                {...props}
-                {...datetimeProps}
-            >
                 {children}
             </Input>
         );
@@ -54,8 +57,12 @@ export function DataTimeInput({
     return (
         <Datetime
             locale={ForisTranslations.locale}
-            dateFormat={dateFormat !== undefined ? dateFormat : DEFAULT_DATE_FORMAT}
+            dateFormat={
-            timeFormat={timeFormat !== undefined ? timeFormat : DEFAULT_TIME_FORMAT}
+                dateFormat !== undefined ? dateFormat : DEFAULT_DATE_FORMAT
+            }
+            timeFormat={
+                timeFormat !== undefined ? timeFormat : DEFAULT_TIME_FORMAT
+            }
             value={value}
             onChange={onChange}
             isValidDate={isValidDate}

src/bootstrap/DataTimeInput.md

@@ -1,25 +1,26 @@
-Adopted from `react-datetime/DateTime` datatime picker component.
+Adopted from `react-datetime/DateTime` datatime picker component. It uses
-It uses `momentjs` see example.
+`momentjs` see example.
 
-It requires `ForisTranslations.locale` to be defined in order to use right locale.
+It requires `ForisTranslations.locale` to be defined in order to use right
+locale.
 
 ```js
-ForisTranslations={locale:'en'};
+ForisTranslations = { locale: "en" };
 
-import {useState, useEffect} from 'react';
+import { useState, useEffect } from "react";
-import moment from 'moment/moment';
+import moment from "moment/moment";
 
 const [dataTime, setDataTime] = useState(moment());
 const [error, setError] = useState();
-useEffect(()=>{
+useEffect(() => {
-   dataTime.isValid() ? setError(null) : setError('Invalid value!');
+    dataTime.isValid() ? setError(null) : setError("Invalid value!");
-},[dataTime]);
+}, [dataTime]);
- 
+
 <DataTimeInput
-    label='Time to sleep'
+    label="Time to sleep"
     value={dataTime}
     error={error}
-    helpText='Example helptext...'
+    helpText="Example helptext..."
-    onChange={value => setDataTime(value)}
+    onChange={(value) => setDataTime(value)}
-/>
+/>;
 ```

src/bootstrap/DownloadButton.js

@@ -23,11 +23,7 @@ DownloadButton.defaultProps = {
 
 export function DownloadButton({ href, className, children }) {
     return (
-        <a
+        <a href={href} className={`btn ${className}`.trim()} download>
-            href={href}
-            className={`btn ${className}`.trim()}
-            download
-        >
             {children}
         </a>
     );

src/bootstrap/DownloadButton.md

@@ -1,6 +1,9 @@
 Hyperlink with apperance of a button.
 
-It has `download` attribute, which prevents closing WebSocket connection on Firefox. See [related issue](https://bugzilla.mozilla.org/show_bug.cgi?id=858538) for more details.
+It has `download` attribute, which prevents closing WebSocket connection on
+Firefox. See
+[related issue](https://bugzilla.mozilla.org/show_bug.cgi?id=858538) for more
+details.
 
 ```js
 <DownloadButton href="example.zip">Download</DownloadButton>

src/bootstrap/EmailInput.md

@@ -1,18 +1,19 @@
-Bootstrap component of email input with label with predefined sizes and structure for using in foris forms.
+Bootstrap component of email input with label with predefined sizes and
-It use built-in browser email address checking. It's only meaningful using inside `<form>`.
+structure for using in foris forms. It use built-in browser email address
+checking. It's only meaningful using inside `<form>`.
 
 All additional `props` are passed to the `<input type="email">` HTML component.
 
 ```js
-import {useState} from 'react';
+import { useState } from "react";
-const [email, setEmail] = useState('Wrong email');
+const [email, setEmail] = useState("Wrong email");
-<form onSubmit={e=>e.preventDefault()}>
+<form onSubmit={(e) => e.preventDefault()}>
     <EmailInput
         value={email}
-        label="Some label" 
+        label="Some label"
         helpText="Read the small text!"
-        onChange={event =>setEmail(event.target.value)}
+        onChange={(event) => setEmail(event.target.value)}
     />
     <button type="submit">Try to submit</button>
-</form>
+</form>;
 ```

src/bootstrap/FileInput.js

@@ -19,6 +19,8 @@ FileInput.propTypes = {
     helpText: PropTypes.string,
     /** Email value. */
     value: PropTypes.string,
+    /** Allow selecting multiple files. */
+    multiple: PropTypes.bool,
 };
 
 export function FileInput({ ...props }) {

src/bootstrap/FileInput.md

@@ -1,15 +1,48 @@
-Bootstrap component for file input. Includes label and has predefined sizes and structure for using in foris forms. 
+Bootstrap component for file input. Includes label and has predefined sizes and
+structure for using in foris forms.
 
 All additional `props` are passed to the `<input type="file">` HTML component.
 
 ```js
-import {useState} from 'react';
+import { useState } from "react";
+
 const [files, setFiles] = useState([]);
 
-<FileInput
+// Note that files is not an array but FileList.
-    files={files}
+const label = files.length === 1 ? files[0].name : "Choose file";
-    label="Some file"
+
-    helpText="Will be uploaded"
+<form className="col">
-    onChange={event =>setFiles(event.target.files)}
+    <FileInput
-/>
+        files={files}
+        label={label}
+        helpText="Will be uploaded"
+        onChange={(event) => setFiles(event.target.files)}
+    />
+</form>;
+```
+
+### FileInput with multiple files
+
+```js
+import { useState } from "react";
+
+const [files, setFiles] = useState([]);
+
+// Note that files is not an array but FileList.
+const label =
+    files.length > 0
+        ? Array.from(files)
+              .map((file) => file.name)
+              .join(", ")
+        : "Choose files";
+
+<form className="col">
+    <FileInput
+        files={files}
+        label={label}
+        helpText="Will be uploaded"
+        onChange={(event) => setFiles(event.target.files)}
+        multiple
+    />
+</form>;
 ```

src/bootstrap/Input.js

@@ -1,11 +1,11 @@
 /*
- * Copyright (C) 2019 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ * Copyright (C) 2019-2021 CZ.NIC z.s.p.o. (http://www.nic.cz/)
  *
  * This is free software, licensed under the GNU General Public License v3.
  * See /LICENSE for more information.
  */
 
-import React from "react";
+import React, { useState } from "react";
 import { useUID } from "react-uid";
 import PropTypes from "prop-types";
 
@@ -25,25 +25,43 @@ Input.propTypes = {
 
 /** Base bootstrap input component. */
 export function Input({
-    type, label, helpText, error, className, children, labelClassName, groupClassName, ...props
+    type,
+    label,
+    helpText,
+    error,
+    className,
+    children,
+    labelClassName,
+    groupClassName,
+    ...props
 }) {
     const uid = useUID();
-    const inputClassName = `form-control ${className || ""} ${(error ? "is-invalid" : "")}`.trim();
+    const [initialErrorFeedbackState, setErrorFeedbackState] = useState(false);
+    const errorFeedbackIsVisible = !!(initialErrorFeedbackState && error);
+    const inputClassName = `form-control ${className || ""} ${
+        errorFeedbackIsVisible ? "is-invalid" : ""
+    }`.trim();
     return (
         <div className="form-group">
-            <label className={labelClassName} htmlFor={uid}>{label}</label>
+            <label className={labelClassName} htmlFor={uid}>
+                {label}
+            </label>
             <div className={`input-group ${groupClassName || ""}`.trim()}>
                 <input
                     className={inputClassName}
                     type={type}
                     id={uid}
-
+                    onFocus={() => setErrorFeedbackState(true)}
                     {...props}
                 />
                 {children}
             </div>
-            {error ? <div className="invalid-feedback">{error}</div> : null}
+            {errorFeedbackIsVisible && (
-            {helpText ? <small className="form-text text-muted">{helpText}</small> : null}
+                <div className="invalid-feedback">{error}</div>
+            )}
+            {helpText && (
+                <small className="form-text text-muted">{helpText}</small>
+            )}
         </div>
     );
 }

src/bootstrap/Modal.css

@@ -0,0 +1,15 @@
+@keyframes modalFade {
+    from {
+        opacity: 0;
+    }
+    to {
+        opacity: 1;
+    }
+}
+
+.modal.show {
+    display: block;
+    animation-name: modalFade;
+    animation-duration: 0.3s;
+    background: rgba(0, 0, 0, 0.2);
+}

src/bootstrap/Modal.js

@@ -1,21 +1,24 @@
 /*
- * Copyright (C) 2019 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ * Copyright (C) 2020 CZ.NIC z.s.p.o. (http://www.nic.cz/)
  *
  * This is free software, licensed under the GNU General Public License v3.
  * See /LICENSE for more information.
  */
 
-import React, { useRef } from "react";
+import React, { useRef, useEffect } from "react";
 import PropTypes from "prop-types";
 
 import { Portal } from "../utils/Portal";
 import { useClickOutside } from "../utils/hooks";
+import "./Modal.css";
 
 Modal.propTypes = {
     /** Is modal shown value */
     shown: PropTypes.bool.isRequired,
     /** Callback to manage modal visibility */
     setShown: PropTypes.func.isRequired,
+    scrollable: PropTypes.bool,
+    size: PropTypes.string,
 
     /** Modal content use following: `ModalHeader`, `ModalBody`, `ModalFooter` */
     children: PropTypes.oneOfType([
@@ -24,18 +27,54 @@ Modal.propTypes = {
     ]).isRequired,
 };
 
-export function Modal({ shown, setShown, children }) {
+export function Modal({ shown, setShown, scrollable, size, children }) {
     const dialogRef = useRef();
+    let modalSize = "modal-";
 
     useClickOutside(dialogRef, () => setShown(false));
 
+    useEffect(() => {
+        const handleEsc = (event) => {
+            if (event.keyCode === 27) {
+                setShown(false);
+            }
+        };
+        window.addEventListener("keydown", handleEsc);
+
+        return () => {
+            window.removeEventListener("keydown", handleEsc);
+        };
+    }, [setShown]);
+
+    switch (size) {
+        case "sm":
+            modalSize += "sm";
+            break;
+        case "lg":
+            modalSize += "lg";
+            break;
+        case "xl":
+            modalSize += "xl";
+            break;
+        default:
+            modalSize = "";
+            break;
+    }
+
     return (
         <Portal containerId="modal-container">
-            <div className={`modal fade ${shown ? "show" : ""}`} role="dialog">
+            <div
-                <div ref={dialogRef} className="modal-dialog modal-dialog-centered" role="document">
+                className={`modal fade ${shown ? "show" : ""}`.trim()}
-                    <div className="modal-content">
+                role="dialog"
-                        {children}
+            >
-                    </div>
+                <div
+                    ref={dialogRef}
+                    className={`${modalSize.trim()} modal-dialog modal-dialog-centered ${
+                        scrollable ? "modal-dialog-scrollable" : ""
+                    }`.trim()}
+                    role="document"
+                >
+                    <div className="modal-content">{children}</div>
                 </div>
             </div>
         </Portal>
@@ -51,7 +90,11 @@ export function ModalHeader({ setShown, title }) {
     return (
         <div className="modal-header">
             <h5 className="modal-title">{title}</h5>
-            <button type="button" className="close" onClick={() => setShown(false)}>
+            <button
+                type="button"
+                className="close"
+                onClick={() => setShown(false)}
+            >
                 <span aria-hidden="true">&times;</span>
             </button>
         </div>
@@ -77,9 +120,5 @@ ModalFooter.propTypes = {
 };
 
 export function ModalFooter({ children }) {
-    return (
+    return <div className="modal-footer">{children}</div>;
-        <div className="modal-footer">
-            {children}
-        </div>
-    );
 }

src/bootstrap/Modal.md

@@ -1,33 +1,47 @@
 Bootstrap modal component.
 
-it's required to have an element `<div id={"modal-container"}/>` somewhere on the page since modals are rendered in portals.
+It's required to have an element `<div id={"modal-container"}/>` somewhere on
+the page since modals are rendered in portals.
+
+Modals also have three optional sizes, which can be defined through the `size`
+prop:
+
+-   small - `sm`
+-   large - `lg`
+-   extra-large - `xl`
+
+For more details please visit Bootstrap
+<a href="https://getbootstrap.com/docs/4.5/components/modal/#optional-sizes" target="_blank">
+documentation</a>.
 
 ```js
-    <div id="modal-container"/>
+<div id="modal-container" />
 ```
 
-I have no idea why example doesn't work here but you can investigate HTML code and Foris project.
-
 ```js
-import {ModalHeader, ModalBody, ModalFooter} from './Modal';
+import { ModalHeader, ModalBody, ModalFooter } from "./Modal";
 
-import {useState} from 'react';
+import { useState } from "react";
 const [shown, setShown] = useState(false);
 
 <>
-    <Modal setShown={setShown} shown={shown}>
+    <Modal setShown={setShown} shown={shown} size="sm">
-        <ModalHeader setShown={setShown} title='Warning!'/>
+        <ModalHeader setShown={setShown} title="Warning!" />
-        <ModalBody><p>Bla bla bla...</p></ModalBody>
+        <ModalBody>
+            <p>Bla bla bla...</p>
+        </ModalBody>
         <ModalFooter>
-            <button 
+            <button
-                className='btn btn-secondary' 
+                className="btn btn-secondary"
                 onClick={() => setShown(false)}
-            >Skip it</button>
+            >
+                Skip it
+            </button>
         </ModalFooter>
     </Modal>
-    
+
-    <button className='btn btn-secondary' onClick={()=>setShown(true)}>
+    <button className="btn btn-secondary" onClick={() => setShown(true)}>
         Show modal
     </button>
-</>
+</>;
 ```

src/bootstrap/NumberInput.css

@@ -4,7 +4,7 @@ input[type="number"] {
     appearance: textfield;
 }
 
-input[type=number]::-webkit-inner-spin-button,
+input[type="number"]::-webkit-inner-spin-button,
-input[type=number]::-webkit-outer-spin-button {
+input[type="number"]::-webkit-outer-spin-button {
     -webkit-appearance: none;
 }

src/bootstrap/NumberInput.js

@@ -20,10 +20,7 @@ NumberInput.propTypes = {
     /** Help text message. */
     helpText: PropTypes.string,
     /** Number value. */
-    value: PropTypes.oneOfType([
+    value: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
-        PropTypes.string,
-        PropTypes.number,
-    ]),
     /** Function called when value changes. */
     onChange: PropTypes.func.isRequired,
     /** Additional description dispaled to the right of input value. */
@@ -34,15 +31,21 @@ NumberInput.defaultProps = {
     value: 0,
 };
 
-export function NumberInput({
+export function NumberInput({ onChange, inlineText, value, ...props }) {
-    onChange, inlineText, value, ...props
-}) {
     function updateValue(initialValue, difference) {
         onChange({ target: { value: initialValue + difference } });
     }
 
-    const enableIncrease = useConditionalTimeout({ callback: updateValue }, value, 1);
+    const enableIncrease = useConditionalTimeout(
-    const enableDecrease = useConditionalTimeout({ callback: updateValue }, value, -1);
+        { callback: updateValue },
+        value,
+        1
+    );
+    const enableDecrease = useConditionalTimeout(
+        { callback: updateValue },
+        value,
+        -1
+    );
 
     return (
         <Input type="number" onChange={onChange} value={value} {...props}>

src/bootstrap/NumberInput.md

@@ -1,17 +1,18 @@
-Bootstrap component of number input with label with predefined sizes and structure for using in foris forms.
+Bootstrap component of number input with label with predefined sizes and
+structure for using in foris forms.
 
 All additional `props` are passed to the `<input type="number">` HTML component.
 
 ```js
-import {useState} from 'react';
+import { useState } from "react";
 const [value, setValue] = useState(42);
 
 <NumberInput
     value={value}
-    label="Some number" 
+    label="Some number"
     helpText="Read the small text!"
-    min='33'
+    min="33"
-    max='54'
+    max="54"
-    onChange={event =>setValue(event.target.value)}
+    onChange={(event) => setValue(event.target.value)}
-/>
+/>;
 ```

src/bootstrap/PasswordInput.js

@@ -31,22 +31,24 @@ export function PasswordInput({ withEye, ...props }) {
             autoComplete={isHidden ? "new-password" : null}
             {...props}
         >
-            {withEye
+            {withEye ? (
-                ? (
+                <div className="input-group-append">
-                    <div className="input-group-append">
+                    <button
-                        <button
+                        type="button"
-                            type="button"
+                        className="input-group-text"
-                            className="input-group-text"
+                        onClick={(e) => {
-                            onClick={(e) => {
+                            e.preventDefault();
-                                e.preventDefault();
+                            setHidden((shouldBeHidden) => !shouldBeHidden);
-                                setHidden((shouldBeHidden) => !shouldBeHidden);
+                        }}
-                            }}
+                    >
-                        >
+                        <i
-                            <i className={`fa ${isHidden ? "fa-eye" : "fa-eye-slash"}`} />
+                            className={`fa ${
-                        </button>
+                                isHidden ? "fa-eye" : "fa-eye-slash"
-                    </div>
+                            }`}
-                )
+                        />
-                : null}
+                    </button>
+                </div>
+            ) : null}
         </Input>
     );
 }

src/bootstrap/PasswordInput.md

@@ -1,17 +1,18 @@
-Password Bootstrap component input with label and predefined sizes and structure for using in foris forms.
+Password Bootstrap component input with label and predefined sizes and structure
-Can be used with "eye" button, see example.
+for using in foris forms. Can be used with "eye" button, see example.
 
-All additional `props` are passed to the `<input type="password">` HTML component.
+All additional `props` are passed to the `<input type="password">` HTML
+component.
 
 ```js
-import {useState} from 'react';
+import { useState } from "react";
-const [value, setValue] = useState('secret');
+const [value, setValue] = useState("secret");
 
 <PasswordInput
     withEye
     value={value}
-    label="Some password" 
+    label="Some password"
     helpText="Read the small text!"
-    onChange={event =>setValue(event.target.value)}
+    onChange={(event) => setValue(event.target.value)}
-/>
+/>;
 ```

src/bootstrap/RadioSet.js

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2019 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ * Copyright (C) 2020 CZ.NIC z.s.p.o. (http://www.nic.cz/)
  *
  * This is free software, licensed under the GNU General Public License v3.
  * See /LICENSE for more information.
@@ -15,20 +15,35 @@ RadioSet.propTypes = {
     /** RadioSet label . */
     label: PropTypes.string,
     /** Choices . */
-    choices: PropTypes.arrayOf(PropTypes.shape({
+    choices: PropTypes.arrayOf(
-        /** Choice lable . */
+        PropTypes.shape({
-        label: PropTypes.string.isRequired,
+            /** Choice lable . */
-        /** Choice value . */
+            label: PropTypes.oneOfType([
-        value: PropTypes.oneOfType([PropTypes.string, PropTypes.number]).isRequired,
+                PropTypes.string,
-    })).isRequired,
+                PropTypes.element,
+                PropTypes.node,
+                PropTypes.arrayOf(PropTypes.node),
+            ]).isRequired,
+            /** Choice value . */
+            value: PropTypes.oneOfType([PropTypes.string, PropTypes.number])
+                .isRequired,
+        })
+    ).isRequired,
     /** Initial value . */
     value: PropTypes.string,
     /** Help text message . */
     helpText: PropTypes.string,
+    inline: PropTypes.bool,
 };
 
 export function RadioSet({
-    name, label, choices, value, helpText, ...props
+    name,
+    label,
+    choices,
+    value,
+    helpText,
+    inline,
+    ...props
 }) {
     const uid = useUID();
     const radios = choices.map((choice, key) => {
@@ -42,7 +57,7 @@ export function RadioSet({
                 value={choice.value}
                 helpText={choice.helpText}
                 checked={choice.value === value}
-
+                inline={inline}
                 {...props}
             />
         );
@@ -50,34 +65,53 @@ export function RadioSet({
 
     return (
         <div className="form-group">
-            {label && <label htmlFor={uid} className="d-block">{label}</label>}
+            {label && (
+                <label htmlFor={uid} className="d-block">
+                    {label}
+                </label>
+            )}
             {radios}
-            {helpText && <small className="form-text text-muted">{helpText}</small>}
+            {helpText && (
+                <small className="form-text text-muted">{helpText}</small>
+            )}
         </div>
     );
 }
 
 Radio.propTypes = {
-    label: PropTypes.string.isRequired,
+    label: PropTypes.oneOfType([
+        PropTypes.string,
+        PropTypes.element,
+        PropTypes.node,
+        PropTypes.arrayOf(PropTypes.node),
+    ]).isRequired,
     id: PropTypes.string.isRequired,
+    inline: PropTypes.bool,
     helpText: PropTypes.string,
 };
 
-export function Radio({
+export function Radio({ label, id, helpText, inline, ...props }) {
-    label, id, helpText, ...props
-}) {
     return (
         <>
-            <div className={`custom-control custom-radio ${!helpText ? "custom-control-inline" : ""}`.trim()}>
+            <div
+                className={`custom-control custom-radio ${
+                    inline ? "custom-control-inline" : ""
+                }`.trim()}
+            >
                 <input
                     id={id}
                     className="custom-control-input"
                     type="radio"
-
                     {...props}
                 />
-                <label className="custom-control-label" htmlFor={id}>{label}</label>
+                <label className="custom-control-label" htmlFor={id}>
-                {helpText && <small className="form-text text-muted mt-0 mb-3">{helpText}</small>}
+                    {label}
+                </label>
+                {helpText && (
+                    <small className="form-text text-muted mt-0 mb-3">
+                        {helpText}
+                    </small>
+                )}
             </div>
         </>
     );

src/bootstrap/RadioSet.md

@@ -1,15 +1,16 @@
-Set of radio Bootstrap component input with label and predefined sizes and structure for using in foris forms.
+Set of radio Bootstrap component input with label and predefined sizes and
+structure for using in foris forms.
 
 All additional `props` are passed to the `<input type="number">` HTML component.
 
 Unless `helpText` is set for one of the options they are displayed inline.
 
 ```js
-import {useState} from 'react';
+import { useState } from "react";
-const CHOICES=[
+const CHOICES = [
-    {value:'one',label:'1'},
+    { value: "one", label: "1" },
-    {value:'two',label:'2'},
+    { value: "two", label: "2" },
-    {value:'three',label:'3'},
+    { value: "three", label: "3" },
 ];
 const [value, setValue] = useState(CHOICES[0].value);
 
@@ -17,10 +18,10 @@ const [value, setValue] = useState(CHOICES[0].value);
     {/*Yeah, it gets event, not value!*/}
     <RadioSet
         value={value}
-        name='some-radio'
+        name="some-radio"
         choices={CHOICES}
-        onChange={event =>setValue(event.target.value)}
+        onChange={(event) => setValue(event.target.value)}
     />
     <p>Selected value: {value}</p>
-</>
+</>;
 ```

src/bootstrap/Select.js

@@ -15,34 +15,30 @@ Select.propTypes = {
     /** Choices if form of {value : "Label",...}. */
     choices: PropTypes.object.isRequired,
     /** Current value. */
-    value: PropTypes.oneOfType([
+    value: PropTypes.oneOfType([PropTypes.string, PropTypes.number]).isRequired,
-        PropTypes.string,
-        PropTypes.number,
-    ]).isRequired,
     /** Help text message. */
     helpText: PropTypes.string,
 };
 
-export function Select({
+export function Select({ label, choices, helpText, ...props }) {
-    label, choices, helpText, ...props
-}) {
     const uid = useUID();
 
-    const options = Object.keys(choices).map(
+    const options = Object.keys(choices)
-        (key) => <option key={key} value={key}>{choices[key]}</option>,
+        .sort((a, b) => a - b || a.toString().localeCompare(b.toString()))
-    );
+        .map((key) => (
-
+            <option key={key} value={key}>
+                {choices[key]}
+            </option>
+        ));
     return (
         <div className="form-group">
             <label htmlFor={uid}>{label}</label>
-            <select
+            <select className="custom-select" id={uid} {...props}>
-                className="custom-select"
-                id={uid}
-                {...props}
-            >
                 {options}
             </select>
-            {helpText ? <small className="form-text text-muted">{helpText}</small> : null}
+            {helpText ? (
+                <small className="form-text text-muted">{helpText}</small>
+            ) : null}
         </div>
     );
 }

src/bootstrap/Select.md

@@ -1,13 +1,14 @@
-Select with options Bootstrap component input with label and predefined sizes and structure for using in foris forms.
+Select with options Bootstrap component input with label and predefined sizes
+and structure for using in foris forms.
 
 All additional `props` are passed to the `<select>` HTML component.
 
 ```js
-import {useState} from 'react';
+import { useState } from "react";
-const CHOICES={
+const CHOICES = {
-    apple:'Apple',
+    apple: "Apple",
-    banana:'Banana',
+    banana: "Banana",
-    peach:'Peach',
+    peach: "Peach",
 };
 const [value, setValue] = useState(Object.keys(CHOICES)[0]);
 
@@ -17,9 +18,9 @@ const [value, setValue] = useState(Object.keys(CHOICES)[0]);
         label="Fruit"
         value={value}
         choices={CHOICES}
-        onChange={event=>setValue(event.target.value)}
+        onChange={(event) => setValue(event.target.value)}
     />
     <p>Selected choice label: {CHOICES[value]}</p>
     <p>Selected choice value: {value}</p>
-</>
+</>;
 ```

src/bootstrap/Spinner.css

@@ -0,0 +1,33 @@
+.spinner-wrapper .spinner-border {
+    width: 4rem;
+    height: 4rem;
+    color: #00a2e2;
+}
+
+.spinner-fs-background {
+    background-color: rgba(2, 2, 2, 0.5);
+    color: rgb(230, 230, 230);
+    position: fixed;
+    width: 100%;
+    height: 100%;
+    top: 0;
+
+    display: flex;
+    align-items: center;
+    justify-content: center;
+    flex-direction: column;
+    /*
+     * Set to high value to me sure that it always overlaps all components
+     * https://getbootstrap.com/docs/4.3/layout/overview/#z-index
+     */
+    z-index: 1100;
+}
+
+.spinner-fs-wrapper .spinner-border {
+    width: 6rem;
+    height: 6rem;
+}
+
+.spinner-fs-wrapper .spinner-text {
+    margin: 1rem;
+}

src/bootstrap/Spinner.js

@@ -8,6 +8,8 @@
 import React from "react";
 import PropTypes from "prop-types";
 
+import "./Spinner.css";
+
 Spinner.propTypes = {
     /** Children components put into `div` with "spinner-text" class. */
     children: PropTypes.oneOfType([
@@ -23,12 +25,12 @@ Spinner.defaultProps = {
     fullScreen: false,
 };
 
-export function Spinner({
+export function Spinner({ fullScreen, children, className }) {
-    fullScreen, children, className,
-}) {
     if (!fullScreen) {
         return (
-            <div className={`spinner-wrapper ${className || "my-3 text-center"}`}>
+            <div
+                className={`spinner-wrapper ${className || "my-3 text-center"}`}
+            >
                 <SpinnerElement>{children}</SpinnerElement>
             </div>
         );
@@ -59,7 +61,9 @@ export function SpinnerElement({ small, className, children }) {
     return (
         <>
             <div
-                className={`spinner-border ${small ? "spinner-border-sm" : ""} ${className || ""}`.trim()}
+                className={`spinner-border ${
+                    small ? "spinner-border-sm" : ""
+                } ${className || ""}`.trim()}
                 role="status"
             >
                 <span className="sr-only" />

src/bootstrap/Switch.js

@@ -0,0 +1,49 @@
+/*
+ * Copyright (c) 2020 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ *
+ * This is free software, licensed under the GNU General Public License v3.
+ * See /LICENSE for more information.
+ */
+
+import React from "react";
+import PropTypes from "prop-types";
+import { useUID } from "react-uid";
+
+Switch.propTypes = {
+    label: PropTypes.oneOfType([
+        PropTypes.string,
+        PropTypes.element,
+        PropTypes.node,
+        PropTypes.arrayOf(PropTypes.node),
+    ]).isRequired,
+    helpText: PropTypes.string,
+    switchHeading: PropTypes.bool,
+};
+
+export function Switch({ label, helpText, switchHeading, ...props }) {
+    const uid = useUID();
+    return (
+        <div className={`form-group ${switchHeading ? "switch" : ""}`.trim()}>
+            <div
+                className={`custom-control custom-switch ${
+                    !helpText ? "custom-control-inline" : ""
+                } ${switchHeading ? "switch-heading" : ""}`.trim()}
+            >
+                <input
+                    type="checkbox"
+                    className="custom-control-input"
+                    id={uid}
+                    {...props}
+                />
+                <label className="custom-control-label" htmlFor={uid}>
+                    {label}
+                </label>
+                {helpText && (
+                    <small className="form-text text-muted mt-0 mb-3">
+                        {helpText}
+                    </small>
+                )}
+            </div>
+        </div>
+    );
+}

src/bootstrap/TextInput.md

@@ -1,15 +1,16 @@
-Text Bootstrap component input with label and predefined sizes and structure for using in foris forms.
+Text Bootstrap component input with label and predefined sizes and structure for
+using in foris forms.
 
 All additional `props` are passed to the `<input type="text">` HTML component.
 
 ```js
-import {useState} from 'react';
+import { useState } from "react";
-const [value, setValue] = useState('Bla bla');
+const [value, setValue] = useState("Bla bla");
 
 <TextInput
     value={value}
-    label="Some text" 
+    label="Some text"
     helpText="Read the small text!"
-    onChange={event =>setValue(event.target.value)}
+    onChange={(event) => setValue(event.target.value)}
-/>
+/>;
 ```

src/bootstrap/__tests__/Button.test.js

@@ -14,19 +14,18 @@ import { Button } from "../Button";
 describe("<Button />", () => {
     it("Render button correctly", () => {
         const { container } = render(<Button>Test Button</Button>);
-        expect(container.firstChild)
+        expect(container.firstChild).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 
     it("Render button with custom classes", () => {
-        const { container } = render(<Button className="one two three">Test Button</Button>);
+        const { container } = render(
-        expect(container.firstChild)
+            <Button className="one two three">Test Button</Button>
-            .toMatchSnapshot();
+        );
+        expect(container.firstChild).toMatchSnapshot();
     });
 
     it("Render button with spinner", () => {
         const { container } = render(<Button loading>Test Button</Button>);
-        expect(container.firstChild)
+        expect(container.firstChild).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 });

src/bootstrap/__tests__/CheckBox.test.js

@@ -18,22 +18,16 @@ describe("<Checkbox/>", () => {
                 label="Test label"
                 checked
                 helpText="Some help text"
-                onChange={() => {
+                onChange={() => {}}
-                }}
+            />
-            />,
         );
-        expect(container.firstChild)
+        expect(container.firstChild).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 
     it("Render uncheked checkbox", () => {
         const { container } = render(
-            <CheckBox
+            <CheckBox label="Test label" helpText="Some help text" />
-                label="Test label"
-                helpText="Some help text"
-            />,
         );
-        expect(container.firstChild)
+        expect(container.firstChild).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 });

src/bootstrap/__tests__/DownloadButton.test.js

@@ -13,7 +13,11 @@ import { DownloadButton } from "../DownloadButton";
 
 describe("<DownloadButton />", () => {
     it("should have download attribute", () => {
-        const { container } = render(<DownloadButton href="http://example.com">Test Button</DownloadButton>);
+        const { container } = render(
+            <DownloadButton href="http://example.com">
+                Test Button
+            </DownloadButton>
+        );
         expect(container.firstChild.getAttribute("download")).not.toBeNull();
     });
 });

src/bootstrap/__tests__/NumberInput.test.js

@@ -7,9 +7,7 @@
 
 import React from "react";
 
-import {
+import { render, fireEvent, getByLabelText, wait } from "customTestRender";
-    render, fireEvent, getByLabelText, wait,
-} from "customTestRender";
 
 import { NumberInput } from "../NumberInput";
 
@@ -24,7 +22,7 @@ describe("<NumberInput/>", () => {
                 helpText="Some help text"
                 value={1}
                 onChange={onChangeMock}
-            />,
+            />
         );
         componentContainer = container;
     });
@@ -36,12 +34,16 @@ describe("<NumberInput/>", () => {
     it("Increase number with button", async () => {
         const increaseButton = getByLabelText(componentContainer, "Increase");
         fireEvent.mouseDown(increaseButton);
-        await wait(() => expect(onChangeMock).toHaveBeenCalledWith({ target: { value: 2 } }));
+        await wait(() =>
+            expect(onChangeMock).toHaveBeenCalledWith({ target: { value: 2 } })
+        );
     });
 
     it("Decrease number with button", async () => {
         const decreaseButton = getByLabelText(componentContainer, "Decrease");
         fireEvent.mouseDown(decreaseButton);
-        await wait(() => expect(onChangeMock).toHaveBeenCalledWith({ target: { value: 0 } }));
+        await wait(() =>
+            expect(onChangeMock).toHaveBeenCalledWith({ target: { value: 0 } })
+        );
     });
 });

src/bootstrap/__tests__/PasswordInput.test.js

@@ -18,11 +18,9 @@ describe("<PasswordInput/>", () => {
                 label="Test label"
                 helpText="Some help text"
                 value="Some password"
-                onChange={() => {
+                onChange={() => {}}
-                }}
+            />
-            />,
         );
-        expect(container.firstChild)
+        expect(container.firstChild).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 });

src/bootstrap/__tests__/RadioSet.test.js

@@ -35,11 +35,9 @@ describe("<RadioSet/>", () => {
                 value="value"
                 choices={TEST_CHOICES}
                 helpText="Some help text"
-                onChange={() => {
+                onChange={() => {}}
-                }}
+            />
-            />,
         );
-        expect(container.firstChild)
+        expect(container.firstChild).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 });

src/bootstrap/__tests__/Select.test.js

@@ -8,7 +8,10 @@
 import React from "react";
 
 import {
-    fireEvent, getByDisplayValue, getByText, render,
+    fireEvent,
+    getByDisplayValue,
+    getByText,
+    render,
 } from "customTestRender";
 
 import { Select } from "../Select";
@@ -29,29 +32,24 @@ describe("<Select/>", () => {
                 value="1"
                 choices={TEST_CHOICES}
                 helpText="Help text"
-
                 onChange={onChangeHandler}
-            />,
+            />
         );
         selectContainer = container;
     });
 
     it("Test with snapshot.", () => {
-        expect(selectContainer)
+        expect(selectContainer).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 
     it("Test onChange handling.", () => {
         const select = getByDisplayValue(selectContainer, "one");
-        expect(select.value)
+        expect(select.value).toBe("1");
-            .toBe("1");
         fireEvent.change(select, { target: { value: "2" } });
 
         const option = getByText(selectContainer, "two");
-        expect(onChangeHandler)
+        expect(onChangeHandler).toBeCalled();
-            .toBeCalled();
 
-        expect(option.value)
+        expect(option.value).toBe("2");
-            .toBe("2");
     });
 });

src/bootstrap/__tests__/Switch.test.js

@@ -0,0 +1,33 @@
+/*
+ * Copyright (C) 2020 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ *
+ * This is free software, licensed under the GNU General Public License v3.
+ * See /LICENSE for more information.
+ */
+
+import React from "react";
+
+import { render } from "customTestRender";
+
+import { Switch } from "../Switch";
+
+describe("<Switch/>", () => {
+    it("Render switch", () => {
+        const { container } = render(
+            <Switch
+                label="Test label"
+                checked
+                helpText="Some help text"
+                onChange={() => {}}
+            />
+        );
+        expect(container.firstChild).toMatchSnapshot();
+    });
+
+    it("Render uncheked switch", () => {
+        const { container } = render(
+            <Switch label="Test label" helpText="Some help text" />
+        );
+        expect(container.firstChild).toMatchSnapshot();
+    });
+});

src/bootstrap/__tests__/TextInput.test.js

@@ -18,11 +18,9 @@ describe("<TextInput/>", () => {
                 label="Test label"
                 helpText="Some help text"
                 value="Some text"
-                onChange={() => {
+                onChange={() => {}}
-                }}
+            />
-            />,
         );
-        expect(container.firstChild)
+        expect(container.firstChild).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 });

src/bootstrap/__tests__/__snapshots__/Button.test.js.snap

@@ -5,8 +5,6 @@ exports[`<Button /> Render button correctly 1`] = `
   class="btn btn-primary "
   type="button"
 >
-   
-   
   Test Button
 </button>
 `;
@@ -16,8 +14,6 @@ exports[`<Button /> Render button with custom classes 1`] = `
   class="btn one two three"
   type="button"
 >
-   
-   
   Test Button
 </button>
 `;
@@ -33,8 +29,6 @@ exports[`<Button /> Render button with spinner 1`] = `
     role="status"
   />
    
-   
-   
   Test Button
 </button>
 `;

src/bootstrap/__tests__/__snapshots__/RadioSet.test.js.snap

@@ -11,7 +11,7 @@ exports[`<RadioSet/> Render radio set 1`] = `
     Radios set label
   </label>
   <div
-    class="custom-control custom-radio custom-control-inline"
+    class="custom-control custom-radio"
   >
     <input
       checked=""
@@ -29,7 +29,7 @@ exports[`<RadioSet/> Render radio set 1`] = `
     </label>
   </div>
   <div
-    class="custom-control custom-radio custom-control-inline"
+    class="custom-control custom-radio"
   >
     <input
       class="custom-control-input"
@@ -46,7 +46,7 @@ exports[`<RadioSet/> Render radio set 1`] = `
     </label>
   </div>
   <div
-    class="custom-control custom-radio custom-control-inline"
+    class="custom-control custom-radio"
   >
     <input
       class="custom-control-input"

src/bootstrap/__tests__/__snapshots__/Switch.test.js.snap

@@ -0,0 +1,56 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`<Switch/> Render switch 1`] = `
+<div
+  class="form-group"
+>
+  <div
+    class="custom-control custom-switch"
+  >
+    <input
+      checked=""
+      class="custom-control-input"
+      id="1"
+      type="checkbox"
+    />
+    <label
+      class="custom-control-label"
+      for="1"
+    >
+      Test label
+    </label>
+    <small
+      class="form-text text-muted mt-0 mb-3"
+    >
+      Some help text
+    </small>
+  </div>
+</div>
+`;
+
+exports[`<Switch/> Render uncheked switch 1`] = `
+<div
+  class="form-group"
+>
+  <div
+    class="custom-control custom-switch"
+  >
+    <input
+      class="custom-control-input"
+      id="1"
+      type="checkbox"
+    />
+    <label
+      class="custom-control-label"
+      for="1"
+    >
+      Test label
+    </label>
+    <small
+      class="form-text text-muted mt-0 mb-3"
+    >
+      Some help text
+    </small>
+  </div>
+</div>
+`;

src/bootstrap/constants.js

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2019 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ * Copyright (C) 2019-2021 CZ.NIC z.s.p.o. (http://www.nic.cz/)
  *
  * This is free software, licensed under the GNU General Public License v3.
  * See /LICENSE for more information.
@@ -7,4 +7,5 @@
 
 /** Bootstrap column size for form fields */
 // eslint-disable-next-line import/prefer-default-export
-export const formFieldsSize = "col-sm-12 offset-lg-1 col-lg-10 p-0 mb-3";
+export const formFieldsSize = "card p-4 col-sm-12 col-lg-12 p-0 mb-4";
+export const buttonFormFieldsSize = "col-sm-12 col-lg-12 p-0 mb-3";

src/common/RebootButton.js

@@ -10,18 +10,12 @@ import PropTypes from "prop-types";
 
 import { useAPIPost } from "../api/hooks";
 import { API_STATE } from "../api/utils";
-import { ForisURLs } from "../forisUrls";
+import { ForisURLs } from "../utils/forisUrls";
 
 import { Button } from "../bootstrap/Button";
-import {
+import { Modal, ModalHeader, ModalBody, ModalFooter } from "../bootstrap/Modal";
-    Modal, ModalHeader, ModalBody, ModalFooter,
-} from "../bootstrap/Modal";
 import { useAlert } from "../alertContext/AlertContext";
 
-RebootButton.propTypes = {
-    forisFormSize: PropTypes.bool,
-};
-
 export function RebootButton(props) {
     const [triggered, setTriggered] = useState(false);
     const [modalShown, setModalShown] = useState(false);
@@ -42,13 +36,16 @@ export function RebootButton(props) {
 
     return (
         <>
-            <RebootModal shown={modalShown} setShown={setModalShown} onReboot={rebootHandler} />
+            <RebootModal
+                shown={modalShown}
+                setShown={setModalShown}
+                onReboot={rebootHandler}
+            />
             <Button
                 className="btn-danger"
                 loading={triggered}
                 disabled={triggered}
                 onClick={() => setModalShown(true)}
-
                 {...props}
             >
                 {_("Reboot")}
@@ -66,11 +63,15 @@ RebootModal.propTypes = {
 function RebootModal({ shown, setShown, onReboot }) {
     return (
         <Modal shown={shown} setShown={setShown}>
-            <ModalHeader setShown={setShown} title={_("Reboot confirmation")} />
+            <ModalHeader setShown={setShown} title={_("Warning!")} />
-            <ModalBody><p>{_("Are you sure you want to restart the router?")}</p></ModalBody>
+            <ModalBody>
+                <p>{_("Are you sure you want to restart the router?")}</p>
+            </ModalBody>
             <ModalFooter>
                 <Button onClick={() => setShown(false)}>{_("Cancel")}</Button>
-                <Button className="btn-danger" onClick={onReboot}>{_("Confirm reboot")}</Button>
+                <Button className="btn-danger" onClick={onReboot}>
+                    {_("Confirm reboot")}
+                </Button>
             </ModalFooter>
         </Modal>
     );

src/common/WiFiSettings/ResetWiFiSettings.js

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2019 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ * Copyright (C) 2019-2021 CZ.NIC z.s.p.o. (http://www.nic.cz/)
  *
  * This is free software, licensed under the GNU General Public License v3.
  * See /LICENSE for more information.
@@ -25,12 +25,10 @@ export default function ResetWiFiSettings({ ws, endpoint }) {
 
     useEffect(() => {
         const module = "wifi";
-        ws.subscribe(module)
+        ws.subscribe(module).bind(module, "reset", () => {
-            .bind(module, "reset", () => {
+            // eslint-disable-next-line no-restricted-globals
-                setIsLoading(true);
+            setTimeout(() => location.reload(), 1000);
-                // eslint-disable-next-line no-restricted-globals
+        });
-                setTimeout(() => location.reload(), 1000);
-            });
     }, [ws]);
 
     const [postResetResponse, postReset] = useAPIPost(endpoint);
@@ -39,36 +37,39 @@ export default function ResetWiFiSettings({ ws, endpoint }) {
         if (postResetResponse.state === API_STATE.ERROR) {
             setAlert(_("An error occurred during resetting Wi-Fi settings."));
         } else if (postResetResponse.state === API_STATE.SUCCESS) {
-            setAlert(_("Wi-Fi settings are set to defaults."), ALERT_TYPES.SUCCESS);
+            setAlert(
+                _("Wi-Fi settings are set to defaults."),
+                ALERT_TYPES.SUCCESS
+            );
         }
     }, [postResetResponse, setAlert]);
 
     function onReset() {
         dismissAlert();
+        setIsLoading(true);
         postReset();
     }
 
     return (
-        <>
+        <div className={formFieldsSize}>
-            <h4>{_("Reset Wi-Fi Settings")}</h4>
+            <h2>{_("Reset Wi-Fi Settings")}</h2>
             <p>
                 {_(`
 If a number of wireless cards doesn't match, you may try to reset the Wi-Fi settings. Note that this will remove the
 current Wi-Fi configuration and restore the default values.
         `)}
             </p>
-            <div className={`${formFieldsSize} text-right`}>
+            <div className="text-right">
                 <Button
                     className="btn-warning"
                     forisFormSize
                     loading={isLoading}
                     disabled={isLoading}
-
                     onClick={onReset}
                 >
                     {_("Reset Wi-Fi Settings")}
                 </Button>
             </div>
-        </>
+        </div>
     );
 }

src/common/WiFiSettings/WiFiForm.js

@@ -7,7 +7,7 @@
 
 import React from "react";
 import PropTypes from "prop-types";
-
+import { Switch } from "../../bootstrap/Switch";
 import { CheckBox } from "../../bootstrap/CheckBox";
 import { PasswordInput } from "../../bootstrap/PasswordInput";
 import { RadioSet } from "../../bootstrap/RadioSet";
@@ -18,13 +18,9 @@ import WifiGuestForm from "./WiFiGuestForm";
 import { HELP_TEXTS, HTMODES, HWMODES } from "./constants";
 
 WiFiForm.propTypes = {
-    formData: PropTypes.shape(
+    formData: PropTypes.shape({ devices: PropTypes.arrayOf(PropTypes.object) })
-        { devices: PropTypes.arrayOf(PropTypes.object) },
+        .isRequired,
-    ).isRequired,
+    formErrors: PropTypes.oneOfType([PropTypes.object, PropTypes.array]),
-    formErrors: PropTypes.oneOfType([
-        PropTypes.object,
-        PropTypes.array,
-    ]),
     setFormValue: PropTypes.func.isRequired,
     hasGuestNetwork: PropTypes.bool,
 };
@@ -36,17 +32,22 @@ WiFiForm.defaultProps = {
 };
 
 export default function WiFiForm({
-    formData, formErrors, setFormValue, hasGuestNetwork, ...props
+    formData,
+    formErrors,
+    setFormValue,
+    hasGuestNetwork,
+    disabled,
 }) {
-    return formData.devices.map((device) => (
+    return formData.devices.map((device, index) => (
         <DeviceForm
             key={device.id}
             formData={device}
-            formErrors={(formErrors || [])[device.id]}
+            deviceIndex={index}
+            formErrors={(formErrors || [])[index]}
             setFormValue={setFormValue}
             hasGuestNetwork={hasGuestNetwork}
-
+            disabled={disabled}
-            {...props}
+            divider={index + 1 !== formData.devices.length}
         />
     ));
 }
@@ -66,6 +67,8 @@ DeviceForm.propTypes = {
     formErrors: PropTypes.object.isRequired,
     setFormValue: PropTypes.func.isRequired,
     hasGuestNetwork: PropTypes.bool,
+    deviceIndex: PropTypes.number,
+    divider: PropTypes.bool,
 };
 
 DeviceForm.defaultProps = {
@@ -74,138 +77,137 @@ DeviceForm.defaultProps = {
 };
 
 function DeviceForm({
-    formData, formErrors, setFormValue, hasGuestNetwork, ...props
+    formData,
+    formErrors,
+    setFormValue,
+    hasGuestNetwork,
+    deviceIndex,
+    divider,
+    ...props
 }) {
     const deviceID = formData.id;
     return (
         <>
-            <h3>{_(`Wi-Fi ${deviceID + 1}`)}</h3>
+            <Switch
-            <CheckBox
+                label={<h2>{_(`Wi-Fi ${deviceID + 1}`)}</h2>}
-                label={_("Enable")}
                 checked={formData.enabled}
-
+                onChange={setFormValue((value) => ({
-                onChange={setFormValue(
+                    devices: {
-                    (value) => ({ devices: { [deviceID]: { enabled: { $set: value } } } }),
+                        [deviceIndex]: { enabled: { $set: value } },
-                )}
+                    },
-
+                }))}
+                switchHeading
                 {...props}
             />
-            {formData.enabled
+            {formData.enabled ? (
-                ? (
+                <>
-                    <>
+                    <TextInput
-                        <TextInput
+                        label="SSID"
-                            label="SSID"
+                        value={formData.SSID}
-                            value={formData.SSID}
+                        error={formErrors.SSID || null}
-                            error={formErrors.SSID || null}
+                        helpText={HELP_TEXTS.ssid}
-                            required
+                        required
-                            onChange={setFormValue(
+                        onChange={setFormValue((value) => ({
-                                (value) => ({ devices: { [deviceID]: { SSID: { $set: value } } } }),
+                            devices: {
-                            )}
+                                [deviceIndex]: {
-
+                                    SSID: { $set: value },
-                            {...props}
+                                },
-                        >
+                            },
-                            <div className="input-group-append">
+                        }))}
-                                <WiFiQRCode
+                        {...props}
-                                    SSID={formData.SSID}
+                    >
-                                    password={formData.password}
+                        <div className="input-group-append">
-                                />
+                            <WiFiQRCode
-                            </div>
+                                SSID={formData.SSID}
-                        </TextInput>
+                                password={formData.password}
-
-                        <PasswordInput
-                            withEye
-                            label="Password"
-                            value={formData.password}
-                            error={formErrors.password}
-                            helpText={HELP_TEXTS.password}
-                            required
-
-                            onChange={setFormValue(
-                                (value) => (
-                                    { devices: { [deviceID]: { password: { $set: value } } } }
-                                ),
-                            )}
-
-                            {...props}
-                        />
-
-                        <CheckBox
-                            label="Hide SSID"
-                            helpText={HELP_TEXTS.hidden}
-                            checked={formData.hidden}
-
-                            onChange={setFormValue(
-                                (value) => (
-                                    { devices: { [deviceID]: { hidden: { $set: value } } } }
-                                ),
-                            )}
-
-                            {...props}
-                        />
-
-                        <RadioSet
-                            name={`hwmode-${deviceID}`}
-                            label="GHz"
-                            choices={getHwmodeChoices(formData)}
-                            value={formData.hwmode}
-                            helpText={HELP_TEXTS.hwmode}
-
-                            onChange={setFormValue(
-                                (value) => ({
-                                    devices: {
-                                        [deviceID]: {
-                                            hwmode: { $set: value },
-                                            channel: { $set: "0" },
-                                        },
-                                    },
-                                }),
-                            )}
-
-                            {...props}
-                        />
-
-                        <Select
-                            label="802.11n/ac mode"
-                            choices={getHtmodeChoices(formData)}
-                            value={formData.htmode}
-                            helpText={HELP_TEXTS.htmode}
-
-                            onChange={setFormValue(
-                                (value) => (
-                                    { devices: { [deviceID]: { htmode: { $set: value } } } }
-                                ),
-                            )}
-
-                            {...props}
-                        />
-
-                        <Select
-                            label="Channel"
-                            choices={getChannelChoices(formData)}
-                            value={formData.channel}
-
-                            onChange={setFormValue(
-                                (value) => (
-                                    { devices: { [deviceID]: { channel: { $set: value } } } }
-                                ),
-                            )}
-
-                            {...props}
-                        />
-
-                        {hasGuestNetwork && (
-                            <WifiGuestForm
-                                formData={{ id: deviceID, ...formData.guest_wifi }}
-                                formErrors={formErrors.guest_wifi || {}}
-
-                                setFormValue={setFormValue}
-
-                                {...props}
                             />
-                        )}
+                        </div>
-                    </>
+                    </TextInput>
-                )
+
-                : null}
+                    <PasswordInput
+                        withEye
+                        label={_("Password")}
+                        value={formData.password}
+                        error={formErrors.password}
+                        helpText={HELP_TEXTS.password}
+                        required
+                        onChange={setFormValue((value) => ({
+                            devices: {
+                                [deviceIndex]: { password: { $set: value } },
+                            },
+                        }))}
+                        {...props}
+                    />
+
+                    <CheckBox
+                        label={_("Hide SSID")}
+                        helpText={HELP_TEXTS.hidden}
+                        checked={formData.hidden}
+                        onChange={setFormValue((value) => ({
+                            devices: {
+                                [deviceIndex]: { hidden: { $set: value } },
+                            },
+                        }))}
+                        {...props}
+                    />
+
+                    <RadioSet
+                        name={`hwmode-${deviceID}`}
+                        label="GHz"
+                        choices={getHwmodeChoices(formData)}
+                        value={formData.hwmode}
+                        helpText={HELP_TEXTS.hwmode}
+                        inline
+                        onChange={setFormValue((value) => ({
+                            devices: {
+                                [deviceIndex]: {
+                                    hwmode: { $set: value },
+                                    channel: { $set: "0" },
+                                },
+                            },
+                        }))}
+                        {...props}
+                    />
+
+                    <Select
+                        label={_("802.11n/ac mode")}
+                        choices={getHtmodeChoices(formData)}
+                        value={formData.htmode}
+                        helpText={HELP_TEXTS.htmode}
+                        onChange={setFormValue((value) => ({
+                            devices: {
+                                [deviceIndex]: { htmode: { $set: value } },
+                            },
+                        }))}
+                        {...props}
+                    />
+
+                    <Select
+                        label={_("Channel")}
+                        choices={getChannelChoices(formData)}
+                        value={formData.channel}
+                        onChange={setFormValue((value) => ({
+                            devices: {
+                                [deviceIndex]: { channel: { $set: value } },
+                            },
+                        }))}
+                        {...props}
+                    />
+
+                    {hasGuestNetwork && (
+                        <WifiGuestForm
+                            formData={{
+                                id: deviceIndex,
+                                ...formData.guest_wifi,
+                            }}
+                            formErrors={formErrors.guest_wifi || {}}
+                            setFormValue={setFormValue}
+                            {...props}
+                        />
+                    )}
+                </>
+            ) : null}
+            {divider ? <hr /> : null}
         </>
     );
 }
@@ -221,7 +223,9 @@ function getChannelChoices(device) {
         availableBand.available_channels.forEach((availableChannel) => {
             channelChoices[availableChannel.number.toString()] = `
                         ${availableChannel.number}
-                        (${availableChannel.frequency} MHz ${availableChannel.radar ? " ,DFS" : ""})
+                        (${availableChannel.frequency} MHz ${
+                availableChannel.radar ? " ,DFS" : ""
+            })
                     `;
         });
     });

src/common/WiFiSettings/WiFiGuestForm.js

@@ -8,8 +8,8 @@
 import React from "react";
 import PropTypes from "prop-types";
 
-import { CheckBox } from "../../bootstrap/CheckBox";
 import { TextInput } from "../../bootstrap/TextInput";
+import { Switch } from "../../bootstrap/Switch";
 import { PasswordInput } from "../../bootstrap/PasswordInput";
 import WiFiQRCode from "./WiFiQRCode";
 import { HELP_TEXTS } from "./constants";
@@ -26,75 +26,73 @@ WifiGuestForm.propTypes = {
         password: PropTypes.string,
     }),
     setFormValue: PropTypes.func.isRequired,
+    deviceIndex: PropTypes.string,
 };
 
 export default function WifiGuestForm({
-    formData, formErrors, setFormValue, ...props
+    formData,
+    formErrors,
+    setFormValue,
+    deviceIndex,
+    ...props
 }) {
     return (
         <>
-            <CheckBox
+            <Switch
-                label={_("Enable Guest Wifi")}
+                label={_("Enable Guest Wi-Fi")}
                 checked={formData.enabled}
                 helpText={HELP_TEXTS.guest_wifi_enabled}
-
+                onChange={setFormValue((value) => ({
-                onChange={setFormValue(
+                    devices: {
-                    (value) => (
+                        [formData.id]: {
-                        { devices: { [formData.id]: { guest_wifi: { enabled: { $set: value } } } } }
+                            guest_wifi: { enabled: { $set: value } },
-                    ),
+                        },
-                )}
+                    },
-
+                }))}
                 {...props}
             />
-            {formData.enabled
+            {formData.enabled ? (
-                ? (
+                <>
-                    <>
+                    <TextInput
-                        <TextInput
+                        label="SSID"
-                            label="SSID"
+                        value={formData.SSID}
-                            value={formData.SSID}
+                        error={formErrors.SSID}
-                            error={formErrors.SSID}
+                        helpText={HELP_TEXTS.ssid}
+                        onChange={setFormValue((value) => ({
+                            devices: {
+                                [formData.id]: {
+                                    guest_wifi: { SSID: { $set: value } },
+                                },
+                            },
+                        }))}
+                        {...props}
+                    >
+                        <div className="input-group-append">
+                            <WiFiQRCode
+                                SSID={formData.SSID}
+                                password={formData.password}
+                            />
+                        </div>
+                    </TextInput>
 
-                            onChange={setFormValue(
+                    <PasswordInput
-                                (value) => ({
+                        withEye
-                                    devices: {
+                        label={_("Password")}
-                                        [formData.id]: { guest_wifi: { SSID: { $set: value } } },
+                        value={formData.password}
-                                    },
+                        helpText={HELP_TEXTS.password}
-                                }),
+                        error={formErrors.password}
-                            )}
+                        required
-
+                        onChange={setFormValue((value) => ({
-                            {...props}
+                            devices: {
-                        >
+                                [formData.id]: {
-                            <div className="input-group-append">
+                                    guest_wifi: { password: { $set: value } },
-                                <WiFiQRCode
+                                },
-                                    SSID={formData.SSID}
+                            },
-                                    password={formData.password}
+                        }))}
-                                />
+                        {...props}
-                            </div>
+                    />
-                        </TextInput>
+                </>
-
+            ) : null}
-                        <PasswordInput
-                            withEye
-                            label={_("Password")}
-                            value={formData.password}
-                            helpText={HELP_TEXTS.password}
-                            error={formErrors.password}
-                            required
-
-                            onChange={setFormValue(
-                                (value) => ({
-                                    devices: {
-                                        [formData.id]: {
-                                            guest_wifi: { password: { $set: value } },
-                                        },
-                                    },
-                                }),
-                            )}
-
-                            {...props}
-                        />
-                    </>
-                )
-                : null}
         </>
     );
 }

src/common/WiFiSettings/WiFiQRCode.js

@@ -9,10 +9,13 @@ import React, { useState } from "react";
 import QRCode from "qrcode.react";
 import PropTypes from "prop-types";
 
-import { ForisURLs } from "../../forisUrls";
+import { ForisURLs } from "../../utils/forisUrls";
 import { Button } from "../../bootstrap/Button";
 import {
-    Modal, ModalBody, ModalFooter, ModalHeader,
+    Modal,
+    ModalBody,
+    ModalFooter,
+    ModalHeader,
 } from "../../bootstrap/Modal";
 import { createAndDownloadPdf, toQRCodeContent } from "./qrCodeHelpers";
 
@@ -36,11 +39,21 @@ export default function WiFiQRCode({ SSID, password }) {
                     setModal(true);
                 }}
             >
-                <img width="20" src={QR_ICON_PATH} alt="QR" style={{ opacity: 0.67 }} />
+                <img
+                    width="20"
+                    src={QR_ICON_PATH}
+                    alt="QR"
+                    style={{ opacity: 0.67 }}
+                />
             </button>
-            {modal
+            {modal ? (
-                ? <QRCodeModal setShown={setModal} shown={modal} SSID={SSID} password={password} />
+                <QRCodeModal
-                : null}
+                    setShown={setModal}
+                    shown={modal}
+                    SSID={SSID}
+                    password={password}
+                />
+            ) : null}
         </>
     );
 }
@@ -52,9 +65,7 @@ QRCodeModal.propTypes = {
     setShown: PropTypes.func.isRequired,
 };
 
-function QRCodeModal({
+function QRCodeModal({ shown, setShown, SSID, password }) {
-    shown, setShown, SSID, password,
-}) {
     return (
         <Modal setShown={setShown} shown={shown}>
             <ModalHeader setShown={setShown} title={_("Wi-Fi QR Code")} />

src/common/WiFiSettings/WiFiSettings.js

@@ -19,9 +19,7 @@ WiFiSettings.propTypes = {
     hasGuestNetwork: PropTypes.bool,
 };
 
-export function WiFiSettings({
+export function WiFiSettings({ ws, endpoint, resetEndpoint, hasGuestNetwork }) {
-    ws, endpoint, resetEndpoint, hasGuestNetwork,
-}) {
     return (
         <>
             <ForisForm
@@ -59,35 +57,51 @@ function prepDataToSubmit(formData) {
             return;
         }
 
-        if (!device.guest_wifi.enabled) formData.devices[idx].guest_wifi = { enabled: false };
+        if (!device.guest_wifi.enabled)
+            formData.devices[idx].guest_wifi = { enabled: false };
     });
     return formData;
 }
 
-function validator(formData) {
+export function byteCount(string) {
-    const formErrors = formData.devices.map(
+    const buffer = Buffer.from(string, "utf-8");
-        (device) => {
+    const count = buffer.byteLength;
-            if (!device.enabled) return {};
+    return count;
-
+}
-            const errors = {};
+
-            if (device.SSID.length > 32) errors.SSID = _("SSID can't be longer than 32 symbols");
+export function validator(formData) {
-            if (device.SSID.length === 0) errors.SSID = _("SSID can't be empty");
+    const formErrors = formData.devices.map((device) => {
-
+        if (!device.enabled) return {};
-            if (device.password.length < 8) errors.password = _("Password must contain at least 8 symbols");
+
-
+        const errors = {};
-            if (!device.guest_wifi.enabled) return errors;
+        if (device.SSID.length > 32)
-
+            errors.SSID = _("SSID can't be longer than 32 symbols");
-            const guest_wifi_errors = {};
+        if (device.SSID.length === 0) errors.SSID = _("SSID can't be empty");
-            if (device.guest_wifi.SSID.length > 32) guest_wifi_errors.SSID = _("SSID can't be longer than 32 symbols");
+        if (byteCount(device.SSID) > 32)
-            if (device.guest_wifi.SSID.length === 0) guest_wifi_errors.SSID = _("SSID can't be empty");
+            errors.SSID = _("SSID can't be longer than 32 bytes");
-
+
-            if (device.guest_wifi.password.length < 8) guest_wifi_errors.password = _("Password must contain at least 8 symbols");
+        if (device.password.length < 8)
-
+            errors.password = _("Password must contain at least 8 symbols");
-            if (guest_wifi_errors.SSID || guest_wifi_errors.password) {
+
-                errors.guest_wifi = guest_wifi_errors;
+        if (!device.guest_wifi.enabled) return errors;
-            }
+
-            return errors;
+        const guest_wifi_errors = {};
-        },
+        if (device.guest_wifi.SSID.length > 32)
-    );
+            guest_wifi_errors.SSID = _("SSID can't be longer than 32 symbols");
-    return JSON.stringify(formErrors) === "[{},{}]" ? null : formErrors;
+        if (device.guest_wifi.SSID.length === 0)
+            guest_wifi_errors.SSID = _("SSID can't be empty");
+        if (byteCount(device.guest_wifi.SSID) > 32)
+            guest_wifi_errors.SSID = _("SSID can't be longer than 32 bytes");
+
+        if (device.guest_wifi.password.length < 8)
+            guest_wifi_errors.password = _(
+                "Password must contain at least 8 symbols"
+            );
+
+        if (guest_wifi_errors.SSID || guest_wifi_errors.password) {
+            errors.guest_wifi = guest_wifi_errors;
+        }
+        return errors;
+    });
+    return JSON.stringify(formErrors).match(/\[[{},?]+\]/) ? null : formErrors;
 }

src/common/WiFiSettings/__tests__/ResetWiFiSettings.test.js

@@ -22,19 +22,34 @@ describe("<ResetWiFiSettings/>", () => {
     let getAllByText;
 
     beforeEach(() => {
-        ({ getAllByText } = render(<ResetWiFiSettings ws={webSockets} endpoint={endpoint} />));
+        ({ getAllByText } = render(
+            <ResetWiFiSettings ws={webSockets} endpoint={endpoint} />
+        ));
     });
 
     it("should display alert on open ports - success", async () => {
         fireEvent.click(getAllByText("Reset Wi-Fi Settings")[1]);
-        expect(mockAxios.post).toBeCalledWith(endpoint, undefined, expect.anything());
+        expect(mockAxios.post).toBeCalledWith(
+            endpoint,
+            undefined,
+            expect.anything()
+        );
         mockAxios.mockResponse({ data: { foo: "bar" } });
-        await wait(() => expect(mockSetAlert).toBeCalledWith("Wi-Fi settings are set to defaults.", ALERT_TYPES.SUCCESS));
+        await wait(() =>
+            expect(mockSetAlert).toBeCalledWith(
+                "Wi-Fi settings are set to defaults.",
+                ALERT_TYPES.SUCCESS
+            )
+        );
     });
 
     it("should display alert on open ports - failure", async () => {
         fireEvent.click(getAllByText("Reset Wi-Fi Settings")[1]);
         mockJSONError();
-        await wait(() => expect(mockSetAlert).toBeCalledWith("An error occurred during resetting Wi-Fi settings."));
+        await wait(() =>
+            expect(mockSetAlert).toBeCalledWith(
+                "An error occurred during resetting Wi-Fi settings."
+            )
+        );
     });
 });

src/common/WiFiSettings/__tests__/WiFiSettings.test.js

@@ -13,8 +13,13 @@ import { fireEvent, render, wait } from "customTestRender";
 import { WebSockets } from "webSockets/WebSockets";
 import { mockJSONError } from "testUtils/network";
 
-import { wifiSettingsFixture } from "./__fixtures__/wifiSettings";
+import {
-import { WiFiSettings } from "../WiFiSettings";
+    wifiSettingsFixture,
+    oneDevice,
+    twoDevices,
+    threeDevices,
+} from "./__fixtures__/wifiSettings";
+import { WiFiSettings, validator, byteCount } from "../WiFiSettings";
 
 describe("<WiFiSettings/>", () => {
     let firstRender;
@@ -26,7 +31,13 @@ describe("<WiFiSettings/>", () => {
 
     beforeEach(async () => {
         const webSockets = new WebSockets();
-        const renderRes = render(<WiFiSettings ws={webSockets} endpoint={endpoint} resetEndpoint="foo" />);
+        const renderRes = render(
+            <WiFiSettings
+                ws={webSockets}
+                endpoint={endpoint}
+                resetEndpoint="foo"
+            />
+        );
         asFragment = renderRes.asFragment;
         getAllByText = renderRes.getAllByText;
         getAllByLabelText = renderRes.getAllByLabelText;
@@ -38,10 +49,18 @@ describe("<WiFiSettings/>", () => {
 
     it("should handle error", async () => {
         const webSockets = new WebSockets();
-        const { getByText } = render(<WiFiSettings ws={webSockets} ws={webSockets} endpoint={endpoint} resetEndpoint="foo" />);
+        const { getByText } = render(
-        mockJSONError();
+            <WiFiSettings
+                ws={webSockets}
+                ws={webSockets}
+                endpoint={endpoint}
+                resetEndpoint="foo"
+            />
+        );
+        const errorMessage = "An API error occurred.";
+        mockJSONError(errorMessage);
         await wait(() => {
-            expect(getByText("An error occurred while fetching data.")).toBeTruthy();
+            expect(getByText(errorMessage)).toBeTruthy();
         });
     });
 
@@ -50,21 +69,21 @@ describe("<WiFiSettings/>", () => {
     });
 
     it("Snapshot one module enabled.", () => {
-        fireEvent.click(getAllByText("Enable")[0]);
+        fireEvent.click(getByText("Wi-Fi 1"));
         expect(diffSnapshot(firstRender, asFragment())).toMatchSnapshot();
     });
 
     it("Snapshot 2.4 GHz", () => {
-        fireEvent.click(getAllByText("Enable")[0]);
+        fireEvent.click(getByText("Wi-Fi 1"));
         const enabledRender = asFragment();
         fireEvent.click(getAllByText("2.4")[0]);
         expect(diffSnapshot(enabledRender, asFragment())).toMatchSnapshot();
     });
 
     it("Snapshot guest network.", () => {
-        fireEvent.click(getAllByText("Enable")[0]);
+        fireEvent.click(getByText("Wi-Fi 1"));
         const enabledRender = asFragment();
-        fireEvent.click(getAllByText("Enable Guest Wifi")[0]);
+        fireEvent.click(getAllByText("Enable Guest Wi-Fi")[0]);
         expect(diffSnapshot(enabledRender, asFragment())).toMatchSnapshot();
     });
 
@@ -77,11 +96,15 @@ describe("<WiFiSettings/>", () => {
                 { enabled: false, id: 1 },
             ],
         };
-        expect(mockAxios.post).toHaveBeenCalledWith(endpoint, data, expect.anything());
+        expect(mockAxios.post).toHaveBeenCalledWith(
+            endpoint,
+            data,
+            expect.anything()
+        );
     });
 
     it("Post form: one module enabled.", () => {
-        fireEvent.click(getAllByText("Enable")[0]);
+        fireEvent.click(getByText("Wi-Fi 1"));
 
         fireEvent.click(getByText("Save"));
         expect(mockAxios.post).toBeCalled();
@@ -101,11 +124,15 @@ describe("<WiFiSettings/>", () => {
                 { enabled: false, id: 1 },
             ],
         };
-        expect(mockAxios.post).toHaveBeenCalledWith(endpoint, data, expect.anything());
+        expect(mockAxios.post).toHaveBeenCalledWith(
+            endpoint,
+            data,
+            expect.anything()
+        );
     });
 
     it("Post form: 2.4 GHz", () => {
-        fireEvent.click(getAllByText("Enable")[0]);
+        fireEvent.click(getByText("Wi-Fi 1"));
         fireEvent.click(getAllByText("2.4")[0]);
 
         fireEvent.click(getByText("Save"));
@@ -126,13 +153,19 @@ describe("<WiFiSettings/>", () => {
                 { enabled: false, id: 1 },
             ],
         };
-        expect(mockAxios.post).toHaveBeenCalledWith(endpoint, data, expect.anything());
+        expect(mockAxios.post).toHaveBeenCalledWith(
+            endpoint,
+            data,
+            expect.anything()
+        );
     });
 
     it("Post form: guest network.", () => {
-        fireEvent.click(getAllByText("Enable")[0]);
+        fireEvent.click(getByText("Wi-Fi 1"));
-        fireEvent.click(getAllByText("Enable Guest Wifi")[0]);
+        fireEvent.click(getAllByText("Enable Guest Wi-Fi")[0]);
-        fireEvent.change(getAllByLabelText("Password")[1], { target: { value: "test_password" } });
+        fireEvent.change(getAllByLabelText("Password")[1], {
+            target: { value: "test_password" },
+        });
 
         fireEvent.click(getByText("Save"));
         expect(mockAxios.post).toBeCalled();
@@ -156,6 +189,32 @@ describe("<WiFiSettings/>", () => {
                 { enabled: false, id: 1 },
             ],
         };
-        expect(mockAxios.post).toHaveBeenCalledWith(endpoint, data, expect.anything());
+        expect(mockAxios.post).toHaveBeenCalledWith(
+            endpoint,
+            data,
+            expect.anything()
+        );
+    });
+
+    it("Validator function using regex for one device", () => {
+        expect(validator(oneDevice)).toEqual(null);
+    });
+
+    it("Validator function using regex for two devices", () => {
+        const twoDevicesFormErrors = [{ SSID: "SSID can't be empty" }, {}];
+        expect(validator(twoDevices)).toEqual(twoDevicesFormErrors);
+    });
+
+    it("Validator function using regex for three devices", () => {
+        const threeDevicesFormErrors = [
+            {},
+            {},
+            { password: "Password must contain at least 8 symbols" },
+        ];
+        expect(validator(threeDevices)).toEqual(threeDevicesFormErrors);
+    });
+
+    it("ByteCount function", () => {
+        expect(byteCount("abc")).toEqual(3);
     });
 });

src/common/WiFiSettings/__tests__/__fixtures__/wifiSettings.js

@@ -292,11 +292,7 @@ export function wifiSettingsFixture() {
                                 radar: false,
                             },
                         ],
-                        available_htmodes: [
+                        available_htmodes: ["NOHT", "HT20", "HT40"],
-                            "NOHT",
-                            "HT20",
-                            "HT40",
-                        ],
                         hwmode: "11g",
                     },
                 ],
@@ -316,3 +312,85 @@ export function wifiSettingsFixture() {
         ],
     };
 }
+const oneDevice = {
+    devices: [
+        {
+            SSID: "Turris1",
+            channel: 60,
+            enabled: true,
+            guest_wifi: { enabled: false },
+            hidden: false,
+            htmode: "HT40",
+            hwmode: "11a",
+            id: 0,
+            password: "TestPass",
+        },
+    ],
+};
+
+const twoDevices = {
+    devices: [
+        {
+            SSID: "",
+            channel: 60,
+            enabled: true,
+            guest_wifi: { enabled: false },
+            hidden: false,
+            htmode: "HT40",
+            hwmode: "11a",
+            id: 0,
+            password: "TestPass",
+        },
+        {
+            SSID: "Turris2",
+            channel: 60,
+            enabled: true,
+            guest_wifi: { enabled: false },
+            hidden: false,
+            htmode: "HT40",
+            hwmode: "11a",
+            id: 0,
+            password: "TestPass",
+        },
+    ],
+};
+
+const threeDevices = {
+    devices: [
+        {
+            SSID: "Turris1",
+            channel: 60,
+            enabled: true,
+            guest_wifi: { enabled: false },
+            hidden: false,
+            htmode: "HT40",
+            hwmode: "11a",
+            id: 0,
+            password: "TestPass",
+        },
+        {
+            SSID: "Turris2",
+            channel: 60,
+            enabled: false,
+            guest_wifi: { enabled: false },
+            hidden: false,
+            htmode: "HT40",
+            hwmode: "11a",
+            id: 0,
+            password: "TestPass",
+        },
+        {
+            SSID: "Turris3",
+            channel: 60,
+            enabled: true,
+            guest_wifi: { enabled: false },
+            hidden: false,
+            htmode: "HT40",
+            hwmode: "11a",
+            id: 0,
+            password: "",
+        },
+    ],
+};
+
+export { oneDevice, twoDevices, threeDevices };

src/common/WiFiSettings/__tests__/__snapshots__/WiFiSettings.test.js.snap

@@ -5,7 +5,7 @@ exports[`<WiFiSettings/> Snapshot 2.4 GHz 1`] = `
 - First value
 + Second value
 
-@@ -246,207 +246,95 @@
+@@ -250,207 +250,95 @@
               value=\\"0\\"
             >
               auto
@@ -251,17 +251,14 @@ exports[`<WiFiSettings/> Snapshot 2.4 GHz 1`] = `
 exports[`<WiFiSettings/> Snapshot both modules disabled. 1`] = `
 <DocumentFragment>
   <div
-    class="col-sm-12 offset-lg-1 col-lg-10 p-0 mb-3"
+    class="card p-4 col-sm-12 col-lg-12 p-0 mb-4"
   >
     <form>
-      <h3>
-        Wi-Fi 1
-      </h3>
       <div
-        class="form-group"
+        class="form-group switch"
       >
         <div
-          class="custom-control custom-checkbox "
+          class="custom-control custom-switch custom-control-inline switch-heading"
         >
           <input
             class="custom-control-input"
@@ -272,18 +269,18 @@ exports[`<WiFiSettings/> Snapshot both modules disabled. 1`] = `
             class="custom-control-label"
             for="1"
           >
-            Enable
+            <h2>
+              Wi-Fi 1
+            </h2>
           </label>
         </div>
       </div>
-      <h3>
+      <hr />
-        Wi-Fi 2
-      </h3>
       <div
-        class="form-group"
+        class="form-group switch"
       >
         <div
-          class="custom-control custom-checkbox "
+          class="custom-control custom-switch custom-control-inline switch-heading"
         >
           <input
             class="custom-control-input"
@@ -294,7 +291,9 @@ exports[`<WiFiSettings/> Snapshot both modules disabled. 1`] = `
             class="custom-control-label"
             for="2"
           >
-            Enable
+            <h2>
+              Wi-Fi 2
+            </h2>
           </label>
         </div>
       </div>
@@ -302,32 +301,36 @@ exports[`<WiFiSettings/> Snapshot both modules disabled. 1`] = `
         class="text-right"
       >
         <button
-          class="btn btn-primary  col-sm-12 col-lg-3"
+          class="btn btn-primary  col-sm-12 col-md-3 col-lg-2"
           type="submit"
         >
-            Save
+          Save
         </button>
       </div>
     </form>
   </div>
-  <h4>
+  <div
-    Reset Wi-Fi Settings
+    class="card p-4 col-sm-12 col-lg-12 p-0 mb-4"
-  </h4>
+  >
-  <p>
+    <h2>
-    
+      Reset Wi-Fi Settings
+    </h2>
+    <p>
+      
 If a number of wireless cards doesn't match, you may try to reset the Wi-Fi settings. Note that this will remove the
 current Wi-Fi configuration and restore the default values.
         
-  </p>
+    </p>
-  <div
+    <div
-    class="col-sm-12 offset-lg-1 col-lg-10 p-0 mb-3 text-right"
+      class="text-right"
-  >
-    <button
-      class="btn btn-warning col-sm-12 col-lg-3"
-      type="button"
     >
+      <button
+        class="btn btn-warning col-sm-12 col-md-3 col-lg-2"
+        type="button"
+      >
         Reset Wi-Fi Settings
-    </button>
+      </button>
+    </div>
   </div>
 </DocumentFragment>
 `;
@@ -337,10 +340,10 @@ exports[`<WiFiSettings/> Snapshot guest network. 1`] = `
 - First value
 + Second value
 
-@@ -475,10 +475,89 @@
+@@ -479,10 +479,89 @@
+          Parameters of the guest network can be set in the Guest network tab.
           
-              </small>
+            </small>
-            </label>
           </div>
         </div>
 +       <div
@@ -376,6 +379,11 @@ exports[`<WiFiSettings/> Snapshot guest network. 1`] = `
 +             </button>
 +           </div>
 +         </div>
++         <small
++           class=\\"form-text text-muted\\"
++         >
++           SSID which contains non-standard characters could cause problems on some devices.
++         </small>
 +       </div>
 +       <div
 +         class=\\"form-group\\"
@@ -390,7 +398,7 @@ exports[`<WiFiSettings/> Snapshot guest network. 1`] = `
 +         >
 +           <input
 +             autocomplete=\\"new-password\\"
-+             class=\\"form-control  is-invalid\\"
++             class=\\"form-control\\"
 +             id=\\"21\\"
 +             required=\\"\\"
 +             type=\\"password\\"
@@ -409,11 +417,6 @@ exports[`<WiFiSettings/> Snapshot guest network. 1`] = `
 +             </button>
 +           </div>
 +         </div>
-+         <div
-+           class=\\"invalid-feedback\\"
-+         >
-+           Password must contain at least 8 symbols
-+         </div>
 +         <small
 +           class=\\"form-text text-muted\\"
 +         >
@@ -422,21 +425,21 @@ exports[`<WiFiSettings/> Snapshot guest network. 1`] = `
 +     
 +         </small>
 +       </div>
-        <h3>
+        <hr />
-          Wi-Fi 2
-        </h3>
         <div
-          class=\\"form-group\\"
+          class=\\"form-group switch\\"
-@@ -502,10 +581,11 @@
+        >
+          <div
+@@ -506,10 +585,11 @@
         <div
           class=\\"text-right\\"
         >
           <button
-            class=\\"btn btn-primary  col-sm-12 col-lg-3\\"
+            class=\\"btn btn-primary  col-sm-12 col-md-3 col-lg-2\\"
 +           disabled=\\"\\"
             type=\\"submit\\"
           >
-              Save
+            Save
           </button>
         </div>"
 `;
@@ -446,9 +449,9 @@ exports[`<WiFiSettings/> Snapshot one module enabled. 1`] = `
 - First value
 + Second value
 
-@@ -23,10 +23,462 @@
+@@ -22,10 +22,467 @@
-            >
+                Wi-Fi 1
-              Enable
+              </h2>
             </label>
           </div>
         </div>
@@ -486,6 +489,11 @@ exports[`<WiFiSettings/> Snapshot one module enabled. 1`] = `
 +             </button>
 +           </div>
 +         </div>
++         <small
++           class=\\"form-text text-muted\\"
++         >
++           SSID which contains non-standard characters could cause problems on some devices.
++         </small>
 +       </div>
 +       <div
 +         class=\\"form-group\\"
@@ -617,11 +625,6 @@ exports[`<WiFiSettings/> Snapshot one module enabled. 1`] = `
 +           id=\\"8\\"
 +         >
 +           <option
-+             value=\\"NOHT\\"
-+           >
-+             Disabled
-+           </option>
-+           <option
 +             value=\\"HT20\\"
 +           >
 +             802.11n - 20 MHz wide channel
@@ -632,6 +635,11 @@ exports[`<WiFiSettings/> Snapshot one module enabled. 1`] = `
 +             802.11n - 40 MHz wide channel
 +           </option>
 +           <option
++             value=\\"NOHT\\"
++           >
++             Disabled
++           </option>
++           <option
 +             value=\\"VHT20\\"
 +           >
 +             802.11ac - 20 MHz wide channel
@@ -880,7 +888,7 @@ exports[`<WiFiSettings/> Snapshot one module enabled. 1`] = `
 +         class=\\"form-group\\"
 +       >
 +         <div
-+           class=\\"custom-control custom-checkbox \\"
++           class=\\"custom-control custom-switch\\"
 +         >
 +           <input
 +             class=\\"custom-control-input\\"
@@ -891,22 +899,22 @@ exports[`<WiFiSettings/> Snapshot one module enabled. 1`] = `
 +             class=\\"custom-control-label\\"
 +             for=\\"10\\"
 +           >
-+             Enable Guest Wifi
++             Enable Guest Wi-Fi
-+             <small
++           </label>
-+               class=\\"form-text text-muted\\"
++           <small
-+             >
++             class=\\"form-text text-muted mt-0 mb-3\\"
-+               
++           >
++             
 +         Enables Wi-Fi for guests, which is separated from LAN network. Devices connected to this network are allowed to
 +         access the internet, but aren't allowed to access other devices and the configuration interface of the router.
 +         Parameters of the guest network can be set in the Guest network tab.
 +         
-+             </small>
++           </small>
-+           </label>
 +         </div>
 +       </div>
-        <h3>
+        <hr />
-          Wi-Fi 2
-        </h3>
         <div
-          class=\\"form-group\\""
+          class=\\"form-group switch\\"
+        >
+          <div"
 `;

src/common/WiFiSettings/constants.js

@@ -12,16 +12,22 @@ export const HTMODES = {
     VHT20: _("802.11ac - 20 MHz wide channel"),
     VHT40: _("802.11ac - 40 MHz wide channel"),
     VHT80: _("802.11ac - 80 MHz wide channel"),
+    VHT160: _("802.11ac - 160 MHz wide channel"),
 };
 export const HWMODES = {
     "11g": "2.4",
     "11a": "5",
 };
 export const HELP_TEXTS = {
+    ssid: _(
+        `SSID which contains non-standard characters could cause problems on some devices.`
+    ),
     password: _(`
         WPA2 pre-shared key, that is required to connect to the network.
     `),
-    hidden: _("If set, network is not visible when scanning for available networks."),
+    hidden: _(
+        "If set, network is not visible when scanning for available networks."
+    ),
     hwmode: _(`
         The 2.4 GHz band is more widely supported by clients, but tends to have more interference. The 5 GHz band is a
         newer standard and may not be supported by all your devices. It usually has less interference, but the signal

src/common/WiFiSettings/qrCodeHelpers.js

@@ -5,22 +5,36 @@
  * See /LICENSE for more information.
  */
 
-import pdfMake from "pdfmake";
-
 export function createAndDownloadPdf(SSID, password) {
     const docDefinition = {
         content: [
             {
-                text: "Wi-Fi", style: "header", fontSize: 55, alignment: "center",
+                text: "Wi-Fi",
+                style: "header",
+                fontSize: 55,
+                alignment: "center",
             },
             {
-                qr: toQRCodeContent(SSID, password), fit: "350", margin: [0, 80], alignment: "center",
+                qr: toQRCodeContent(SSID, password),
+                fit: "350",
+                margin: [0, 80],
+                alignment: "center",
+            },
+            {
+                text: `SSID: ${SSID}`,
+                fontSize: 25,
+                alignment: "center",
+            },
+            {
+                text: `Password: ${password}`,
+                fontSize: 25,
+                alignment: "center",
             },
-            { text: `SSID: ${SSID}`, fontSize: 25, alignment: "center" },
-            { text: `Password: ${password}`, fontSize: 25, alignment: "center" },
         ],
     };
-    pdfMake.createPdf(docDefinition).download("wifi.pdf");
+
+    // pdfmake is exposed by reForis main application. Thus we can use it from globals.
+    window.pdfMake.createPdf(docDefinition).download("wifi.pdf");
 }
 
 export function toQRCodeContent(SSID, password) {

src/common/__tests__/RebootButton.test.js

@@ -8,7 +8,11 @@
 import React from "react";
 
 import {
-    fireEvent, getByText, queryByText, render, wait,
+    fireEvent,
+    getByText,
+    queryByText,
+    render,
+    wait,
 } from "customTestRender";
 import mockAxios from "jest-mock-axios";
 import { mockJSONError } from "testUtils/network";
@@ -19,38 +23,41 @@ import { RebootButton } from "../RebootButton";
 describe("<RebootButton/>", () => {
     let componentContainer;
     beforeEach(() => {
-        const { container } = render(<>
+        const { container } = render(
-            <div id="modal-container" />
+            <>
-            <RebootButton />
+                <div id="modal-container" />
-        </>);
+                <RebootButton />
+            </>
+        );
         componentContainer = container;
     });
 
     it("Render.", () => {
-        expect(componentContainer)
+        expect(componentContainer).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 
     it("Render modal.", () => {
-        expect(queryByText(componentContainer, "Confirm reboot"))
+        expect(queryByText(componentContainer, "Confirm reboot")).toBeNull();
-            .toBeNull();
         fireEvent.click(getByText(componentContainer, "Reboot"));
-        expect(componentContainer)
+        expect(componentContainer).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 
     it("Confirm reboot.", () => {
         fireEvent.click(getByText(componentContainer, "Reboot"));
         fireEvent.click(getByText(componentContainer, "Confirm reboot"));
-        expect(mockAxios.post)
+        expect(mockAxios.post).toHaveBeenCalledWith(
-            .toHaveBeenCalledWith("/reforis/api/reboot", undefined, expect.anything());
+            "/reforis/api/reboot",
+            undefined,
+            expect.anything()
+        );
     });
 
     it("Hold error.", async () => {
         fireEvent.click(getByText(componentContainer, "Reboot"));
         fireEvent.click(getByText(componentContainer, "Confirm reboot"));
         mockJSONError();
-        await wait(() => expect(mockSetAlert)
+        await wait(() =>
-            .toBeCalledWith("Reboot request failed."));
+            expect(mockSetAlert).toBeCalledWith("Reboot request failed.")
+        );
     });
 });

src/common/__tests__/__snapshots__/RebootButton.test.js.snap

@@ -22,7 +22,7 @@ exports[`<RebootButton/> Render modal. 1`] = `
             <h5
               class="modal-title"
             >
-              Reboot confirmation
+              Warning!
             </h5>
             <button
               class="close"
@@ -49,16 +49,12 @@ exports[`<RebootButton/> Render modal. 1`] = `
               class="btn btn-primary "
               type="button"
             >
-               
-               
               Cancel
             </button>
             <button
               class="btn btn-danger"
               type="button"
             >
-               
-               
               Confirm reboot
             </button>
           </div>
@@ -70,8 +66,6 @@ exports[`<RebootButton/> Render modal. 1`] = `
     class="btn btn-danger"
     type="button"
   >
-     
-     
     Reboot
   </button>
 </div>
@@ -86,8 +80,6 @@ exports[`<RebootButton/> Render. 1`] = `
     class="btn btn-danger"
     type="button"
   >
-     
-     
     Reboot
   </button>
 </div>

src/form/__tests__/SubmitButton.test.js

@@ -13,17 +13,14 @@ import { STATES, SubmitButton } from "../components/SubmitButton";
 describe("<SubmitButton/>", () => {
     it("Render ready", () => {
         const { container } = render(<SubmitButton state={STATES.READY} />);
-        expect(container)
+        expect(container).toMatchSnapshot();
-            .toMatchSnapshot();
     });
     it("Render saving", () => {
         const { container } = render(<SubmitButton state={STATES.SAVING} />);
-        expect(container)
+        expect(container).toMatchSnapshot();
-            .toMatchSnapshot();
     });
     it("Render load", () => {
         const { container } = render(<SubmitButton state={STATES.LOAD} />);
-        expect(container)
+        expect(container).toMatchSnapshot();
-            .toMatchSnapshot();
     });
 });

src/form/__tests__/__snapshots__/SubmitButton.test.js.snap

@@ -3,7 +3,7 @@
 exports[`<SubmitButton/> Render load 1`] = `
 <div>
   <button
-    class="btn btn-primary  col-sm-12 col-lg-3"
+    class="btn btn-primary  col-sm-12 col-md-3 col-lg-2"
     disabled=""
     type="submit"
   >
@@ -13,8 +13,6 @@ exports[`<SubmitButton/> Render load 1`] = `
       role="status"
     />
      
-     
-     
     Load settings
   </button>
 </div>
@@ -23,11 +21,9 @@ exports[`<SubmitButton/> Render load 1`] = `
 exports[`<SubmitButton/> Render ready 1`] = `
 <div>
   <button
-    class="btn btn-primary  col-sm-12 col-lg-3"
+    class="btn btn-primary  col-sm-12 col-md-3 col-lg-2"
     type="submit"
   >
-     
-     
     Save
   </button>
 </div>
@@ -36,7 +32,7 @@ exports[`<SubmitButton/> Render ready 1`] = `
 exports[`<SubmitButton/> Render saving 1`] = `
 <div>
   <button
-    class="btn btn-primary  col-sm-12 col-lg-3"
+    class="btn btn-primary  col-sm-12 col-md-3 col-lg-2"
     disabled=""
     type="submit"
   >
@@ -46,8 +42,6 @@ exports[`<SubmitButton/> Render saving 1`] = `
       role="status"
     />
      
-     
-     
     Updating
   </button>
 </div>

src/form/__tests__/hooks.test.js

@@ -7,9 +7,7 @@
 
 import React from "react";
 
-import {
+import { act, fireEvent, render, waitForElement } from "customTestRender";
-    act, fireEvent, render, waitForElement,
-} from "customTestRender";
 import mockAxios from "jest-mock-axios";
 import { WebSockets } from "webSockets/WebSockets";
 import { ForisForm } from "../components/ForisForm";
@@ -38,8 +36,12 @@ describe("useForm hook.", () => {
 
     beforeEach(async () => {
         mockPrepData = jest.fn(() => ({ field: "preparedData" }));
-        mockPrepDataToSubmit = jest.fn(() => ({ field: "preparedDataToSubmit" }));
+        mockPrepDataToSubmit = jest.fn(() => ({
-        mockValidator = jest.fn((data) => (data.field === "invalidValue" ? { field: "Error" } : {}));
+            field: "preparedDataToSubmit",
+        }));
+        mockValidator = jest.fn((data) =>
+            data.field === "invalidValue" ? { field: "Error" } : {}
+        );
         const { getByTestId, container } = render(
             <ForisForm
                 ws={new WebSockets()}
@@ -53,7 +55,7 @@ describe("useForm hook.", () => {
                 validator={mockValidator}
             >
                 <Child />
-            </ForisForm>,
+            </ForisForm>
         );
         mockAxios.mockResponse({ field: "fetchedData" });
 
@@ -67,16 +69,22 @@ describe("useForm hook.", () => {
         expect(Child.mock.calls[0][0].formErrors).toMatchObject({});
 
         act(() => {
-            fireEvent.change(input, { target: { value: "invalidValue", type: "text" } });
+            fireEvent.change(input, {
+                target: { value: "invalidValue", type: "text" },
+            });
         });
 
         expect(Child).toHaveBeenCalledTimes(2);
         expect(mockValidator).toHaveBeenCalledTimes(2);
-        expect(Child.mock.calls[1][0].formErrors).toMatchObject({ field: "Error" });
+        expect(Child.mock.calls[1][0].formErrors).toMatchObject({
+            field: "Error",
+        });
     });
 
     it("Update text value.", () => {
-        fireEvent.change(input, { target: { value: "newValue", type: "text" } });
+        fireEvent.change(input, {
+            target: { value: "newValue", type: "text" },
+        });
         expect(input.value).toBe("newValue");
     });
 
@@ -86,14 +94,21 @@ describe("useForm hook.", () => {
     });
 
     it("Update checkbox value.", () => {
-        fireEvent.change(input, { target: { checked: true, type: "checkbox" } });
+        fireEvent.change(input, {
+            target: { checked: true, type: "checkbox" },
+        });
         expect(input.checked).toBe(true);
     });
 
     it("Fetch data.", () => {
-        expect(mockAxios.get).toHaveBeenCalledWith("testEndpoint", expect.anything());
+        expect(mockAxios.get).toHaveBeenCalledWith(
+            "testEndpoint",
+            expect.anything()
+        );
         expect(mockPrepData).toHaveBeenCalledTimes(1);
-        expect(Child.mock.calls[0][0].formData).toMatchObject({ field: "preparedData" });
+        expect(Child.mock.calls[0][0].formData).toMatchObject({
+            field: "preparedData",
+        });
     });
 
     it("Submit.", () => {
@@ -107,7 +122,7 @@ describe("useForm hook.", () => {
         expect(mockAxios.post).toHaveBeenCalledWith(
             "testEndpoint",
             { field: "preparedDataToSubmit" },
-            expect.anything(),
+            expect.anything()
         );
     });
 });

src/form/__tests__/validation.test.js

@@ -12,124 +12,79 @@ import {
     validateIPv6Address,
     validateIPv6Prefix,
     validateMAC,
-} from "validations";
+} from "utils/validations";
 
 describe("Validation functions", () => {
     it("validateIPv4Address valid", () => {
-        expect(validateIPv4Address("192.168.1.1"))
+        expect(validateIPv4Address("192.168.1.1")).toBe(undefined);
-            .toBe(undefined);
+        expect(validateIPv4Address("1.1.1.1")).toBe(undefined);
-        expect(validateIPv4Address("1.1.1.1"))
+        expect(validateIPv4Address("0.0.0.0")).toBe(undefined);
-            .toBe(undefined);
-        expect(validateIPv4Address("0.0.0.0"))
-            .toBe(undefined);
     });
     it("validateIPv4Address invalid", () => {
-        expect(validateIPv4Address("invalid"))
+        expect(validateIPv4Address("invalid")).not.toBe(undefined);
-            .not
+        expect(validateIPv4Address("192.256.1.1")).not.toBe(undefined);
-            .toBe(undefined);
+        expect(validateIPv4Address("192.168.256.1")).not.toBe(undefined);
-        expect(validateIPv4Address("192.256.1.1"))
+        expect(validateIPv4Address("192.168.1.256")).not.toBe(undefined);
-            .not
+        expect(validateIPv4Address("192.168.1.256")).not.toBe(undefined);
-            .toBe(undefined);
-        expect(validateIPv4Address("192.168.256.1"))
-            .not
-            .toBe(undefined);
-        expect(validateIPv4Address("192.168.1.256"))
-            .not
-            .toBe(undefined);
-        expect(validateIPv4Address("192.168.1.256"))
-            .not
-            .toBe(undefined);
     });
 
     it("validateIPv6Address valid", () => {
-        expect(validateIPv6Address("2001:0db8:85a3:0000:0000:8a2e:0370:7334"))
+        expect(
-            .toBe(undefined);
+            validateIPv6Address("2001:0db8:85a3:0000:0000:8a2e:0370:7334")
-        expect(validateIPv6Address("0:0:0:0:0:0:0:1"))
+        ).toBe(undefined);
-            .toBe(undefined);
+        expect(validateIPv6Address("0:0:0:0:0:0:0:1")).toBe(undefined);
-        expect(validateIPv6Address("::1"))
+        expect(validateIPv6Address("::1")).toBe(undefined);
-            .toBe(undefined);
+        expect(validateIPv6Address("::")).toBe(undefined);
-        expect(validateIPv6Address("::"))
-            .toBe(undefined);
     });
     it("validateIPv6Address invalid", () => {
-        expect(validateIPv6Address("invalid"))
+        expect(validateIPv6Address("invalid")).not.toBe(undefined);
-            .not
+        expect(validateIPv6Address("1.1.1.1")).not.toBe(undefined);
-            .toBe(undefined);
+        expect(validateIPv6Address("1200::AB00:1234::2552:7777:1313")).not.toBe(
-        expect(validateIPv6Address("1.1.1.1"))
+            undefined
-            .not
+        );
-            .toBe(undefined);
+        expect(
-        expect(validateIPv6Address("1200::AB00:1234::2552:7777:1313"))
+            validateIPv6Address("1200:0000:AB00:1234:O000:2552:7777:1313")
-            .not
+        ).not.toBe(undefined);
-            .toBe(undefined);
-        expect(validateIPv6Address("1200:0000:AB00:1234:O000:2552:7777:1313"))
-            .not
-            .toBe(undefined);
     });
 
     it("validateIPv6Prefix valid", () => {
-        expect(validateIPv6Prefix("2002:0000::/16"))
+        expect(validateIPv6Prefix("2002:0000::/16")).toBe(undefined);
-            .toBe(undefined);
+        expect(validateIPv6Prefix("0::/0")).toBe(undefined);
-        expect(validateIPv6Prefix("0::/0"))
-            .toBe(undefined);
     });
     it("validateIPv6Prefix invalid", () => {
-        expect(validateIPv6Prefix("2001:0db8:85a3:0000:0000:8a2e:0370:7334"))
+        expect(
-            .not
+            validateIPv6Prefix("2001:0db8:85a3:0000:0000:8a2e:0370:7334")
-            .toBe(undefined);
+        ).not.toBe(undefined);
-        expect(validateIPv6Prefix("::1"))
+        expect(validateIPv6Prefix("::1")).not.toBe(undefined);
-            .not
+        expect(validateIPv6Prefix("2002:0000::/999")).not.toBe(undefined);
-            .toBe(undefined);
-        expect(validateIPv6Prefix("2002:0000::/999"))
-            .not
-            .toBe(undefined);
     });
 
     it("validateDomain valid", () => {
-        expect(validateDomain("example.com"))
+        expect(validateDomain("example.com")).toBe(undefined);
-            .toBe(undefined);
+        expect(validateDomain("one.two.three")).toBe(undefined);
-        expect(validateDomain("one.two.three"))
-            .toBe(undefined);
     });
     it("validateDomain invalid", () => {
-        expect(validateDomain("test/"))
+        expect(validateDomain("test/")).not.toBe(undefined);
-            .not
+        expect(validateDomain(".")).not.toBe(undefined);
-            .toBe(undefined);
-        expect(validateDomain("."))
-            .not
-            .toBe(undefined);
     });
 
     it("validateDUID valid", () => {
-        expect(validateDUID("abcdefAB"))
+        expect(validateDUID("abcdefAB")).toBe(undefined);
-            .toBe(undefined);
+        expect(validateDUID("ABCDEF12")).toBe(undefined);
-        expect(validateDUID("ABCDEF12"))
+        expect(validateDUID("ABCDEF12AB")).toBe(undefined);
-            .toBe(undefined);
-        expect(validateDUID("ABCDEF12AB"))
-            .toBe(undefined);
     });
     it("validateDUID invalid", () => {
-        expect(validateDUID("gggggggg"))
+        expect(validateDUID("gggggggg")).not.toBe(undefined);
-            .not
+        expect(validateDUID("abcdefABa")).not.toBe(undefined);
-            .toBe(undefined);
-        expect(validateDUID("abcdefABa"))
-            .not
-            .toBe(undefined);
     });
 
     it("validateMAC valid", () => {
-        expect(validateMAC("00:D0:56:F2:B5:12"))
+        expect(validateMAC("00:D0:56:F2:B5:12")).toBe(undefined);
-            .toBe(undefined);
+        expect(validateMAC("00:26:DD:14:C4:EE")).toBe(undefined);
-        expect(validateMAC("00:26:DD:14:C4:EE"))
+        expect(validateMAC("06:00:00:00:00:00")).toBe(undefined);
-            .toBe(undefined);
-        expect(validateMAC("06:00:00:00:00:00"))
-            .toBe(undefined);
     });
     it("validateMAC invalid", () => {
-        expect(validateMAC("00:00:00:00:00:0G"))
+        expect(validateMAC("00:00:00:00:00:0G")).not.toBe(undefined);
-            .not
+        expect(validateMAC("06:00:00:00:00:00:00")).not.toBe(undefined);
-            .toBe(undefined);
-        expect(validateMAC("06:00:00:00:00:00:00"))
-            .not
-            .toBe(undefined);
     });
 });

src/form/components/ForisForm.js

@@ -7,11 +7,10 @@
 
 import React, { useEffect } from "react";
 import PropTypes from "prop-types";
-import { Prompt } from "react-router";
+import { Prompt } from "react-router-dom";
 
 import { ALERT_TYPES } from "../../bootstrap/Alert";
 import { API_STATE } from "../../api/utils";
-import { ErrorMessage } from "../../utils/ErrorMessage";
 import { formFieldsSize } from "../../bootstrap/constants";
 import { Spinner } from "../../bootstrap/Spinner";
 import { useAlert } from "../../alertContext/AlertContext";
@@ -19,6 +18,7 @@ import { useAPIPost } from "../../api/hooks";
 
 import { useForisModule, useForm } from "../hooks";
 import { STATES as SUBMIT_BUTTON_STATES, SubmitButton } from "./SubmitButton";
+import { ErrorMessage } from "../../utils/ErrorMessage";
 
 ForisForm.propTypes = {
     /** Optional WebSocket object. See `scr/common/WebSockets.js`.
@@ -27,7 +27,7 @@ ForisForm.propTypes = {
     ws: PropTypes.object,
     /** Foris configuration object. See usage in main components. */
     forisConfig: PropTypes.shape({
-        /** reForis Flask aplication API endpoint from `src/common/API.js`. */
+        /** reForis Flask application API endpoint from `src/common/API.js`. */
         endpoint: PropTypes.string.isRequired,
         /** `foris-controller` module name to be used via WebSockets.
          *  It can be use only with `ws` prop.
@@ -38,29 +38,39 @@ ForisForm.propTypes = {
          * */
         wsAction: PropTypes.string,
     }).isRequired,
-    /** Function to prepare data recived from the API before using in forms. */
+    /** Function to prepare data received from the API before using in forms. */
-    prepData: PropTypes.func.isRequired,
+    prepData: PropTypes.func,
     /** Function to prepare data from form before submitting. */
-    prepDataToSubmit: PropTypes.func.isRequired,
+    prepDataToSubmit: PropTypes.func,
     /** Function to handle response to POST request. */
-    postCallback: PropTypes.func.isRequired,
+    postCallback: PropTypes.func,
     /** Validate data and provide validation object. Then validation errors passed to children. */
-    validator: PropTypes.func.isRequired,
+    validator: PropTypes.func,
     /** Disables form */
     disabled: PropTypes.bool,
-    /** reForis form components. */
-    children: PropTypes.node.isRequired,
     /** Optional override of form submit callback */
     onSubmitOverridden: PropTypes.func,
+    /** Reference to actual form element (useful for programmatically submitting it).
+     * Pass the output of useRef hook to this prop.
+     */
+    formReference: PropTypes.object,
+    /** reForis form components. */
+    children: PropTypes.node.isRequired,
 
     // eslint-disable-next-line react/no-unused-prop-types
     customWSProp(props) {
-        const wsModuleIsSpecified = !!(props.forisConfig && props.forisConfig.wsModule);
+        const wsModuleIsSpecified = !!(
+            props.forisConfig && props.forisConfig.wsModule
+        );
         if (props.ws && !wsModuleIsSpecified) {
-            return new Error("forisConfig.wsModule should be specified when ws object is passed.");
+            return new Error(
+                "forisConfig.wsModule should be specified when ws object is passed."
+            );
         }
         if (!props.ws && wsModuleIsSpecified) {
-            return new Error("forisConfig.wsModule is specified without passing ws object.");
+            return new Error(
+                "forisConfig.wsModule is specified without passing ws object."
+            );
         }
     },
 };
@@ -73,7 +83,12 @@ ForisForm.defaultProps = {
     disabled: false,
 };
 
-/** Serves as HOC for all foris forms components. */
+/** Serves as HOC for all foris forms components.
+ *
+ * As `<Prompt />` from `react-router-dom` is used in this component then it required to
+ * use exposed `ReactRouterDOM` object from `react-router-dom` library which is exposed by reForis.
+ * See README for more information.
+ * */
 export function ForisForm({
     ws,
     forisConfig,
@@ -83,10 +98,14 @@ export function ForisForm({
     validator,
     disabled,
     onSubmitOverridden,
+    formReference,
     children,
 }) {
-    const [formState, onFormChangeHandler, resetFormData] = useForm(validator, prepData);
+    const [formState, onFormChangeHandler, resetFormData] = useForm(
-    const [setAlert] = useAlert();
+        validator,
+        prepData
+    );
+    const [setAlert, dismissAlert] = useAlert();
 
     const [forisModuleState] = useForisModule(ws, forisConfig);
     useEffect(() => {
@@ -106,7 +125,7 @@ export function ForisForm({
     }, [postCallback, postState.state, postState.data, setAlert]);
 
     if (forisModuleState.state === API_STATE.ERROR) {
-        return <ErrorMessage />;
+        return <ErrorMessage message={forisModuleState.data} />;
     }
     if (!formState.data) {
         return <Spinner />;
@@ -115,6 +134,7 @@ export function ForisForm({
     function onSubmitHandler(event) {
         event.preventDefault();
         resetFormData();
+        dismissAlert();
         const copiedFormData = JSON.parse(JSON.stringify(formState.data));
         const preparedData = prepDataToSubmit(copiedFormData);
         post({ data: preparedData });
@@ -130,34 +150,45 @@ export function ForisForm({
         return SUBMIT_BUTTON_STATES.READY;
     }
 
-    const formIsDisabled = (disabled
+    const formIsDisabled =
-        || forisModuleState.state === API_STATE.SENDING
+        disabled ||
-        || postState.state === API_STATE.SENDING);
+        forisModuleState.state === API_STATE.SENDING ||
+        postState.state === API_STATE.SENDING;
     const submitButtonIsDisabled = disabled || !!formState.errors;
 
-    const childrenWithFormProps = React.Children.map(
+    const childrenWithFormProps = React.Children.map(children, (child) =>
-        children,
+        React.cloneElement(child, {
-        (child) => React.cloneElement(child, {
+            initialData: formState.initialData,
             formData: formState.data,
             formErrors: formState.errors,
             setFormValue: onFormChangeHandler,
             disabled: formIsDisabled,
-        }),
+        })
     );
 
     const onSubmit = onSubmitOverridden
-        ? onSubmitOverridden(formState.data, onFormChangeHandler, onSubmitHandler)
+        ? onSubmitOverridden(
+              formState.data,
+              onFormChangeHandler,
+              onSubmitHandler
+          )
         : onSubmitHandler;
 
     function getMessageOnLeavingPage() {
-        if (JSON.stringify(formState.data) === JSON.stringify(formState.initialData)) return true;
+        if (
-        return _("Changes you made may not be saved. Are you sure you want to leave?");
+            JSON.stringify(formState.data) ===
+            JSON.stringify(formState.initialData)
+        )
+            return true;
+        return _(
+            "Changes you made may not be saved. Are you sure you want to leave?"
+        );
     }
 
     return (
         <div className={formFieldsSize}>
             <Prompt message={getMessageOnLeavingPage} />
-            <form onSubmit={onSubmit}>
+            <form onSubmit={onSubmit} ref={formReference}>
                 {childrenWithFormProps}
                 <div className="text-right">
                     <SubmitButton

src/form/components/ForisForm.md

@@ -1,8 +1,11 @@
-`<ForisForm/>` is Higher-Order Component which encapsulates entire form logic and provides with children required props.
+`<ForisForm/>` is Higher-Order Component which encapsulates entire form logic
-This component structure provides comfort API and allows to create typical Foris module forms easily.
+and provides with children required props. This component structure provides
+comfort API and allows to create typical Foris module forms easily.
 
 ## Example of usage of `<ForisForm/>`
+
 You can pass more forms as children.
+
 ```js
 <ForisForm
     ws={ws}
@@ -24,7 +27,10 @@ You can pass more forms as children.
 
 ```js
 export default function MACForm({
-    formData, formErrors, setFormValue, ...props
+    formData,
+    formErrors,
+    setFormValue,
+    ...props
 }) {
     const macSettings = formData.mac_settings;
     const errors = (formErrors || {}).mac_settings || {};
@@ -35,38 +41,33 @@ export default function MACForm({
                 label={_("Custom MAC address")}
                 checked={macSettings.custom_mac_enabled}
                 helpText={HELP_TEXTS.custom_mac_enabled}
-
+                onChange={setFormValue((value) => ({
-                onChange={setFormValue(
+                    mac_settings: { custom_mac_enabled: { $set: value } },
-                    (value) => ({ mac_settings: { custom_mac_enabled: { $set: value } } }),
+                }))}
-                )}
-
                 {...props}
             />
-            {macSettings.custom_mac_enabled
+            {macSettings.custom_mac_enabled ? (
-                ? (
+                <TextInput
-                    <TextInput
+                    label={_("MAC address")}
-                        label={_("MAC address")}
+                    value={macSettings.custom_mac || ""}
-                        value={macSettings.custom_mac || ""}
+                    helpText={HELP_TEXTS.custom_mac}
-                        helpText={HELP_TEXTS.custom_mac}
+                    error={errors.custom_mac}
-                        error={errors.custom_mac}
+                    required
-                        required
+                    onChange={setFormValue((value) => ({
-
+                        mac_settings: { custom_mac: { $set: value } },
-                        onChange={setFormValue(
+                    }))}
-                            (value) => ({ mac_settings: { custom_mac: { $set: value } } }),
+                    {...props}
-                        )}
+                />
-
+            ) : null}
-                        {...props}
-                    />
-                )
-                : null}
         </>
     );
 }
 ```
+
 The <ForisForm/> passes subsequent `props` to the child components.
 
 | Prop           | Type   | Description                                                                |
-|----------------|--------|----------------------------------------------------------------------------|
+| -------------- | ------ | -------------------------------------------------------------------------- |
 | `formData`     | object | Data returned from API.                                                    |
 | `formErrors`   | object | Errors returned after validation via validator.                            |
 | `setFormValue` | func   | Function for data update. It takes update rule as arg (see example above). |

src/form/components/SubmitButton.js

@@ -18,8 +18,7 @@ export const STATES = {
 
 SubmitButton.propTypes = {
     disabled: PropTypes.bool,
-    state: PropTypes.oneOf(Object.keys(STATES)
+    state: PropTypes.oneOf(Object.keys(STATES).map((key) => STATES[key])),
-        .map((key) => STATES[key])),
 };
 
 export function SubmitButton({ disabled, state, ...props }) {
@@ -28,14 +27,14 @@ export function SubmitButton({ disabled, state, ...props }) {
 
     let labelSubmitButton;
     switch (state) {
-    case STATES.SAVING:
+        case STATES.SAVING:
-        labelSubmitButton = _("Updating");
+            labelSubmitButton = _("Updating");
-        break;
+            break;
-    case STATES.LOAD:
+        case STATES.LOAD:
-        labelSubmitButton = _("Load settings");
+            labelSubmitButton = _("Load settings");
-        break;
+            break;
-    default:
+        default:
-        labelSubmitButton = _("Save");
+            labelSubmitButton = _("Save");
     }
 
     return (
@@ -44,7 +43,6 @@ export function SubmitButton({ disabled, state, ...props }) {
             loading={loadingSubmitButton}
             disabled={disableSubmitButton}
             forisFormSize
-
             {...props}
         >
             {labelSubmitButton}

src/form/hooks.js

@@ -23,57 +23,61 @@ export function useForm(validator, dataPreprocessor) {
         errors: {},
     });
 
-    const onFormReload = useCallback((data) => {
+    const onFormReload = useCallback(
-        dispatch({
+        (data) => {
-            type: FORM_ACTIONS.resetData,
+            dispatch({
-            data,
+                type: FORM_ACTIONS.resetData,
-            dataPreprocessor,
+                data,
-            validator,
+                dataPreprocessor,
-        });
+                validator,
-    }, [dataPreprocessor, validator]);
+            });
+        },
+        [dataPreprocessor, validator]
+    );
 
-    const onFormChangeHandler = useCallback((updateRule) => (event) => {
+    const onFormChangeHandler = useCallback(
-        dispatch({
+        (updateRule) => (event) => {
-            type: FORM_ACTIONS.updateValue,
+            dispatch({
-            value: getChangedValue(event.target),
+                type: FORM_ACTIONS.updateValue,
-            updateRule,
+                value: getChangedValue(event.target),
-            validator,
+                updateRule,
-        });
+                validator,
-    }, [validator]);
+            });
+        },
+        [validator]
+    );
 
-    return [
+    return [state, onFormChangeHandler, onFormReload];
-        state,
-        onFormChangeHandler,
-        onFormReload,
-    ];
 }
 
 function formReducer(state, action) {
     switch (action.type) {
-    case FORM_ACTIONS.updateValue: {
+        case FORM_ACTIONS.updateValue: {
-        const newData = update(state.data, action.updateRule(action.value));
+            const newData = update(state.data, action.updateRule(action.value));
-        const errors = action.validator(newData);
+            const errors = action.validator(newData);
-        return {
+            return {
-            ...state,
+                ...state,
-            data: newData,
+                data: newData,
-            errors,
+                errors,
-        };
+            };
-    }
-    case FORM_ACTIONS.resetData: {
-        if (!action.data) {
-            return { ...state, initialData: state.data };
         }
+        case FORM_ACTIONS.resetData: {
+            if (!action.data) {
+                return { ...state, initialData: state.data };
+            }
 
-        const data = action.dataPreprocessor ? action.dataPreprocessor(action.data) : action.data;
+            const data = action.dataPreprocessor
-        return {
+                ? action.dataPreprocessor(action.data)
-            data,
+                : action.data;
-            initialData: data,
+            return {
-            errors: action.data ? action.validator(data) : undefined,
+                data,
-        };
+                initialData: data,
-    }
+                errors: action.data ? action.validator(data) : undefined,
-    default: {
+            };
-        throw new Error();
+        }
-    }
+        default: {
+            throw new Error();
+        }
     }
 }
 

src/index.js

@@ -30,26 +30,21 @@ export { PasswordInput } from "./bootstrap/PasswordInput";
 export { Radio, RadioSet } from "./bootstrap/RadioSet";
 export { Select } from "./bootstrap/Select";
 export { TextInput } from "./bootstrap/TextInput";
-export { formFieldsSize } from "./bootstrap/constants";
+export { formFieldsSize, buttonFormFieldsSize } from "./bootstrap/constants";
+export { Switch } from "./bootstrap/Switch";
 
-export {
+export { Spinner, SpinnerElement } from "./bootstrap/Spinner";
-    Spinner,
+export { Modal, ModalBody, ModalFooter, ModalHeader } from "./bootstrap/Modal";
-    SpinnerElement,
-} from "./bootstrap/Spinner";
-export {
-    Modal,
-    ModalBody,
-    ModalFooter,
-    ModalHeader,
-} from "./bootstrap/Modal";
 
 // Common
 export { RebootButton } from "./common/RebootButton";
 export { WiFiSettings } from "./common/WiFiSettings/WiFiSettings";
-
 // Form
 export { ForisForm } from "./form/components/ForisForm";
-export { SubmitButton, STATES as SUBMIT_BUTTON_STATES } from "./form/components/SubmitButton";
+export {
+    SubmitButton,
+    STATES as SUBMIT_BUTTON_STATES,
+} from "./form/components/SubmitButton";
 export { useForisModule, useForm } from "./form/hooks";
 
 // WebSockets
@@ -58,15 +53,27 @@ export { WebSockets } from "./webSockets/WebSockets";
 
 // Utils
 export { Portal } from "./utils/Portal";
-export { undefinedIfEmpty, withoutUndefinedKeys, onlySpecifiedKeys } from "./utils/objectHelpers";
 export {
-    withEither, withSpinner, withSending, withSpinnerOnSending, withError, withErrorMessage,
+    undefinedIfEmpty,
+    withoutUndefinedKeys,
+    onlySpecifiedKeys,
+} from "./utils/objectHelpers";
+export {
+    withEither,
+    withSpinner,
+    withSending,
+    withSpinnerOnSending,
+    withError,
+    withErrorMessage,
 } from "./utils/conditionalHOCs";
 export { ErrorMessage } from "./utils/ErrorMessage";
 export { useClickOutside } from "./utils/hooks";
+export { toLocaleDateString } from "./utils/datetime";
+export { displayCard } from "./utils/displayCard";
+export { isPluginInstalled } from "./utils/isPluginInstalled";
 
 // Foris URL
-export { ForisURLs, REFORIS_URL_PREFIX } from "./forisUrls";
+export { ForisURLs, REFORIS_URL_PREFIX } from "./utils/forisUrls";
 
 // Validation
 export {
@@ -77,7 +84,7 @@ export {
     validateDUID,
     validateMAC,
     validateMultipleEmails,
-} from "./validations";
+} from "./utils/validations";
 
 // Alert context
 export { AlertContextProvider, useAlert } from "./alertContext/AlertContext";

src/testUtils/alertContextMock.js

@@ -15,7 +15,7 @@ window.AlertContext = React.createContext();
 function AlertContextMock({ children }) {
     return (
         <AlertContext.Provider value={[mockSetAlert, mockDismissAlert]}>
-            { children }
+            {children}
         </AlertContext.Provider>
     );
 }

src/testUtils/customTestRender.js

@@ -9,7 +9,7 @@
 
 import React from "react";
 import { UIDReset } from "react-uid";
-import { StaticRouter } from "react-router";
+import { StaticRouter } from "react-router-dom";
 import { render } from "@testing-library/react";
 import PropTypes from "prop-types";
 
@@ -26,15 +26,14 @@ function Wrapper({ children }) {
     return (
         <AlertContextMock>
             <StaticRouter>
-                <UIDReset>
+                <UIDReset>{children}</UIDReset>
-                    {children}
-                </UIDReset>
             </StaticRouter>
         </AlertContextMock>
     );
 }
 
-const customTestRender = (ui, options) => render(ui, { wrapper: Wrapper, ...options });
+const customTestRender = (ui, options) =>
+    render(ui, { wrapper: Wrapper, ...options });
 
 // re-export everything
 export * from "@testing-library/react";

src/testUtils/mockGlobals.js

@@ -0,0 +1,12 @@
+/*
+ * Copyright (C) 2020 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ *
+ * This is free software, licensed under the GNU General Public License v3.
+ * See /LICENSE for more information.
+ */
+
+// Mock babel (gettext)
+global._ = (str) => str;
+global.ngettext = (str) => str;
+global.babel = { format: (str) => str };
+global.ForisTranslations = { locale: "en" };

src/testUtils/network.js

@@ -8,5 +8,7 @@
 import mockAxios from "jest-mock-axios";
 
 export function mockJSONError(data) {
-    mockAxios.mockError({ response: { data, headers: { "content-type": "application/json" } } });
+    mockAxios.mockError({
+        response: { data, headers: { "content-type": "application/json" } },
+    });
 }

src/testUtils/setup.js

@@ -7,18 +7,13 @@
 
 import mockAxios from "jest-mock-axios";
 import moment from "moment-timezone";
+import "./mockGlobals";
 
 // Setup axios cleanup
 global.afterEach(() => {
     mockAxios.reset();
 });
 
-// Mock babel (gettext)
-global._ = (str) => str;
-global.ngettext = (str) => str;
-global.babel = { format: (str) => str };
-global.ForisTranslations = {};
-
 // Mock web sockets
 window.WebSocket = jest.fn();
 

src/utils/ErrorMessage.js

@@ -6,11 +6,16 @@
  */
 
 import React from "react";
+import PropTypes from "prop-types";
 
-export function ErrorMessage() {
+ErrorMessage.propTypes = {
-    return (
+    message: PropTypes.string,
-        <p className="text-center text-danger">
+};
-            {_("An error occurred while fetching data.")}
+
-        </p>
+ErrorMessage.defaultProps = {
-    );
+    message: _("An error occurred while fetching data."),
+};
+
+export function ErrorMessage({ message }) {
+    return <p className="text-center text-danger">{message}</p>;
 }

src/utils/__tests__/conditionalHOCs.test.js

@@ -9,7 +9,12 @@ import React from "react";
 import { render } from "customTestRender";
 import { API_STATE } from "api/utils";
 import {
-    withEither, withSpinner, withSending, withSpinnerOnSending, withError, withErrorMessage,
+    withEither,
+    withSpinner,
+    withSending,
+    withSpinnerOnSending,
+    withError,
+    withErrorMessage,
 } from "../conditionalHOCs";
 
 describe("conditional HOCs", () => {
@@ -52,14 +57,18 @@ describe("conditional HOCs", () => {
         it("should render First component", () => {
             const withAlternative = withSending(Alternative);
             const FirstWithConditional = withAlternative(First);
-            const { getByText } = render(<FirstWithConditional apiState={API_STATE.SUCCESS} />);
+            const { getByText } = render(
+                <FirstWithConditional apiState={API_STATE.SUCCESS} />
+            );
             expect(getByText("First")).toBeDefined();
         });
 
         it("should render Alternative component", () => {
             const withAlternative = withSending(Alternative);
             const FirstWithConditional = withAlternative(First);
-            const { getByText } = render(<FirstWithConditional apiState={API_STATE.SENDING} />);
+            const { getByText } = render(
+                <FirstWithConditional apiState={API_STATE.SENDING} />
+            );
             expect(getByText("Alternative")).toBeDefined();
         });
     });
@@ -67,13 +76,17 @@ describe("conditional HOCs", () => {
     describe("withSpinnerOnSending", () => {
         it("should render First component", () => {
             const FirstWithConditional = withSpinnerOnSending(First);
-            const { getByText } = render(<FirstWithConditional apiState={API_STATE.SUCCESS} />);
+            const { getByText } = render(
+                <FirstWithConditional apiState={API_STATE.SUCCESS} />
+            );
             expect(getByText("First")).toBeDefined();
         });
 
         it("should render spinner", () => {
             const FirstWithConditional = withSpinnerOnSending(First);
-            const { container } = render(<FirstWithConditional apiState={API_STATE.SENDING} />);
+            const { container } = render(
+                <FirstWithConditional apiState={API_STATE.SENDING} />
+            );
             expect(container).toMatchSnapshot();
         });
     });
@@ -97,13 +110,17 @@ describe("conditional HOCs", () => {
     describe("withErrorMessage", () => {
         it("should render First component", () => {
             const FirstWithConditional = withErrorMessage(First);
-            const { getByText } = render(<FirstWithConditional apiState={API_STATE.SUCCESS} />);
+            const { getByText } = render(
+                <FirstWithConditional apiState={API_STATE.SUCCESS} />
+            );
             expect(getByText("First")).toBeDefined();
         });
 
         it("should render error message", () => {
             const FirstWithConditional = withErrorMessage(First);
-            const { container } = render(<FirstWithConditional apiState={API_STATE.ERROR} />);
+            const { container } = render(
+                <FirstWithConditional apiState={API_STATE.ERROR} />
+            );
             expect(container).toMatchSnapshot();
         });
     });

src/utils/__tests__/datetime.test.js

@@ -0,0 +1,49 @@
+/*
+ * Copyright (C) 2019 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ *
+ * This is free software, licensed under the GNU General Public License v3.
+ * See /LICENSE for more information.
+ */
+
+import { toLocaleDateString } from "../datetime";
+
+describe("toLocaleDateString", () => {
+    it("should work with different locale", () => {
+        global.ForisTranslations = { locale: "fr" };
+        expect(toLocaleDateString("2020-02-20T12:51:36+00:00")).toBe(
+            "20 février 2020 12:51"
+        );
+        global.ForisTranslations = { locale: "en" };
+    });
+
+    it("should convert with default format", () => {
+        expect(toLocaleDateString("2020-02-20T12:51:36+00:00")).toBe(
+            "February 20, 2020 12:51 PM"
+        );
+    });
+
+    it("should convert with custom input format", () => {
+        expect(
+            toLocaleDateString("2020-02-20 12:51:36 +0000", {
+                inputFormat: "YYYY-MM-DD HH:mm:ss Z",
+            })
+        ).toBe("February 20, 2020 12:51 PM");
+    });
+
+    it("should convert with custom output format", () => {
+        expect(
+            toLocaleDateString("2020-02-20T12:51:36+00:00", {
+                outputFormat: "LL",
+            })
+        ).toBe("February 20, 2020");
+    });
+
+    it("should convert with custom input and output format", () => {
+        expect(
+            toLocaleDateString("2020-02-20 12:51:36 +0000", {
+                inputFormat: "YYYY-MM-DD HH:mm:ss Z",
+                outputFormat: "LL",
+            })
+        ).toBe("February 20, 2020");
+    });
+});

src/utils/conditionalHOCs.js

@@ -24,8 +24,8 @@ function withEither(conditionalFn, Either) {
 
 function isSending(props) {
     if (Array.isArray(props.apiState)) {
-        return props.apiState.some(
+        return props.apiState.some((state) =>
-            (state) => [API_STATE.INIT, API_STATE.SENDING].includes(state),
+            [API_STATE.INIT, API_STATE.SENDING].includes(state)
         );
     }
     return [API_STATE.INIT, API_STATE.SENDING].includes(props.apiState);
@@ -38,15 +38,18 @@ const withSpinnerOnSending = withSpinner(isSending);
 // Error handling
 
 const withError = (conditionalFn) => withEither(conditionalFn, ErrorMessage);
-const withErrorMessage = withError(
+const withErrorMessage = withError((props) => {
-    (props) => {
+    if (Array.isArray(props.apiState)) {
-        if (Array.isArray(props.apiState)) {
+        return props.apiState.includes(API_STATE.ERROR);
-            return props.apiState.includes(API_STATE.ERROR);
+    }
-        }
+    return props.apiState === API_STATE.ERROR;
-        return props.apiState === API_STATE.ERROR;
+});
-    },
-);
 
 export {
-    withEither, withSpinner, withSending, withSpinnerOnSending, withError, withErrorMessage,
+    withEither,
+    withSpinner,
+    withSending,
+    withSpinnerOnSending,
+    withError,
+    withErrorMessage,
 };

src/utils/datetime.js

@@ -0,0 +1,9 @@
+import moment from "moment";
+
+export function toLocaleDateString(
+    date,
+    { inputFormat, outputFormat = "LLL" } = {}
+) {
+    const parsedDate = inputFormat ? moment(date, inputFormat) : moment(date);
+    return parsedDate.locale(ForisTranslations.locale).format(outputFormat);
+}

src/utils/displayCard.js

@@ -0,0 +1,23 @@
+/*
+ * Copyright (C) 2020 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ *
+ * This is free software, licensed under the GNU General Public License v3.
+ * See /LICENSE for more information.
+ */
+
+export function displayCard({ package_lists: packages }, cardName) {
+    const enabledPackagesNames = [];
+    packages
+        .filter((item) => item.enabled)
+        .map((item) => {
+            enabledPackagesNames.push(item.name);
+            item.options
+                .filter((option) => option.enabled)
+                .map((option) => {
+                    enabledPackagesNames.push(option.name);
+                    return null;
+                });
+            return null;
+        });
+    return enabledPackagesNames.includes(cardName);
+}

src/utils/forisUrls.js

@@ -10,6 +10,8 @@ export const REFORIS_API_URL_PREFIX = `${REFORIS_URL_PREFIX}/api`;
 
 export const ForisURLs = {
     login: `${REFORIS_URL_PREFIX}/login`,
+    logout: `${REFORIS_URL_PREFIX}/logout`,
+
     static: `${REFORIS_URL_PREFIX}/static/reforis`,
     wifi: `${REFORIS_URL_PREFIX}/network-settings/wifi`,
 
@@ -18,10 +20,16 @@ export const ForisURLs = {
         updates: `${REFORIS_URL_PREFIX}/package-management/updates`,
     },
 
+    storage: `${REFORIS_URL_PREFIX}/storage`,
+
     // Notifications links are used with <Link/> inside Router, thus url subdir is not required.
-    notifications: "/notifications",
+    overview: "/overview",
+    notifications: "/overview#notifications",
     notificationsSettings: "/administration/notifications-settings",
 
+    approveUpdates: "/package-management/updates",
+    languages: "/package-management/languages",
+    rebootPage: "/administration/reboot",
     luci: "/cgi-bin/luci",
 
     // API

src/utils/hooks.js

@@ -8,11 +8,17 @@
 import { useState, useEffect } from "react";
 
 /** Execute callback when condition is set to true. */
-export function useConditionalTimeout({ callback, timeout = 125 }, ...callbackArgs) {
+export function useConditionalTimeout(
+    { callback, timeout = 125 },
+    ...callbackArgs
+) {
     const [condition, setCondition] = useState(false);
     useEffect(() => {
         if (condition) {
-            const interval = setTimeout(() => callback(...callbackArgs), timeout);
+            const interval = setTimeout(
+                () => callback(...callbackArgs),
+                timeout
+            );
             return () => setTimeout(interval);
         }
     }, [condition, callback, timeout, callbackArgs]);

src/utils/isPluginInstalled.js

@@ -0,0 +1,9 @@
+/*
+ * Copyright (C) 2020 CZ.NIC z.s.p.o. (http://www.nic.cz/)
+ *
+ * This is free software, licensed under the GNU General Public License v3.
+ * See /LICENSE for more information.
+ */
+
+export const isPluginInstalled = (pluginName) =>
+    ForisPlugins.some((plugin) => plugin.name === pluginName);

src/utils/objectHelpers.js

@@ -15,21 +15,18 @@ export function undefinedIfEmpty(instance) {
 
 /** Return object without keys that have undefined value. */
 export function withoutUndefinedKeys(instance) {
-    return Object.keys(instance).reduce(
+    return Object.keys(instance).reduce((accumulator, key) => {
-        (accumulator, key) => {
+        if (instance[key] !== undefined) {
-            if (instance[key] !== undefined) {
+            accumulator[key] = instance[key];
-                accumulator[key] = instance[key];
+        }
-            }
+        return accumulator;
-            return accumulator;
+    }, {});
-        },
-        {},
-    );
 }
 
 /** Return copy of passed object that has only desired keys. */
 export function onlySpecifiedKeys(object, desiredKeys) {
-    return desiredKeys.reduce(
+    return desiredKeys.reduce((accumulator, key) => {
-        (accumulator, key) => { accumulator[key] = object[key]; return accumulator; },
+        accumulator[key] = object[key];
-        {},
+        return accumulator;
-    );
+    }, {});
 }