Merge pull request #66 from c-jimenez/fix/rpc_error_on_changeavailability_response

c-jimenez · web-flow · commit 508fdb1d8329 · 2022-06-29T23:22:48.000+02:00
[chargepoint - status] Fix RPC error when returning Scheduled or Refused to a ChangeAvailability request
diff --git a/src/chargepoint/status/StatusManager.cpp b/src/chargepoint/status/StatusManager.cpp
@@ -372,10 +372,10 @@ bool StatusManager::handleMessage(const ocpp::messages::ChangeAvailabilityReq& r
                 status = ChargePointStatus::Available;
             }
             m_worker_pool.run<void>([this, connector_id, status] { updateConnectorStatus(connector_id, status); });
-            ret = true;
         }
 
         LOG_INFO << "Change availability " << AvailabilityStatusHelper.toString(response.status);
+        ret = true;
     }
     else
     {

Original file line number	Diff line number	Diff line change
`@@ -372,10 +372,10 @@ bool StatusManager::handleMessage(const ocpp::messages::ChangeAvailabilityReq& r`
`372`	`372`	`status = ChargePointStatus::Available;`
`373`	`373`	`}`
`374`	`374`	`m_worker_pool.run<void>([this, connector_id, status] { updateConnectorStatus(connector_id, status); });`
`375`		`- ret = true;`
`376`	`375`	`}`
`377`	`376`
`378`	`377`	`LOG_INFO << "Change availability " << AvailabilityStatusHelper.toString(response.status);`
	`378`	`+ ret = true;`
`379`	`379`	`}`
`380`	`380`	`else`
`381`	`381`	`{`