Change comparison order in handle get variables

Signed-off-by: Soumya Subramanya <[email protected]>
EVerest · Oct 24, 2023 · 5a2d222 · 5a2d222
1 parent b650dd7
commit 5a2d222
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/config/v201/config.json b/config/v201/config.json
@@ -517,7 +517,7 @@
             "BytesPerMessageGetVariables": {
                 "variable_name": "BytesPerMessage",
                 "attributes": {
-                    "Actual": 42
+                    "Actual": 250
                 },
                 "instance": "GetVariables"
             },
@@ -538,7 +538,7 @@
             "ItemsPerMessageGetVariables": {
                 "variable_name": "ItemsPerMessage",
                 "attributes": {
-                    "Actual": 42
+                    "Actual": 2
                 },
                 "instance": "GetVariables"
             },

diff --git a/lib/ocpp/v201/charge_point.cpp b/lib/ocpp/v201/charge_point.cpp
@@ -1414,18 +1414,18 @@ void ChargePoint::handle_get_variables_req(const EnhancedMessage<v201::MessageTy
     const auto max_bytes_per_message =
         this->device_model->get_value<int>(ControllerComponentVariables::BytesPerMessageGetVariables);
 
-    // B06.FR.17
-    if (message.message_size > max_bytes_per_message) {
+    // B06.FR.16
+    if (msg.getVariableData.size() > max_variables_per_message) {
         // send a CALLERROR
-        const auto call_error = CallError(call.uniqueId, "FormatViolation", "", json({}));
+        const auto call_error = CallError(call.uniqueId, "OccurenceConstraintViolation", "", json({}));
         this->send(call_error);
         return;
     }
 
-    // B06.FR.16
-    if (msg.getVariableData.size() > max_variables_per_message) {
+    // B06.FR.17
+    if (message.message_size > max_bytes_per_message) {
         // send a CALLERROR
-        const auto call_error = CallError(call.uniqueId, "OccurenceConstraintViolation", "", json({}));
+        const auto call_error = CallError(call.uniqueId, "FormatViolation", "", json({}));
         this->send(call_error);
         return;
     }