From 48a57c98df940eb8ebf638870c0f92e9b5b86e83 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sevket=20G=C3=B6kay?= <sevketgokay@gmail.com>
Date: Sun, 5 Jan 2025 15:50:38 +0100
Subject: [PATCH] refactor ChargePointSelect to use OcppProtocol

this DTO should include the protocol which contains transport and version info.
this would enable to unify some logic wrt. Invoker and Client classes.
---
 .../idsg/steve/repository/dto/ChargePointSelect.java     | 9 +++++----
 .../steve/repository/impl/ChargePointRepositoryImpl.java | 2 +-
 .../de/rwth/idsg/steve/web/ChargePointSelectEditor.java  | 6 +++---
 .../resources/webapp/WEB-INF/views/00-cp-multiple.jsp    | 2 +-
 src/main/resources/webapp/WEB-INF/views/00-cp-single.jsp | 2 +-
 5 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/src/main/java/de/rwth/idsg/steve/repository/dto/ChargePointSelect.java b/src/main/java/de/rwth/idsg/steve/repository/dto/ChargePointSelect.java
index 84af7f736..8aec91fac 100644
--- a/src/main/java/de/rwth/idsg/steve/repository/dto/ChargePointSelect.java
+++ b/src/main/java/de/rwth/idsg/steve/repository/dto/ChargePointSelect.java
@@ -18,6 +18,7 @@
  */
 package de.rwth.idsg.steve.repository.dto;
 
+import de.rwth.idsg.steve.ocpp.OcppProtocol;
 import de.rwth.idsg.steve.ocpp.OcppTransport;
 import lombok.Getter;
 import lombok.RequiredArgsConstructor;
@@ -29,14 +30,14 @@
 @RequiredArgsConstructor
 @Getter
 public final class ChargePointSelect {
-    private final OcppTransport ocppTransport;
+    private final OcppProtocol ocppProtocol;
     private final String chargeBoxId;
     private final String endpointAddress;
 
-    public ChargePointSelect(OcppTransport ocppTransport, String chargeBoxId) {
+    public ChargePointSelect(OcppProtocol ocppProtocol, String chargeBoxId) {
         // Provide a non-null value (or placeholder if you will) to frontend for JSON charge points.
         // This is clearly a hack. Not my proudest moment.
-        this(ocppTransport, chargeBoxId, "-");
+        this(ocppProtocol, chargeBoxId, "-");
     }
 
     public boolean isEndpointAddressSet() {
@@ -44,6 +45,6 @@ public boolean isEndpointAddressSet() {
     }
 
     public boolean isSoap() {
-        return OcppTransport.SOAP == ocppTransport;
+        return OcppTransport.SOAP == ocppProtocol.getTransport();
     }
 }
diff --git a/src/main/java/de/rwth/idsg/steve/repository/impl/ChargePointRepositoryImpl.java b/src/main/java/de/rwth/idsg/steve/repository/impl/ChargePointRepositoryImpl.java
index 905d4f4b4..b6af1ca6f 100644
--- a/src/main/java/de/rwth/idsg/steve/repository/impl/ChargePointRepositoryImpl.java
+++ b/src/main/java/de/rwth/idsg/steve/repository/impl/ChargePointRepositoryImpl.java
@@ -100,7 +100,7 @@ public List<ChargePointSelect> getChargePointSelect(OcppProtocol protocol, List<
                   .and(CHARGE_BOX.REGISTRATION_STATUS.in(inStatusFilter))
                   .and(chargeBoxIdCondition)
                   .fetch()
-                  .map(r -> new ChargePointSelect(protocol.getTransport(), r.value1(), r.value2()));
+                  .map(r -> new ChargePointSelect(protocol, r.value1(), r.value2()));
     }
 
     @Override
diff --git a/src/main/java/de/rwth/idsg/steve/web/ChargePointSelectEditor.java b/src/main/java/de/rwth/idsg/steve/web/ChargePointSelectEditor.java
index dd5bc41a1..9f257f16e 100644
--- a/src/main/java/de/rwth/idsg/steve/web/ChargePointSelectEditor.java
+++ b/src/main/java/de/rwth/idsg/steve/web/ChargePointSelectEditor.java
@@ -18,7 +18,7 @@
  */
 package de.rwth.idsg.steve.web;
 
-import de.rwth.idsg.steve.ocpp.OcppTransport;
+import de.rwth.idsg.steve.ocpp.OcppProtocol;
 import de.rwth.idsg.steve.repository.dto.ChargePointSelect;
 
 import java.beans.PropertyEditorSupport;
@@ -34,11 +34,11 @@ public void setAsText(String text) {
         if (!text.isEmpty()) {
             String[] chargePointItem = text.split(";");
 
-            // chargePointItem[0] : ocpp transport type
+            // chargePointItem[0] : ocpp protocol
             // chargePointItem[1] : chargebox id
             // chargePointItem[2] : endpoint (IP) address
             ChargePointSelect cps = new ChargePointSelect(
-                    OcppTransport.fromName(chargePointItem[0]),
+                    OcppProtocol.fromCompositeValue(chargePointItem[0]),
                     chargePointItem[1],
                     chargePointItem[2]
             );
diff --git a/src/main/resources/webapp/WEB-INF/views/00-cp-multiple.jsp b/src/main/resources/webapp/WEB-INF/views/00-cp-multiple.jsp
index 66adbb796..1672313e3 100644
--- a/src/main/resources/webapp/WEB-INF/views/00-cp-multiple.jsp
+++ b/src/main/resources/webapp/WEB-INF/views/00-cp-multiple.jsp
@@ -27,7 +27,7 @@
 		<td>
 			<form:select path="chargePointSelectList" size="5" multiple="true">
 				<c:forEach items="${cpList}" var="cp">
-					<form:option value="${cp.ocppTransport};${cp.chargeBoxId};${cp.endpointAddress}" label="${cp.chargeBoxId}"/>
+					<form:option value="${cp.ocppProtocol};${cp.chargeBoxId};${cp.endpointAddress}" label="${cp.chargeBoxId}"/>
 				</c:forEach>
 			</form:select>
 		</td>
diff --git a/src/main/resources/webapp/WEB-INF/views/00-cp-single.jsp b/src/main/resources/webapp/WEB-INF/views/00-cp-single.jsp
index 75e2f2a3a..d9ab333d5 100644
--- a/src/main/resources/webapp/WEB-INF/views/00-cp-single.jsp
+++ b/src/main/resources/webapp/WEB-INF/views/00-cp-single.jsp
@@ -25,7 +25,7 @@
 		<td>
 			<form:select path="chargePointSelectList" size="5" multiple="false">
 				<c:forEach items="${cpList}" var="cp">
-					<form:option value="${cp.ocppTransport};${cp.chargeBoxId};${cp.endpointAddress}" label="${cp.chargeBoxId}"/>
+					<form:option value="${cp.ocppProtocol};${cp.chargeBoxId};${cp.endpointAddress}" label="${cp.chargeBoxId}"/>
 				</c:forEach>
 			</form:select>
 		</td>