@beinformed/ui 1.20.3 → 1.20.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@beinformed/ui",
-  "version": "1.20.3",
+  "version": "1.20.4",
   "description": "Toolbox for be informed javascript layouts",
   "license": "SEE LICENSE IN LICENSE.md",
   "bugs": "http://support.beinformed.com",
@@ -78,7 +78,7 @@
     "styled-components": "^5.0.0"
   },
   "dependencies": {
-    "@babel/runtime-corejs3": "^7.18.3",
+    "@babel/runtime-corejs3": "^7.18.6",
     "big.js": "^6.2.0",
     "date-fns": "^2.28.0",
     "deepmerge": "^4.2.2",
@@ -95,18 +95,18 @@
     "setimmediate": "^1.0.5"
   },
   "devDependencies": {
-    "@babel/cli": "^7.17.10",
-    "@babel/core": "^7.18.5",
+    "@babel/cli": "^7.18.6",
+    "@babel/core": "^7.18.6",
     "@babel/eslint-parser": "^7.18.2",
     "@babel/eslint-plugin": "^7.16.5",
-    "@babel/plugin-proposal-class-properties": "^7.17.12",
+    "@babel/plugin-proposal-class-properties": "^7.18.6",
     "@babel/plugin-syntax-dynamic-import": "^7.8.3",
-    "@babel/plugin-transform-runtime": "^7.18.5",
-    "@babel/preset-env": "^7.18.2",
-    "@babel/preset-flow": "^7.17.12",
-    "@babel/preset-react": "^7.17.12",
-    "@commitlint/cli": "^17.0.2",
-    "@commitlint/config-conventional": "^17.0.2",
+    "@babel/plugin-transform-runtime": "^7.18.6",
+    "@babel/preset-env": "^7.18.6",
+    "@babel/preset-flow": "^7.18.6",
+    "@babel/preset-react": "^7.18.6",
+    "@commitlint/cli": "^17.0.3",
+    "@commitlint/config-conventional": "^17.0.3",
     "@testing-library/react": "^13.3.0",
     "auditjs": "^4.0.37",
     "babel-jest": "^28.1.1",
@@ -121,20 +121,20 @@
     "eslint-plugin-import": "^2.26.0",
     "eslint-plugin-jest": "^26.5.3",
     "eslint-plugin-jsdoc": "^39.3.3",
-    "eslint-plugin-react": "^7.30.0",
+    "eslint-plugin-react": "^7.30.1",
     "eslint-plugin-react-hooks": "^4.5.0",
-    "flow-bin": "^0.180.1",
+    "flow-bin": "^0.181.1",
     "flow-copy-source": "^2.0.9",
-    "flow-typed": "^3.6.1",
+    "flow-typed": "^3.8.0",
     "glob": "^8.0.3",
     "history": "^4.0.0",
     "husky": "^8.0.1",
     "jest": "^28.1.1",
     "jest-environment-jsdom": "^28.1.1",
-    "jest-junit": "^13.2.0",
+    "jest-junit": "^14.0.0",
     "jest-sonar-reporter": "^2.0.0",
     "jscodeshift": "^0.13.1",
-    "lint-staged": "^13.0.2",
+    "lint-staged": "^13.0.3",
     "polished": "^4.0.0",
     "prettier": "^2.7.1",
     "react": "^18.0.0",

package/src/hooks/useAuthentication.js

@@ -27,6 +27,8 @@ type LogoutHook = {
 /**
  */
 export const useLogin = (): LoginHook => {
+  Cache.setItem("isRedirectLogin", false);
+
   const dispatch = useDispatch();
 
   const application = useSelector(getApplication);

package/src/i18n/translations/beinformed_error_messages_en.nl.js

@@ -31,6 +31,7 @@ export default {
   "Constraint.File.InvalidFormat": "Must meet the requirements of the file name",
   "Constraint.File.InvalidMimeType": "Allowed mime types are: ${mime-types}",
   "Constraint.File.MaxFileSizeExceeded": "Maximum file upload size is ${max-filesize}",
+  "Constraint.File.OnlyAllowsOneFile": "Only one file is allowed",
   "Constraint.IBAN.InvalidFormat": "Must be a valid IBAN number",
   "Constraint.InvalidLengthBetween": "Length must be between ${min-length} and ${max-length} characters",
   "Constraint.InvalidLengthExact": "Length must be precisely ${length} characters",
@@ -65,7 +66,7 @@ export default {
   "Constraint.TimeStampRange.InvalidRange": "Start value must be before end value",
   "Constraint.TimeStampRange.Mandatory": "Field is mandatory: must contain at least a start or end value",
   "Constraint.XML.InvalidFormat": "Must be well-formed XML",
-  "Constraint.ZipCode.InvalidFormat": "Must be a valid Dutch ZIP code, e.g. 1234 AB",
+  "Constraint.ZipCode.InvalidFormat": "Must be a valid Dutch ZIP code, e.g. 1234AB",
   "Error.AcceptHeaderRequired": "The 'Accept' request-header field must be provided",
   "Error.Authentication.BlockedUser": "You have been logged out, because your account has been blocked",
   "Error.Authentication.ConcurrentUser": "You have been logged out, since your username has been used at a different location.",
@@ -76,7 +77,7 @@ export default {
   "Error.ChangePasswordRequired": "User password is initial or expired, and must be changed ",
   "Error.CodemapLookup.InvalidToken": "Token for codemap lookup is not valid",
   "Error.DataStore.ConcurrentModification": "Another user has updated a record used in this task",
-  "Error.DataStore.RecordAlreadyExists": "Row cannot be overwritten because it already exists",
+  "Error.DataStore.RecordAlreadyExists": "",
   "Error.DataStore.RowCannotBeLocked": "Another user has already locked a record used in this task",
   "Error.Form.DataNotAvailableForId": "Service can not be completed, data not available for mandatory element ${elementid}",
   "Error.Form.ObjectNotAllowed": "Object '${objectid}' is not allowed",

package/src/i18n/translations/beinformed_error_messages_nl.nl.js

@@ -31,6 +31,7 @@ export default {
   "Constraint.File.InvalidFormat": "Voldoet niet aan de voorwaarden van een bestandsnaam",
   "Constraint.File.InvalidMimeType": "Toegestane mime-types zijn: ${mime-types}",
   "Constraint.File.MaxFileSizeExceeded": "De maximum file grootte voor upload is ${max-filesize}",
+  "Constraint.File.OnlyAllowsOneFile": "Er is slechts één bestand toegestaan",
   "Constraint.IBAN.InvalidFormat": "Moet een geldig IBAN nummer zijn",
   "Constraint.InvalidLengthBetween": "Lengte moet tussen ${min-length} en ${max-length} karakters liggen",
   "Constraint.InvalidLengthExact": "Lengte moet precies ${length} karakters zijn",
@@ -65,18 +66,18 @@ export default {
   "Constraint.TimeStampRange.InvalidRange": "De beginwaarde dient voor de eindwaarde te liggen",
   "Constraint.TimeStampRange.Mandatory": "Veld is verplicht: moet tenminste begin- of eindwaarde bevatten",
   "Constraint.XML.InvalidFormat": "Moet geldige XML zijn",
-  "Constraint.ZipCode.InvalidFormat": "Moet een geldige postcode zijn, bijvoorbeeld 1234 AB",
+  "Constraint.ZipCode.InvalidFormat": "Moet een geldige postcode zijn, bijvoorbeeld 1234AB",
   "Error.AcceptHeaderRequired": "De 'Accept' request-header moet worden meegegeven",
   "Error.Authentication.BlockedUser": "U bent uitgelogd omdat uw account geblokkeerd is",
   "Error.Authentication.ConcurrentUser": "U bent uitgelogd omdat uw gebruikersnaam op een andere locatie is gebruikt om in te loggen.",
   "Error.Authentication.InvalidCredentials": "Inloggen is mislukt",
-  "Error.Authentication.InvalidUsername": "\"U bent uitgelogd omdat uw gebruikersnaam niet meer geldig is",
+  "Error.Authentication.InvalidUsername": "U bent uitgelogd omdat uw gebruikersnaam niet meer geldig is",
   "Error.Authentication.Required": "Autorisatie is vereist",
   "Error.Case.ConcurrentModification": "Een andere gebruiker heeft het dossier aangepast waardoor deze taak nu niet meer is toegestaan",
   "Error.ChangePasswordRequired": "Gebruikerswachtwoord is initieel of verlopen, en moet worden aangepast",
   "Error.CodemapLookup.InvalidToken": "Token voor keuzelijst lookup is niet valide",
   "Error.DataStore.ConcurrentModification": "Een andere gebruiker heeft de rij in de datastore reeds aangepast",
-  "Error.DataStore.RecordAlreadyExists": "Rij kan niet worden overschreven aangezien deze reeds bestaat",
+  "Error.DataStore.RecordAlreadyExists": "",
   "Error.DataStore.RowCannotBeLocked": "Een andere gebruiker heeft de rij in bewerking en daarom kan deze taak niet worden volbracht.",
   "Error.Form.DataNotAvailableForId": "Service kan niet worden afgrond, data voor verplicht element ${elementid} is niet beschikbaar",
   "Error.Form.ObjectNotAllowed": "Object '${objectid}' is niet toegestaan",

package/src/redux/_modularui/actions.js

@@ -116,10 +116,7 @@ export const loadModel = (
         return {
           type: "NO_ACTION",
         };
-      } else if (
-        errorResponse.isResourceNotFoundAfterReload ||
-        errorResponse.isInvalidUsername
-      ) {
+      } else if (errorResponse.isResourceNotFoundAfterReload) {
         return removeModelByKey(key);
       }
 

package/src/redux/actions/Authorization.js

@@ -1,11 +1,8 @@
 // @flow
-import { NOTIFICATION_TYPES } from "../../constants/Constants";
-
 import ErrorResponse from "../../models/error/ErrorResponse";
 
 import { replace } from "../_router/actions";
 import { logoutSuccess } from "./SignOut";
-import { showNotification } from "./Notification";
 
 import Cache from "../../utils/browser/Cache";
 
@@ -16,6 +13,7 @@ import { reloadApplication } from "./Application";
 import { getSetting } from "../../constants";
 
 import type { ThunkAction } from "../types";
+import { sendAuthenticationError } from "./SignIn";
 
 /**
  */
@@ -40,24 +38,13 @@ export const handleUnauthorized =
       dispatch(reloadApplication());
     }
 
-    // set logged auth
+    // set logged out
     dispatch(logoutSuccess());
 
     Cache.removeItem("auth");
 
     if (errorResponse.isInvalidUsername) {
-      const WARNING_TIMEOUT = 4000;
-      dispatch(
-        showNotification(
-          NOTIFICATION_TYPES.WARNING,
-          {
-            id: errorResponse.id,
-            defaultMessage: errorResponse.message,
-          },
-          null,
-          WARNING_TIMEOUT
-        )
-      );
+      dispatch(sendAuthenticationError(errorResponse.message));
     }
 
     // add current location as from location before redirect

package/src/redux/actions/SignIn.js

@@ -9,7 +9,7 @@ import { push } from "../_router/actions";
 import { CHANGEPASSWORD_PATH } from "../../constants/Constants";
 
 import type {
-  LoginFailedAction,
+  SendAuthenticationErrorAction,
   ResetAuthErrorsAction,
   LoginSuccessAction,
   ThunkAction,
@@ -18,7 +18,16 @@ import type {
 /**
  * Send login failed action
  */
-export const loginFailed = (errorMessage: string): LoginFailedAction => ({
+export const loginFailed = (
+  errorMessage: string
+): SendAuthenticationErrorAction => sendAuthenticationError(errorMessage);
+
+/**
+ * Send authentication error action
+ */
+export const sendAuthenticationError = (
+  errorMessage: string
+): SendAuthenticationErrorAction => ({
   type: "AUTHENTICATION_ERROR",
   payload: errorMessage,
 });

package/src/redux/actions/__tests__/Authorization.spec.js

@@ -60,15 +60,8 @@ describe("authorization actions", () => {
         type: "AUTHENTICATION_LOGOUT",
       },
       {
-        type: "SHOW_NOTIFICATION",
-        payload: {
-          error: null,
-          message: {
-            defaultMessage: "Error.Authentication.InvalidUsername",
-            id: "Error.Authentication.InvalidUsername",
-          },
-          type: "WARNING",
-        },
+        type: "AUTHENTICATION_ERROR",
+        payload: "Error.Authentication.InvalidUsername",
       },
       {
         type: "ROUTER/REPLACE",
@@ -114,15 +107,8 @@ describe("authorization actions", () => {
         type: "AUTHENTICATION_LOGOUT",
       },
       {
-        type: "SHOW_NOTIFICATION",
-        payload: {
-          error: null,
-          message: {
-            defaultMessage: "Error.Authentication.InvalidUsername",
-            id: "Error.Authentication.InvalidUsername",
-          },
-          type: "WARNING",
-        },
+        type: "AUTHENTICATION_ERROR",
+        payload: "Error.Authentication.InvalidUsername",
       },
       {
         type: "ROUTER/REPLACE",

package/src/redux/types.js

@@ -103,7 +103,7 @@ export type UpdateProgressAction = {
   payload: { percentComplete: number },
 };
 
-export type LoginFailedAction = {
+export type SendAuthenticationErrorAction = {
   type: "AUTHENTICATION_ERROR",
   payload: string,
 };
@@ -151,8 +151,8 @@ export type ReduxAction =
   | FinishProgressAction
   | ResetProgressAction
   | UpdateProgressAction
-  | LoginFailedAction
   | ResetAuthErrorsAction
+  | SendAuthenticationErrorAction
   | LoginSuccessAction
   | ChangePasswordAction
   | LogoutSuccessAction