From 92059c8a0f87183ad4f3084db67dbc1c126ff913 Mon Sep 17 00:00:00 2001
From: =?utf8?q?J=C3=A9r=C3=B4me=20Benoit?= <jerome.benoit@sap.com>
Date: Fri, 3 Apr 2026 23:43:43 +0200
Subject: [PATCH] fix(ui): enable authorize for all OCPP versions in
 StartTransaction
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

The OCPP 2.0.x authorize restriction was a UI-only limitation — the
simulator stack already handles Authorize for both 1.6 and 2.0.x.
Remove the isOCPP20x guard so the checkbox is available for all versions.
---
 ui/web/src/components/actions/StartTransaction.vue | 13 +++----------
 ui/web/tests/unit/StartTransaction.test.ts         |  4 ++--
 2 files changed, 5 insertions(+), 12 deletions(-)

diff --git a/ui/web/src/components/actions/StartTransaction.vue b/ui/web/src/components/actions/StartTransaction.vue
index 896641ff..62a7d8f7 100644
--- a/ui/web/src/components/actions/StartTransaction.vue
+++ b/ui/web/src/components/actions/StartTransaction.vue
@@ -20,7 +20,7 @@
       type="text"
     >
   </p>
-  <p v-if="!isOCPP20x">
+  <p>
     Authorize RFID tag:
     <input
       v-model="state.authorizeIdTag"
@@ -42,13 +42,7 @@ import { useRoute, useRouter } from 'vue-router'
 import { useToast } from 'vue-toast-notification'
 
 import Button from '@/components/buttons/Button.vue'
-import {
-  convertToInt,
-  resetToggleButtonState,
-  ROUTE_NAMES,
-  UIClient,
-  useUIClient,
-} from '@/composables'
+import { convertToInt, resetToggleButtonState, ROUTE_NAMES, useUIClient } from '@/composables'
 import { type OCPPVersion } from '@/types'
 
 const props = defineProps<{
@@ -65,7 +59,6 @@ const evseId = computed(() =>
   $route.query.evseId != null ? Number($route.query.evseId) : undefined
 )
 const ocppVersion = computed(() => $route.query.ocppVersion as OCPPVersion | undefined)
-const isOCPP20x = computed(() => UIClient.isOCPP20x(ocppVersion.value))
 
 const state = ref<{ authorizeIdTag: boolean; idTag: string }>({
   authorizeIdTag: false,
@@ -81,7 +74,7 @@ const toggleButtonId = computed(
 const handleStartTransaction = async (): Promise<void> => {
   const idTag = state.value.idTag.length > 0 ? state.value.idTag : undefined
 
-  if (!isOCPP20x.value && state.value.authorizeIdTag) {
+  if (state.value.authorizeIdTag) {
     if (idTag == null) {
       $toast.error('Please provide an RFID tag to authorize')
       return
diff --git a/ui/web/tests/unit/StartTransaction.test.ts b/ui/web/tests/unit/StartTransaction.test.ts
index 4f00c97b..750f24c7 100644
--- a/ui/web/tests/unit/StartTransaction.test.ts
+++ b/ui/web/tests/unit/StartTransaction.test.ts
@@ -93,9 +93,9 @@ describe('StartTransaction', () => {
       expect(wrapper.find('[type="checkbox"]').exists()).toBe(true)
     })
 
-    it('should hide authorize checkbox for OCPP 2.0.x', () => {
+    it('should show authorize checkbox for OCPP 2.0.x', () => {
       const wrapper = mountComponent({ ocppVersion: OCPPVersion.VERSION_20 })
-      expect(wrapper.find('[type="checkbox"]').exists()).toBe(false)
+      expect(wrapper.find('[type="checkbox"]').exists()).toBe(true)
     })
   })
 
-- 
2.53.0