[chargepoint] Fix parent id tag management for transaction stop

Cédric Jimenez · Cédric Jimenez · commit 5231803aef98 · 2022-06-24T14:02:48.000+02:00
diff --git a/src/chargepoint/SimulatedChargePoint.cpp b/src/chargepoint/SimulatedChargePoint.cpp
@@ -502,10 +502,10 @@ bool SimulatedChargePoint::isTransactionStopCondition(MqttManager&
         {
             AuthorizationStatus auth_status;
             std::string         parent_id_tag;
-            auth_status = charge_point.authorize(connector.id, id_tag, connector.parent_id_tag);
+            auth_status = charge_point.authorize(connector.id, id_tag, parent_id_tag);
             if ((auth_status == AuthorizationStatus::Accepted) || (auth_status == AuthorizationStatus::ConcurrentTx))
             {
-                ret = (parent_id_tag == connector.id_tag);
+                ret = !parent_id_tag.empty() && (parent_id_tag == connector.parent_id_tag);
             }
             else
             {

Original file line number	Diff line number	Diff line change
`@@ -502,10 +502,10 @@ bool SimulatedChargePoint::isTransactionStopCondition(MqttManager&`
`502`	`502`	`{`
`503`	`503`	`AuthorizationStatus auth_status;`
`504`	`504`	`std::string parent_id_tag;`
`505`		`- auth_status = charge_point.authorize(connector.id, id_tag, connector.parent_id_tag);`
	`505`	`+ auth_status = charge_point.authorize(connector.id, id_tag, parent_id_tag);`
`506`	`506`	`if ((auth_status == AuthorizationStatus::Accepted) \|\| (auth_status == AuthorizationStatus::ConcurrentTx))`
`507`	`507`	`{`
`508`		`- ret = (parent_id_tag == connector.id_tag);`
	`508`	`+ ret = !parent_id_tag.empty() && (parent_id_tag == connector.parent_id_tag);`
`509`	`509`	`}`
`510`	`510`	`else`
`511`	`511`	`{`