Feature/remove clone 1

pom.xml

@@ -7,7 +7,7 @@
 	<description>eBUS core library - This library handles the communication with heating engineering via the BUS specification. This protocol is used by many heating manufacturers in Europe.</description>
 	<groupId>de.cs-dev.ebus</groupId>
 	<artifactId>ebus-core</artifactId>
-	<version>1.1.4</version>
+	<version>1.1.5-SNAPSHOT</version>
 	<url>https://github.com/csowada/ebus</url>
 	<packaging>bundle</packaging>
 

src/main/java/de/csdev/ebus/cfg/std/EBusConfigurationReader.java

@@ -433,7 +433,7 @@ protected EBusCommand parseTelegramConfiguration(@NonNull IEBusCommandCollection
                 for (EBusCommandValue commandValue : templateCollection) {
 
                     // clone the original value
-                    EBusCommandValue clone = commandValue.clone();
+                    EBusCommandValue clone = commandValue.getClonedInstance();
                     clone.setParent(commandMethod);
 
                     overwritePropertiesFromTemplate(clone, valueDto);
@@ -473,7 +473,7 @@ protected EBusCommand parseTelegramConfiguration(@NonNull IEBusCommandCollection
             if (templateCollection != null && !templateCollection.isEmpty()) {
                 for (EBusCommandValue commandValue : templateCollection) {
 
-                    EBusCommandValue clone = commandValue.clone();
+                    EBusCommandValue clone = commandValue.getClonedInstance();
                     clone.setParent(commandMethod);
 
                     overwritePropertiesFromTemplate(clone, valueDto);

src/main/java/de/csdev/ebus/command/EBusCommandNestedValue.java

@@ -56,12 +56,12 @@ protected EBusCommandNestedValue createInstance() {
     }
 
     @Override
-    public @NonNull EBusCommandValue clone() {
-        EBusCommandNestedValue clone = (EBusCommandNestedValue) super.clone();
+    public @NonNull EBusCommandValue getClonedInstance() {
+        EBusCommandNestedValue clone = (EBusCommandNestedValue) super.getClonedInstance();
 
         // deep clone list
         for (IEBusValue value : list) {
-            clone.add(value.clone());
+            clone.add(value.getClonedInstance());
         }
 
         return clone;

src/main/java/de/csdev/ebus/command/EBusCommandValue.java

@@ -63,7 +63,7 @@ protected EBusCommandValue createInstance() {
     }
 
     @Override
-    public @NonNull EBusCommandValue clone() {
+    public @NonNull EBusCommandValue getClonedInstance() {
 
         EBusCommandValue clone = createInstance();
         clone.defaultValue = this.defaultValue;

src/main/java/de/csdev/ebus/command/IEBusValue.java

@@ -22,7 +22,7 @@
  * @author Christian Sowada - Initial contribution
  *
  */
-public interface IEBusValue extends Cloneable {
+public interface IEBusValue {
 
     /**
      * Returns the type of this value
@@ -115,5 +115,5 @@ public interface IEBusValue extends Cloneable {
      *
      * @return
      */
-    public @NonNull IEBusValue clone();
+    public @NonNull IEBusValue getClonedInstance();
 }

src/main/java/de/csdev/ebus/command/datatypes/ext/EBusTypeBytes.java

@@ -22,7 +22,7 @@
  */
 public class EBusTypeBytes extends EBusAbstractType<byte[]> {
 
-    public static String TYPE_BYTES = "bytes";
+    public static final String TYPE_BYTES = "bytes";
 
     private static String[] supportedTypes = new String[] { TYPE_BYTES };
 

src/main/java/de/csdev/ebus/command/datatypes/ext/EBusTypeDate.java

@@ -31,17 +31,17 @@
  */
 public class EBusTypeDate extends EBusAbstractType<EBusDateTime> {
 
-    public static String TYPE_DATE = "date";
+    public static final String TYPE_DATE = "date";
 
-    public static String DEFAULT = "std"; // BDA - 4
+    public static final String DEFAULT = "std"; // BDA - 4
 
-    public static String SHORT = "short"; // BDA:3 - 3
+    public static final String SHORT = "short"; // BDA:3 - 3
 
-    public static String HEX = "hex"; // BDA:3 - 4
+    public static final String HEX = "hex"; // BDA:3 - 4
 
-    public static String HEX_SHORT = "hex_short"; // BDA:3 - 3
+    public static final String HEX_SHORT = "hex_short"; // BDA:3 - 3
 
-    public static String DAYS = "days"; // DAY - 2
+    public static final String DAYS = "days"; // DAY - 2
 
     private static String[] supportedTypes = new String[] { TYPE_DATE };
 

src/main/java/de/csdev/ebus/command/datatypes/ext/EBusTypeDateTime.java

@@ -32,15 +32,15 @@ public class EBusTypeDateTime extends EBusAbstractType<EBusDateTime> {
 
     private static final Logger logger = LoggerFactory.getLogger(EBusTypeDateTime.class);
 
-    public static String TYPE_DATETIME = "datetime";
+    public static final String TYPE_DATETIME = "datetime";
 
-    private static String[] supportedTypes = new String[] { TYPE_DATETIME };
+    public static final String TIME_FIRST = "timeFirst";
 
-    public static String TIME_FIRST = "timeFirst";
+    public static final String VARIANT_DATE = "variantDate";
 
-    public static String VARIANT_DATE = "variantDate";
+    public static final String VARIANT_TIME = "variantTime";
 
-    public static String VARIANT_TIME = "variantTime";
+    private static String[] supportedTypes = new String[] { TYPE_DATETIME };
 
     private boolean timeFirst = true;
 

src/main/java/de/csdev/ebus/command/datatypes/ext/EBusTypeKWCrc.java

@@ -23,9 +23,9 @@
  */
 public class EBusTypeKWCrc extends EBusAbstractType<Byte> implements IEBusComplexType<Byte> {
 
-    public static String TYPE_KW_CRC = "kw-crc";
+    public static final String TYPE_KW_CRC = "kw-crc";
 
-    public static String POS = "pos";
+    public static final String POS = "pos";
 
     public static int pos = 0;
 

src/main/java/de/csdev/ebus/command/datatypes/ext/EBusTypeMultiWord.java

@@ -24,9 +24,9 @@
  */
 public class EBusTypeMultiWord extends EBusAbstractType<BigDecimal> {
 
-    public static String TYPE_MWORD = "mword";
+    public static final String TYPE_MWORD = "mword";
 
-    public static String BLOCK_MULTIPLIER = "multiplier";
+    public static final String BLOCK_MULTIPLIER = "multiplier";
 
     private static String[] supportedTypes = new String[] { TYPE_MWORD };
 

src/main/java/de/csdev/ebus/command/datatypes/ext/EBusTypeString.java

@@ -18,7 +18,7 @@
  */
 public class EBusTypeString extends EBusAbstractType<String> {
 
-    public static String TYPE_STRING = "string";
+    public static final String TYPE_STRING = "string";
 
     private static String[] supportedTypes = new String[] { TYPE_STRING };
 

src/main/java/de/csdev/ebus/command/datatypes/ext/EBusTypeTime.java

@@ -31,21 +31,21 @@
  */
 public class EBusTypeTime extends EBusAbstractType<EBusDateTime> {
 
-    public static String TYPE_TIME = "time";
+    public static final String TYPE_TIME = "time";
 
-    public static String DEFAULT = "std"; // BTI - 3
-    public static String HEX = "hex"; // HTI - 3
+    public static final String DEFAULT = "std"; // BTI - 3
+    public static final String HEX = "hex"; // HTI - 3
 
-    public static String SHORT = "short"; // BTM - 2
-    public static String HEX_SHORT = "hex_short"; // HTM - 2
+    public static final String SHORT = "short"; // BTM - 2
+    public static final String HEX_SHORT = "hex_short"; // HTM - 2
 
-    public static String MINUTES = "minutes"; // MIN - 2
-    public static String MINUTES_SHORT = "minutes_short"; // MIN - 1
+    public static final String MINUTES = "minutes"; // MIN - 2
+    public static final String MINUTES_SHORT = "minutes_short"; // MIN - 1
 
+    public static final String MINUTE_MULTIPLIER = "minuteMultiplier";
+
     private static String[] supportedTypes = new String[] { TYPE_TIME };
 
-    public static String MINUTE_MULTIPLIER = "minuteMultiplier";
-
     private String variant = DEFAULT;
 
     private BigDecimal minuteMultiplier = BigDecimal.valueOf(1);

src/main/java/de/csdev/ebus/command/datatypes/ext/EBusTypeVersion.java

@@ -23,7 +23,7 @@
  */
 public class EBusTypeVersion extends EBusAbstractType<BigDecimal> {
 
-    public static String TYPE_VERSION = "version";
+    public static final String TYPE_VERSION = "version";
 
     private static String[] supportedTypes = new String[] { TYPE_VERSION };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeBCD.java

@@ -24,7 +24,7 @@
  */
 public class EBusTypeBCD extends EBusAbstractType<BigDecimal> {
 
-    public static String TYPE_BCD = "bcd";
+    public static final String TYPE_BCD = "bcd";
 
     private static String[] supportedTypes = new String[] { TYPE_BCD };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeBit.java

@@ -21,12 +21,12 @@
  */
 public class EBusTypeBit extends EBusAbstractType<Boolean> {
 
-    public static String TYPE_BIT = "bit";
+    public static final String TYPE_BIT = "bit";
 
+    public static final String POS = "pos";
+
     private static String[] supportedTypes = new String[] { TYPE_BIT };
 
-    public static String POS = "pos";
-
     private Integer pos = null;
 
     public EBusTypeBit() {

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeByte.java

@@ -16,8 +16,8 @@
  */
 public class EBusTypeByte extends AbstractEBusTypeUnsignedNumber {
 
-    public static String TYPE_UCHAR = "uchar";
-    public static String TYPE_BYTE = "byte";
+    public static final String TYPE_UCHAR = "uchar";
+    public static final String TYPE_BYTE = "byte";
 
     private static String[] supportedTypes = new String[] { TYPE_BYTE, TYPE_UCHAR };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeChar.java

@@ -16,7 +16,7 @@
  */
 public class EBusTypeChar extends AbstractEBusTypeNumber {
 
-    public static String TYPE_CHAR = "char";
+    public static final String TYPE_CHAR = "char";
 
     private static String[] supportedTypes = new String[] { TYPE_CHAR };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeData1b.java

@@ -16,7 +16,7 @@
  */
 public class EBusTypeData1b extends AbstractEBusTypeNumber {
 
-    public static String TYPE_DATA1B = "data1b";
+    public static final String TYPE_DATA1B = "data1b";
 
     private static String[] supportedTypes = new String[] { TYPE_DATA1B };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeData1c.java

@@ -22,7 +22,7 @@
  */
 public class EBusTypeData1c extends AbstractEBusTypeUnsignedNumber {
 
-    public static String TYPE_DATA1C = "data1c";
+    public static final String TYPE_DATA1C = "data1c";
 
     private static String[] supportedTypes = new String[] { TYPE_DATA1C };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeData2b.java

@@ -22,7 +22,7 @@
  */
 public class EBusTypeData2b extends AbstractEBusTypeNumber {
 
-    public static String TYPE_DATA2B = "data2b";
+    public static final String TYPE_DATA2B = "data2b";
 
     private static String[] supportedTypes = new String[] { TYPE_DATA2B };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeData2c.java

@@ -22,7 +22,7 @@
  */
 public class EBusTypeData2c extends AbstractEBusTypeNumber {
 
-    public static String TYPE_DATA2C = "data2c";
+    public static final String TYPE_DATA2C = "data2c";
 
     private static String[] supportedTypes = new String[] { TYPE_DATA2C };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeFloat.java

@@ -29,7 +29,7 @@ public class EBusTypeFloat extends EBusAbstractType<BigDecimal> {
 
     private static final Logger logger = LoggerFactory.getLogger(EBusTypeFloat.class);
 
-    public static String TYPE_FLOAT = "float";
+    public static final String TYPE_FLOAT = "float";
 
     private static String[] supportedTypes = new String[] { TYPE_FLOAT };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeInteger.java

@@ -16,7 +16,7 @@
  */
 public class EBusTypeInteger extends AbstractEBusTypeNumber {
 
-    public static String TYPE_INTEGER = "int";
+    public static final String TYPE_INTEGER = "int";
 
     private static String[] supportedTypes = new String[] { TYPE_INTEGER };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeNumber.java

@@ -16,7 +16,7 @@
  */
 public class EBusTypeNumber extends AbstractEBusTypeNumber {
 
-    public static String TYPE_NUMBER = "number";
+    public static final String TYPE_NUMBER = "number";
 
     private static String[] supportedTypes = new String[] { TYPE_NUMBER };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeUnsignedNumber.java

@@ -16,7 +16,7 @@
  */
 public class EBusTypeUnsignedNumber extends AbstractEBusTypeUnsignedNumber {
 
-    public static String TYPE_UNUMBER = "unumber";
+    public static final String TYPE_UNUMBER = "unumber";
 
     private static String[] supportedTypes = new String[] { TYPE_UNUMBER };
 

src/main/java/de/csdev/ebus/command/datatypes/std/EBusTypeWord.java

@@ -16,8 +16,8 @@
  */
 public class EBusTypeWord extends AbstractEBusTypeUnsignedNumber {
 
-    public static String TYPE_WORD = "word";
-    public static String TYPE_UINT = "uint";
+    public static final String TYPE_WORD = "word";
+    public static final String TYPE_UINT = "uint";
 
     private static String[] supportedTypes = new String[] { TYPE_WORD, TYPE_UINT };