full support for new features from OCPP 1.6

Author: JavaIsJavaScript

src/main/java/de/rwth/idsg/steve/utils/CspUtils.java

@@ -43,10 +43,11 @@ public static boolean checkStartPeriod(Integer[] startPeriod) {
             }
             last = current;
         }
-        //true = good, false = not good
+        // true = passes, false = give error message that values are incorrect
         return startPeriod != null;
     }
 
+    // Check if numberPhases is set and fill the values if needed.
     public static List<Integer> setNumberPhases(Integer[] startPeriod, BigDecimal[] limit, List<Integer> numberPhases) {
         if ((numberPhases == null || numberPhases.size() == 0) && (startPeriod.length == limit.length)) {
             for (int i = 0; i < startPeriod.length; i++) {
@@ -55,11 +56,14 @@ public static List<Integer> setNumberPhases(Integer[] startPeriod, BigDecimal[]
         } else if ((numberPhases.size() == startPeriod.length) && (startPeriod.length == limit.length)) {
             numberPhases = numberPhases;
         } else if ((numberPhases.size() < startPeriod.length) && (startPeriod.length == limit.length)) {
+            // If numberPhases has less values than startPeriod, fill numberPhases with the default value,
+            // "3" as this is the default value according to OCPP 1.6 Specifications
             for (int i = numberPhases.size(); i < startPeriod.length; i++) {
                 numberPhases.add(3);
             }
         } else {
-            //Dirty work around to give an error message that csp values do not have an equivalent amount of values
+            // Dirty work around to give an error message that csp values do not have an equivalent amount of values
+            // When comparing the size of startPeriod and numberPhases.
             numberPhases.add(3);
         }
         return numberPhases;