]> Piment Noir Git Repositories - e-mobility-charging-stations-simulator.git/commitdiff
refactor(ocpp): harmonize buildRequestPayload switch structure across versions
authorJérôme Benoit <jerome.benoit@sap.com>
Wed, 18 Mar 2026 14:49:05 +0000 (15:49 +0100)
committerJérôme Benoit <jerome.benoit@sap.com>
Wed, 18 Mar 2026 14:49:05 +0000 (15:49 +0100)
Group passthrough cases into fall-through blocks in both OCPP 1.6 and
2.0 buildRequestPayload, giving both files a symmetric structure:
grouped passthroughs, heartbeat empty object, enrichment cases, default
error.

src/charging-station/ocpp/1.6/OCPP16RequestService.ts
src/charging-station/ocpp/2.0/OCPP20RequestService.ts

index 5fe55dc83c70410a07c4be15e2e9cf6a36417ba0..d06a817c8379815af926159a1a49ecbd822d0e74 100644 (file)
@@ -179,17 +179,14 @@ export class OCPP16RequestService extends OCPPRequestService {
           ...commandParams,
         } as unknown as Request
       case OCPP16RequestCommand.BOOT_NOTIFICATION:
-        return commandParams as unknown as Request
       case OCPP16RequestCommand.DATA_TRANSFER:
-        return commandParams as unknown as Request
       case OCPP16RequestCommand.DIAGNOSTICS_STATUS_NOTIFICATION:
-        return commandParams as unknown as Request
       case OCPP16RequestCommand.FIRMWARE_STATUS_NOTIFICATION:
+      case OCPP16RequestCommand.METER_VALUES:
+      case OCPP16RequestCommand.STATUS_NOTIFICATION:
         return commandParams as unknown as Request
       case OCPP16RequestCommand.HEARTBEAT:
         return OCPP16Constants.OCPP_REQUEST_EMPTY as unknown as Request
-      case OCPP16RequestCommand.METER_VALUES:
-        return commandParams as unknown as Request
       case OCPP16RequestCommand.START_TRANSACTION:
         return {
           idTag: Constants.DEFAULT_IDTAG,
@@ -213,8 +210,6 @@ export class OCPP16RequestService extends OCPPRequestService {
           }),
           ...commandParams,
         } as unknown as Request
-      case OCPP16RequestCommand.STATUS_NOTIFICATION:
-        return commandParams as unknown as Request
       case OCPP16RequestCommand.STOP_TRANSACTION:
         chargingStation.stationInfo?.transactionDataMeterValues === true &&
           (connectorId = chargingStation.getConnectorIdByTransactionId(
index 21ff7b6bf60546beb7b7719351e241d94fd53dc1..bd240189dd91f493e84dab2bc1b3084abf08ea3d 100644 (file)
@@ -149,7 +149,6 @@ export class OCPP20RequestService extends OCPPRequestService {
     )
     switch (commandName) {
       case OCPP20RequestCommand.BOOT_NOTIFICATION:
-        return commandParams as unknown as Request
       case OCPP20RequestCommand.FIRMWARE_STATUS_NOTIFICATION:
       case OCPP20RequestCommand.GET_15118_EV_CERTIFICATE:
       case OCPP20RequestCommand.GET_CERTIFICATE_STATUS: