From 6483ef0da64ebee5fcf38e1daac7cb9eefb0eb58 Mon Sep 17 00:00:00 2001
From: Shalin Nijel <89510971+shalinnijel2@users.noreply.github.com>
Date: Wed, 13 Mar 2024 15:03:24 +0000
Subject: [PATCH] Added comment

---
 iso15118/secc/states/iso15118_20_states.py | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/iso15118/secc/states/iso15118_20_states.py b/iso15118/secc/states/iso15118_20_states.py
index c561c7d5..02dee0e4 100644
--- a/iso15118/secc/states/iso15118_20_states.py
+++ b/iso15118/secc/states/iso15118_20_states.py
@@ -932,6 +932,11 @@ async def process_message(
             control_mode, schedule_exchange_req
         )
 
+        # As per Table 49 of ISO15118-20 spec: one of scheduled_params/dynamic_params is
+        # required even if EVSEProcessing is ongoing. The SECC shall only omit the
+        # parameter 'ScheduleList' in case EVSEProcessing is set to 'Ongoing'.
+        # However, the schema file doesn't permit this as minOccurs = 0 is not set in
+        # schema here: https://github.com/SwitchEV/iso15118/blob/769eddb0cb780db629b4c736de270d381516abd1/iso15118/shared/schemas/iso15118_20/V2G_CI_CommonMessages.xsd#L467-L466  # noqa
         params = await self.comm_session.evse_controller.get_schedule_exchange_params(
             self.comm_session.selected_energy_service,
             control_mode,