fix: fix error handling default options definition

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

diff --git a/src/charging-station/ocpp/OCPPIncomingRequestService.ts b/src/charging-station/ocpp/OCPPIncomingRequestService.ts
index 7c88547a6c63ce285e2009e6561b9d813fc99b53..b8a184632cf4e1791d7540eb2e35a56faa35f7e6 100644 (file)
--- a/src/charging-station/ocpp/OCPPIncomingRequestService.ts
+++ b/src/charging-station/ocpp/OCPPIncomingRequestService.ts
@@ -56,11 +56,12 @@ export abstract class OCPPIncomingRequestService extends EventEmitter {
     error: Error,
     params: HandleErrorParams<T> = { throwError: true, consoleOut: false }
   ): T | undefined {
-    setDefaultErrorParams(params)
+    params = setDefaultErrorParams(params)
     logger.error(
       `${chargingStation.logPrefix()} ${moduleName}.handleIncomingRequestError: Incoming request command '${commandName}' error:`,
       error
     )
+    console.error('params set', params)
     if (params.throwError === false && params.errorResponse != null) {
       return params.errorResponse
     }

diff --git a/src/performance/storage/Storage.ts b/src/performance/storage/Storage.ts
index 46a3a6db0af2170b54c3b6538ae25c10cc1a1796..bdf7acb1d43617bf177a19cba0a6cb1c0d8a04b5 100644 (file)
--- a/src/performance/storage/Storage.ts
+++ b/src/performance/storage/Storage.ts
@@ -31,7 +31,7 @@ export abstract class Storage {
       consoleOut: false
     }
   ): void {
-    setDefaultErrorParams(params, { throwError: false, consoleOut: false })
+    params = setDefaultErrorParams(params, { throwError: false, consoleOut: false })
     const inTableOrCollectionStr = table != null && ` in table or collection '${table}'`
     logger.error(
       `${this.logPrefix} ${this.getDBNameFromStorageType(type)} error '${

diff --git a/src/utils/ErrorUtils.ts b/src/utils/ErrorUtils.ts
index f74c48e8859923ce0eec21976e3e60c089072239..fb23b133476395afa0bd1dc18b651b8f50614146 100644 (file)
--- a/src/utils/ErrorUtils.ts
+++ b/src/utils/ErrorUtils.ts
@@ -40,7 +40,7 @@ export const handleFileException = (
   logPrefix: string,
   params: HandleErrorParams<EmptyObject> = defaultErrorParams
 ): void => {
-  setDefaultErrorParams(params)
+  params = setDefaultErrorParams(params)
   const prefix = isNotEmptyString(logPrefix) ? `${logPrefix} ` : ''
   let logMsg: string
   switch (error.code) {
@@ -88,7 +88,7 @@ export const handleSendMessageError = (
     consoleOut: false
   }
 ): void => {
-  setDefaultErrorParams(params, { throwError: false, consoleOut: false })
+  params = setDefaultErrorParams(params, { throwError: false, consoleOut: false })
   logger.error(
     `${chargingStation.logPrefix()} Send ${getMessageTypeString(messageType)} command '${commandName}' error:`,
     error
@@ -102,6 +102,5 @@ export const setDefaultErrorParams = <T extends JsonType>(
   params: HandleErrorParams<T>,
   defaultParams: HandleErrorParams<T> = defaultErrorParams
 ): HandleErrorParams<T> => {
-  params = { ...defaultParams, ...params }
-  return params
+  return { ...defaultParams, ...params }
 }