Ocpp: always send confirmation (#15950)

charger/ocpp/cp_core.go

@@ -85,42 +85,39 @@ func (cp *CP) MeterValues(request *core.MeterValuesRequest) (*core.MeterValuesCo
 	default:
 	}
 
-	conn := cp.connectorByID(request.ConnectorId)
-	if conn == nil {
-		return nil, ErrInvalidConnector
+	if conn := cp.connectorByID(request.ConnectorId); conn != nil {
+		conn.MeterValues(request)
 	}
 
-	return conn.MeterValues(request)
+	return new(core.MeterValuesConfirmation), nil
 }
 
 func (cp *CP) StartTransaction(request *core.StartTransactionRequest) (*core.StartTransactionConfirmation, error) {
 	if request == nil {
 		return nil, ErrInvalidRequest
 	}
 
-	conn := cp.connectorByID(request.ConnectorId)
-	if conn == nil {
-		return nil, ErrInvalidConnector
+	if conn := cp.connectorByID(request.ConnectorId); conn != nil {
+		return conn.StartTransaction(request)
 	}
 
-	return conn.StartTransaction(request)
+	return new(core.StartTransactionConfirmation), nil
 }
 
 func (cp *CP) StopTransaction(request *core.StopTransactionRequest) (*core.StopTransactionConfirmation, error) {
 	if request == nil {
 		return nil, ErrInvalidRequest
 	}
 
-	conn := cp.connectorByTransactionID(request.TransactionId)
-	if conn == nil {
-		res := &core.StopTransactionConfirmation{
-			IdTagInfo: &types.IdTagInfo{
-				Status: types.AuthorizationStatusAccepted, // accept old pending stop message during startup
-			},
-		}
+	if conn := cp.connectorByTransactionID(request.TransactionId); conn != nil {
+		return conn.StopTransaction(request)
+	}
 
-		return res, nil
+	res := &core.StopTransactionConfirmation{
+		IdTagInfo: &types.IdTagInfo{
+			Status: types.AuthorizationStatusAccepted, // accept old pending stop message during startup
+		},
 	}
 
-	return conn.StopTransaction(request)
+	return res, nil
 }