Code cleanups in the OCPP incoming requests handling code

Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>

diff --git a/src/charging-station/ChargingStation.ts b/src/charging-station/ChargingStation.ts
index d70223e367eb2ecf0b6ef2b656ae475d1b2d66ad..457e3468704553150fb6527f80dabf6cd07a480d 100644 (file)
--- a/src/charging-station/ChargingStation.ts
+++ b/src/charging-station/ChargingStation.ts
@@ -1489,7 +1489,7 @@ export default class ChargingStation {
             }
             logger.debug(
               `${this.logPrefix()} << Command '${
-                requestCommandName ?? 'unknown'
+                requestCommandName ?? Constants.UNKNOWN_COMMAND
               }' received response payload: ${JSON.stringify(request)}`
             );
             responseCallback(commandPayload, requestPayload);
@@ -1519,7 +1519,7 @@ export default class ChargingStation {
             }
             logger.debug(
               `${this.logPrefix()} << Command '${
-                requestCommandName ?? 'unknown'
+                requestCommandName ?? Constants.UNKNOWN_COMMAND
               }' received error payload: ${JSON.stringify(request)}`
             );
             errorCallback(new OCPPError(errorType, errorMessage, requestCommandName, errorDetails));
@@ -1541,7 +1541,7 @@ export default class ChargingStation {
       // Log
       logger.error(
         `${this.logPrefix()} Incoming OCPP command '${
-          commandName ?? requestCommandName ?? null
+          commandName ?? requestCommandName ?? Constants.UNKNOWN_COMMAND
         }' message '${data.toString()}'${
           messageType !== MessageType.CALL_MESSAGE
             ? ` matching cached request '${JSON.stringify(this.requests.get(messageId))}'`
@@ -1552,7 +1552,7 @@ export default class ChargingStation {
       if (error instanceof OCPPError === false) {
         logger.warn(
           `${this.logPrefix()} Error thrown at incoming OCPP command '${
-            commandName ?? requestCommandName ?? null
+            commandName ?? requestCommandName ?? Constants.UNKNOWN_COMMAND
           }' message '${data.toString()}' handling is not an OCPPError:`,
           error
         );

diff --git a/src/utils/Constants.ts b/src/utils/Constants.ts
index b28674259454fce95bf2e37719a1c2e15ee4719d..6f5c8b8b8a7e9a48a0516439d557101c26a2c0ef 100644 (file)
--- a/src/utils/Constants.ts
+++ b/src/utils/Constants.ts
@@ -120,6 +120,8 @@ export default class Constants {
   static readonly DEFAULT_UI_WEBSOCKET_SERVER_HOST = 'localhost';
   static readonly DEFAULT_UI_WEBSOCKET_SERVER_PORT = 8080;
 
+  static readonly UNKNOWN_COMMAND = 'unknown command';
+
   private constructor() {
     // This is intentional
   }