From afa1765595dfe57db3377cf7d44dea334f4b8531 Mon Sep 17 00:00:00 2001 From: Christofer Dutz Date: Mon, 30 Sep 2024 11:28:48 +0200 Subject: [PATCH] refactor: Changed the names of the new items when serialized in xml to hide the implementation detail (Removed the "Default" prefix) --- .../testing/protocols/ads/DriverTestsuite.xml | 8 +- .../testing/protocols/eip/DriverTestsuite.xml | 8 +- .../modbus/tcp/DriverTestsuiteOptimized.xml | 40 ++--- .../Modbus-all-datatypes-little-endian.xml | 168 +++++++++--------- .../modbus/tcp/Modbus-all-datatypes.xml | 168 +++++++++--------- .../java/ads/protocol/AdsProtocolLogic.java | 4 +- .../readwrite/connection/CtrlXConnection.java | 2 +- .../spi/messages/DefaultPlcBrowseItem.java | 4 +- ...tem.java => DefaultPlcBrowseItemList.java} | 13 +- .../spi/messages/DefaultPlcDiscoveryItem.java | 4 +- .../utils/DefaultPlcResponseItem.java | 4 +- .../utils/DefaultPlcTagErrorItem.java | 4 +- .../spi/messages/utils/DefaultPlcTagItem.java | 4 +- .../utils/DefaultPlcTagValueItem.java | 4 +- .../protocols/ads/DriverTestsuite.xml | 8 +- .../protocols/eip/DriverTestsuite.xml | 8 +- .../modbus/tcp/DriverTestsuiteOptimized.xml | 40 ++--- .../Modbus-all-datatypes-little-endian.xml | 168 +++++++++--------- .../modbus/tcp/Modbus-all-datatypes.xml | 168 +++++++++--------- 19 files changed, 413 insertions(+), 414 deletions(-) rename plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/{DefaultListPlcBrowseItem.java => DefaultPlcBrowseItemList.java} (92%) diff --git a/plc4go/assets/testing/protocols/ads/DriverTestsuite.xml b/plc4go/assets/testing/protocols/ads/DriverTestsuite.xml index c98481d81d5..f91926a2aac 100644 --- a/plc4go/assets/testing/protocols/ads/DriverTestsuite.xml +++ b/plc4go/assets/testing/protocols/ads/DriverTestsuite.xml @@ -1150,13 +1150,13 @@ - + MAIN.hurz_Struct - + @@ -1164,7 +1164,7 @@ - + OK @@ -1236,7 +1236,7 @@ - + diff --git a/plc4go/assets/testing/protocols/eip/DriverTestsuite.xml b/plc4go/assets/testing/protocols/eip/DriverTestsuite.xml index c7b346ad434..b6eca5cd50c 100644 --- a/plc4go/assets/testing/protocols/eip/DriverTestsuite.xml +++ b/plc4go/assets/testing/protocols/eip/DriverTestsuite.xml @@ -398,7 +398,7 @@ - + %rate @@ -406,7 +406,7 @@ 1 - + @@ -414,12 +414,12 @@ - + OK 369229824 - + diff --git a/plc4go/assets/testing/protocols/modbus/tcp/DriverTestsuiteOptimized.xml b/plc4go/assets/testing/protocols/modbus/tcp/DriverTestsuiteOptimized.xml index 5d3260a806a..ac935f38bee 100644 --- a/plc4go/assets/testing/protocols/modbus/tcp/DriverTestsuiteOptimized.xml +++ b/plc4go/assets/testing/protocols/modbus/tcp/DriverTestsuiteOptimized.xml @@ -97,7 +97,7 @@ - +
0
@@ -105,7 +105,7 @@ REAL - + @@ -113,12 +113,12 @@ - + OK 3.1415927410125732 - + @@ -194,7 +194,7 @@ - +
0
@@ -202,7 +202,7 @@ REAL - + @@ -210,7 +210,7 @@ - + OK @@ -218,7 +218,7 @@ 3.1415927410125732 - + @@ -298,7 +298,7 @@ - +
0
@@ -306,10 +306,10 @@ REAL - + - +
2
@@ -317,7 +317,7 @@ REAL - + @@ -325,20 +325,20 @@ - + OK 3.1415927410125732 - + - + OK 3.1415927410125732 - + @@ -417,7 +417,7 @@ - +
0
@@ -428,7 +428,7 @@ 3.1415927410125732 - + @@ -516,7 +516,7 @@ - +
0
@@ -530,7 +530,7 @@ 3.1415927410125732 - + diff --git a/plc4go/assets/testing/protocols/modbus/tcp/Modbus-all-datatypes-little-endian.xml b/plc4go/assets/testing/protocols/modbus/tcp/Modbus-all-datatypes-little-endian.xml index bdac150afad..7e46dc7933f 100644 --- a/plc4go/assets/testing/protocols/modbus/tcp/Modbus-all-datatypes-little-endian.xml +++ b/plc4go/assets/testing/protocols/modbus/tcp/Modbus-all-datatypes-little-endian.xml @@ -101,7 +101,7 @@ - +
0
@@ -109,7 +109,7 @@ BOOL - + @@ -117,12 +117,12 @@ - + OK true - + @@ -201,7 +201,7 @@ - +
0
@@ -212,7 +212,7 @@ true - + @@ -296,7 +296,7 @@ - +
1
@@ -304,7 +304,7 @@ BYTE - + @@ -312,12 +312,12 @@ - + OK 42 - + @@ -396,7 +396,7 @@ - +
1
@@ -407,7 +407,7 @@ 42 - + @@ -491,7 +491,7 @@ - +
2
@@ -499,7 +499,7 @@ WORD - + @@ -507,12 +507,12 @@ - + OK 42424 - + @@ -591,7 +591,7 @@ - +
2
@@ -602,7 +602,7 @@ 42424 - + @@ -686,7 +686,7 @@ - +
3
@@ -694,7 +694,7 @@ DWORD - + @@ -702,12 +702,12 @@ - + OK 4242442424 - + @@ -786,7 +786,7 @@ - +
3
@@ -797,7 +797,7 @@ 4242442424 - + @@ -881,7 +881,7 @@ - +
9
@@ -889,7 +889,7 @@ SINT - + @@ -897,12 +897,12 @@ - + OK -42 - + @@ -981,7 +981,7 @@ - +
9
@@ -992,7 +992,7 @@ -42 - + @@ -1076,7 +1076,7 @@ - +
10
@@ -1084,7 +1084,7 @@ USINT - + @@ -1092,12 +1092,12 @@ - + OK 42 - + @@ -1176,7 +1176,7 @@ - +
10
@@ -1187,7 +1187,7 @@ 42 - + @@ -1271,7 +1271,7 @@ - +
11
@@ -1279,7 +1279,7 @@ INT - + @@ -1287,12 +1287,12 @@ - + OK -2424 - + @@ -1371,7 +1371,7 @@ - +
11
@@ -1382,7 +1382,7 @@ -2424 - + @@ -1466,7 +1466,7 @@ - +
12
@@ -1474,7 +1474,7 @@ UINT - + @@ -1482,12 +1482,12 @@ - + OK 42424 - + @@ -1566,7 +1566,7 @@ - +
12
@@ -1577,7 +1577,7 @@ 42424 - + @@ -1661,7 +1661,7 @@ - +
13
@@ -1669,7 +1669,7 @@ DINT - + @@ -1677,12 +1677,12 @@ - + OK -242442424 - + @@ -1761,7 +1761,7 @@ - +
13
@@ -1772,7 +1772,7 @@ -242442424 - + @@ -1856,7 +1856,7 @@ - +
15
@@ -1864,7 +1864,7 @@ UDINT - + @@ -1872,12 +1872,12 @@ - + OK 4242442424 - + @@ -1956,7 +1956,7 @@ - +
15
@@ -1967,7 +1967,7 @@ 4242442424 - + @@ -2051,7 +2051,7 @@ - +
17
@@ -2059,7 +2059,7 @@ LINT - + @@ -2067,12 +2067,12 @@ - + OK -4242442424242424242 - + @@ -2151,7 +2151,7 @@ - +
17
@@ -2162,7 +2162,7 @@ -4242442424242424242 - + @@ -2246,7 +2246,7 @@ - +
21
@@ -2254,7 +2254,7 @@ ULINT - + @@ -2262,12 +2262,12 @@ - + OK 4242442424242424242 - + @@ -2346,7 +2346,7 @@ - +
21
@@ -2357,7 +2357,7 @@ 4242442424242424242 - + @@ -2441,7 +2441,7 @@ - +
25
@@ -2449,7 +2449,7 @@ REAL - + @@ -2457,12 +2457,12 @@ - + OK 3.1415929794311523 - + @@ -2541,7 +2541,7 @@ - +
25
@@ -2552,7 +2552,7 @@ 3.1415929794311523 - + @@ -2636,7 +2636,7 @@ - +
27
@@ -2644,7 +2644,7 @@ LREAL - + @@ -2652,12 +2652,12 @@ - + OK 2.71828182846 - + @@ -2736,7 +2736,7 @@ - +
27
@@ -2747,7 +2747,7 @@ 2.71828182846 - + diff --git a/plc4go/assets/testing/protocols/modbus/tcp/Modbus-all-datatypes.xml b/plc4go/assets/testing/protocols/modbus/tcp/Modbus-all-datatypes.xml index 5b335dfe8a5..938027aff6f 100644 --- a/plc4go/assets/testing/protocols/modbus/tcp/Modbus-all-datatypes.xml +++ b/plc4go/assets/testing/protocols/modbus/tcp/Modbus-all-datatypes.xml @@ -95,7 +95,7 @@ - +
0
@@ -103,7 +103,7 @@ BOOL - + @@ -111,12 +111,12 @@ - + OK true - + @@ -195,7 +195,7 @@ - +
0
@@ -206,7 +206,7 @@ true - + @@ -290,7 +290,7 @@ - +
1
@@ -298,7 +298,7 @@ BYTE - + @@ -306,12 +306,12 @@ - + OK 42 - + @@ -390,7 +390,7 @@ - +
1
@@ -401,7 +401,7 @@ 42 - + @@ -485,7 +485,7 @@ - +
2
@@ -493,7 +493,7 @@ WORD - + @@ -501,12 +501,12 @@ - + OK 42424 - + @@ -585,7 +585,7 @@ - +
2
@@ -596,7 +596,7 @@ 42424 - + @@ -680,7 +680,7 @@ - +
3
@@ -688,7 +688,7 @@ DWORD - + @@ -696,12 +696,12 @@ - + OK 4242442424 - + @@ -780,7 +780,7 @@ - +
3
@@ -791,7 +791,7 @@ 4242442424 - + @@ -875,7 +875,7 @@ - +
9
@@ -883,7 +883,7 @@ SINT - + @@ -891,12 +891,12 @@ - + OK -42 - + @@ -975,7 +975,7 @@ - +
9
@@ -986,7 +986,7 @@ -42 - + @@ -1070,7 +1070,7 @@ - +
10
@@ -1078,7 +1078,7 @@ USINT - + @@ -1086,12 +1086,12 @@ - + OK 42 - + @@ -1170,7 +1170,7 @@ - +
10
@@ -1181,7 +1181,7 @@ 42 - + @@ -1265,7 +1265,7 @@ - +
11
@@ -1273,7 +1273,7 @@ INT - + @@ -1281,12 +1281,12 @@ - + OK -2424 - + @@ -1365,7 +1365,7 @@ - +
11
@@ -1376,7 +1376,7 @@ -2424 - + @@ -1460,7 +1460,7 @@ - +
12
@@ -1468,7 +1468,7 @@ UINT - + @@ -1476,12 +1476,12 @@ - + OK 42424 - + @@ -1560,7 +1560,7 @@ - +
12
@@ -1571,7 +1571,7 @@ 42424 - + @@ -1655,7 +1655,7 @@ - +
13
@@ -1663,7 +1663,7 @@ DINT - + @@ -1671,12 +1671,12 @@ - + OK -242442424 - + @@ -1755,7 +1755,7 @@ - +
13
@@ -1766,7 +1766,7 @@ -242442424 - + @@ -1850,7 +1850,7 @@ - +
15
@@ -1858,7 +1858,7 @@ UDINT - + @@ -1866,12 +1866,12 @@ - + OK 4242442424 - + @@ -1950,7 +1950,7 @@ - +
15
@@ -1961,7 +1961,7 @@ 4242442424 - + @@ -2045,7 +2045,7 @@ - +
17
@@ -2053,7 +2053,7 @@ LINT - + @@ -2061,12 +2061,12 @@ - + OK -4242442424242424242 - + @@ -2145,7 +2145,7 @@ - +
17
@@ -2156,7 +2156,7 @@ -4242442424242424242 - + @@ -2240,7 +2240,7 @@ - +
21
@@ -2248,7 +2248,7 @@ ULINT - + @@ -2256,12 +2256,12 @@ - + OK 4242442424242424242 - + @@ -2340,7 +2340,7 @@ - +
21
@@ -2351,7 +2351,7 @@ 4242442424242424242 - + @@ -2435,7 +2435,7 @@ - +
25
@@ -2443,7 +2443,7 @@ REAL - + @@ -2451,12 +2451,12 @@ - + OK 3.1415929794311523 - + @@ -2535,7 +2535,7 @@ - +
25
@@ -2546,7 +2546,7 @@ 3.1415929794311523 - + @@ -2630,7 +2630,7 @@ - +
27
@@ -2638,7 +2638,7 @@ LREAL - + @@ -2646,12 +2646,12 @@ - + OK 2.71828182846 - + @@ -2730,7 +2730,7 @@ - +
27
@@ -2741,7 +2741,7 @@ 2.71828182846 - + diff --git a/plc4j/drivers/ads/src/main/java/org/apache/plc4x/java/ads/protocol/AdsProtocolLogic.java b/plc4j/drivers/ads/src/main/java/org/apache/plc4x/java/ads/protocol/AdsProtocolLogic.java index b35c0391ece..126949fa50b 100644 --- a/plc4j/drivers/ads/src/main/java/org/apache/plc4x/java/ads/protocol/AdsProtocolLogic.java +++ b/plc4j/drivers/ads/src/main/java/org/apache/plc4x/java/ads/protocol/AdsProtocolLogic.java @@ -521,7 +521,7 @@ public CompletableFuture browseWithInterceptor(PlcBrowseReque itemArrayInfo.add(new DefaultPlcBrowseItemArrayInfo( adsDataTypeArrayInfo.getLowerBound(), adsDataTypeArrayInfo.getUpperBound())); } - DefaultListPlcBrowseItem item = new DefaultListPlcBrowseItem(new SymbolicAdsTag(symbol.getName(), plc4xPlcValueType, arrayInfo), symbol.getName(), + DefaultPlcBrowseItemList item = new DefaultPlcBrowseItemList(new SymbolicAdsTag(symbol.getName(), plc4xPlcValueType, arrayInfo), symbol.getName(), true, !symbol.getFlagReadOnly(), true, false, childMap, options, itemArrayInfo); // Check if this item should be added to the result @@ -584,7 +584,7 @@ protected List getBrowseItems(String basePath, long baseGroupId, adsDataTypeArrayInfo.getLowerBound(), adsDataTypeArrayInfo.getUpperBound())); } // Add the type itself. - values.add(new DefaultListPlcBrowseItem(new SymbolicAdsTag( + values.add(new DefaultPlcBrowseItemList(new SymbolicAdsTag( basePath + "." + child.getPropertyName(), plc4xPlcValueType, arrayInfo), child.getPropertyName(), true, parentWritable, true, false, childMap, options, itemArrayInfo)); } diff --git a/plc4j/drivers/ctrlx/src/main/java/org/apache/plc4x/java/ctrlx/readwrite/connection/CtrlXConnection.java b/plc4j/drivers/ctrlx/src/main/java/org/apache/plc4x/java/ctrlx/readwrite/connection/CtrlXConnection.java index f5b27bb503a..dfff73c39bc 100644 --- a/plc4j/drivers/ctrlx/src/main/java/org/apache/plc4x/java/ctrlx/readwrite/connection/CtrlXConnection.java +++ b/plc4j/drivers/ctrlx/src/main/java/org/apache/plc4x/java/ctrlx/readwrite/connection/CtrlXConnection.java @@ -274,7 +274,7 @@ public CompletableFuture browseWithInterceptor(PlcBrowseReque e.printStackTrace(); }*/ matchingQueryNames.forEach(queryName -> responseItems.get(queryName).add( - new DefaultListPlcBrowseItem( + new DefaultPlcBrowseItemList( new CtrlXTag(curNode, PlcValueType.BOOL, Collections.emptyList()), curNode, true, true, true, false, Collections.emptyMap(), Collections.emptyMap(), Collections.emptyList()))); diff --git a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcBrowseItem.java b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcBrowseItem.java index 72b1a5a2fc3..304737b5aaf 100644 --- a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcBrowseItem.java +++ b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcBrowseItem.java @@ -137,7 +137,7 @@ public PlcBrowseItem resolveArrayItems() { @Override public void serialize(WriteBuffer writeBuffer) throws SerializationException { - writeBuffer.pushContext(getClass().getSimpleName()); + writeBuffer.pushContext("PlcBrowseItem"); writeBuffer.writeString("address", tag.getAddressString().getBytes(StandardCharsets.UTF_8).length * 8, tag.getAddressString(), WithOption.WithEncoding(StandardCharsets.UTF_8.name())); @@ -169,7 +169,7 @@ public void serialize(WriteBuffer writeBuffer) throws SerializationException { } writeBuffer.popContext("options"); } - writeBuffer.popContext(getClass().getSimpleName()); + writeBuffer.popContext("PlcBrowseItem"); } } diff --git a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultListPlcBrowseItem.java b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcBrowseItemList.java similarity index 92% rename from plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultListPlcBrowseItem.java rename to plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcBrowseItemList.java index 56663f354d6..50ad9363a42 100644 --- a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultListPlcBrowseItem.java +++ b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcBrowseItemList.java @@ -27,15 +27,14 @@ import org.apache.plc4x.java.spi.generation.WriteBuffer; import java.nio.charset.StandardCharsets; -import java.util.ArrayList; import java.util.List; import java.util.Map; -public class DefaultListPlcBrowseItem extends DefaultPlcBrowseItem { +public class DefaultPlcBrowseItemList extends DefaultPlcBrowseItem { private final List arrayInformation; - public DefaultListPlcBrowseItem(PlcTag tag, + public DefaultPlcBrowseItemList(PlcTag tag, String name, boolean readable, boolean writable, @@ -80,7 +79,7 @@ protected PlcBrowseItem resolveInternal(PlcTag tag, List @Override public void serialize(WriteBuffer writeBuffer) throws SerializationException { - writeBuffer.pushContext(getClass().getSimpleName()); + writeBuffer.pushContext("PlcBrowseItemList"); writeBuffer.writeString("address", getTag().getAddressString().getBytes(StandardCharsets.UTF_8).length * 8, getTag().getAddressString(), WithOption.WithEncoding(StandardCharsets.UTF_8.name())); @@ -93,7 +92,7 @@ public void serialize(WriteBuffer writeBuffer) throws SerializationException { writeBuffer.pushContext("children"); for (PlcBrowseItem child : getChildren().values()) { writeBuffer.pushContext("child"); - ((DefaultListPlcBrowseItem) child).serialize(writeBuffer); + ((DefaultPlcBrowseItemList) child).serialize(writeBuffer); writeBuffer.popContext("child"); } writeBuffer.popContext("children"); @@ -107,12 +106,12 @@ public void serialize(WriteBuffer writeBuffer) throws SerializationException { optionEntry.getKey(), WithOption.WithEncoding(StandardCharsets.UTF_8.name())); // TODO: Find out how to serialize a PlcValue //writeBuffer.writeString("value", optionEntry.getValue().getBytes(StandardCharsets.UTF_8).length * 8, StandardCharsets.UTF_8.name(), optionEntry.getValue()); - ((DefaultListPlcBrowseItem) optionEntry).serialize(writeBuffer); + ((DefaultPlcBrowseItemList) optionEntry).serialize(writeBuffer); writeBuffer.popContext("option"); } writeBuffer.popContext("options"); } - writeBuffer.popContext(getClass().getSimpleName()); + writeBuffer.popContext("PlcBrowseItemList"); } } diff --git a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcDiscoveryItem.java b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcDiscoveryItem.java index bc3b6855a55..0aecb610036 100644 --- a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcDiscoveryItem.java +++ b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/DefaultPlcDiscoveryItem.java @@ -103,7 +103,7 @@ public String getConnectionUrl() { @Override public void serialize(WriteBuffer writeBuffer) throws SerializationException { - writeBuffer.pushContext(getClass().getSimpleName()); + writeBuffer.pushContext("PlcDiscoveryItem"); writeBuffer.writeString("protocolCode", protocolCode.getBytes(StandardCharsets.UTF_8).length * 8, @@ -134,7 +134,7 @@ public void serialize(WriteBuffer writeBuffer) throws SerializationException { name, WithOption.WithEncoding(StandardCharsets.UTF_8.name())); } - writeBuffer.popContext(getClass().getSimpleName()); + writeBuffer.popContext("PlcDiscoveryItem"); } } diff --git a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcResponseItem.java b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcResponseItem.java index 391ab837b4e..87e4dec387b 100644 --- a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcResponseItem.java +++ b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcResponseItem.java @@ -48,7 +48,7 @@ public T getValue() { @Override public void serialize(WriteBuffer writeBuffer) throws SerializationException { - writeBuffer.pushContext("DefaultPlcResponseItem"); + writeBuffer.pushContext("PlcResponseItem"); String codeName = code.name(); writeBuffer.writeString("code", codeName.getBytes(StandardCharsets.UTF_8).length * 8, @@ -61,7 +61,7 @@ public void serialize(WriteBuffer writeBuffer) throws SerializationException { ((Serializable) value).serialize(writeBuffer); writeBuffer.popContext("value"); } - writeBuffer.popContext("DefaultPlcResponseItem"); + writeBuffer.popContext("PlcResponseItem"); } } diff --git a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagErrorItem.java b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagErrorItem.java index 0dfcab5486d..211497ec632 100644 --- a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagErrorItem.java +++ b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagErrorItem.java @@ -38,9 +38,9 @@ public PlcResponseCode getResponseCode() { @Override public void serialize(WriteBuffer writeBuffer) throws SerializationException { - writeBuffer.pushContext("DefaultPlcTagErrorItem"); + writeBuffer.pushContext("PlcTagErrorItem"); writeBuffer.writeString("responseCode", responseCode.name().length() * 8, responseCode.name()); - writeBuffer.popContext("DefaultPlcTagErrorItem"); + writeBuffer.popContext("PlcTagErrorItem"); } } diff --git a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagItem.java b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagItem.java index c6902abcdb9..f1b15bb4a4c 100644 --- a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagItem.java +++ b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagItem.java @@ -38,13 +38,13 @@ public PlcTag getTag() { @Override public void serialize(WriteBuffer writeBuffer) throws SerializationException { - writeBuffer.pushContext("DefaultPlcTagItem"); + writeBuffer.pushContext("PlcTagItem"); if(tag instanceof Serializable) { writeBuffer.pushContext("tag"); writeBuffer.writeSerializable((Serializable) tag); writeBuffer.popContext("tag"); } - writeBuffer.popContext("DefaultPlcTagItem"); + writeBuffer.popContext("PlcTagItem"); } } diff --git a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagValueItem.java b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagValueItem.java index 934e100d16a..315b9c62b37 100644 --- a/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagValueItem.java +++ b/plc4j/spi/src/main/java/org/apache/plc4x/java/spi/messages/utils/DefaultPlcTagValueItem.java @@ -39,7 +39,7 @@ public PlcValue getValue() { @Override public void serialize(WriteBuffer writeBuffer) throws SerializationException { - writeBuffer.pushContext("DefaultPlcTagValueItem"); + writeBuffer.pushContext("PlcTagValueItem"); if(getTag() instanceof Serializable) { writeBuffer.pushContext("tag"); writeBuffer.writeSerializable((Serializable) getTag()); @@ -50,7 +50,7 @@ public void serialize(WriteBuffer writeBuffer) throws SerializationException { writeBuffer.writeSerializable((Serializable) value); writeBuffer.popContext("value"); } - writeBuffer.popContext("DefaultPlcTagValueItem"); + writeBuffer.popContext("PlcTagValueItem"); } } diff --git a/protocols/ads/src/test/resources/protocols/ads/DriverTestsuite.xml b/protocols/ads/src/test/resources/protocols/ads/DriverTestsuite.xml index c98481d81d5..f91926a2aac 100644 --- a/protocols/ads/src/test/resources/protocols/ads/DriverTestsuite.xml +++ b/protocols/ads/src/test/resources/protocols/ads/DriverTestsuite.xml @@ -1150,13 +1150,13 @@ - + MAIN.hurz_Struct - + @@ -1164,7 +1164,7 @@ - + OK @@ -1236,7 +1236,7 @@ - + diff --git a/protocols/eip/src/test/resources/protocols/eip/DriverTestsuite.xml b/protocols/eip/src/test/resources/protocols/eip/DriverTestsuite.xml index c7b346ad434..b6eca5cd50c 100644 --- a/protocols/eip/src/test/resources/protocols/eip/DriverTestsuite.xml +++ b/protocols/eip/src/test/resources/protocols/eip/DriverTestsuite.xml @@ -398,7 +398,7 @@ - + %rate @@ -406,7 +406,7 @@ 1 - + @@ -414,12 +414,12 @@ - + OK 369229824 - + diff --git a/protocols/modbus/src/test/resources/protocols/modbus/tcp/DriverTestsuiteOptimized.xml b/protocols/modbus/src/test/resources/protocols/modbus/tcp/DriverTestsuiteOptimized.xml index 5d3260a806a..ac935f38bee 100644 --- a/protocols/modbus/src/test/resources/protocols/modbus/tcp/DriverTestsuiteOptimized.xml +++ b/protocols/modbus/src/test/resources/protocols/modbus/tcp/DriverTestsuiteOptimized.xml @@ -97,7 +97,7 @@ - +
0
@@ -105,7 +105,7 @@ REAL - + @@ -113,12 +113,12 @@ - + OK 3.1415927410125732 - + @@ -194,7 +194,7 @@ - +
0
@@ -202,7 +202,7 @@ REAL - + @@ -210,7 +210,7 @@ - + OK @@ -218,7 +218,7 @@ 3.1415927410125732 - + @@ -298,7 +298,7 @@ - +
0
@@ -306,10 +306,10 @@ REAL - + - +
2
@@ -317,7 +317,7 @@ REAL - + @@ -325,20 +325,20 @@ - + OK 3.1415927410125732 - + - + OK 3.1415927410125732 - + @@ -417,7 +417,7 @@ - +
0
@@ -428,7 +428,7 @@ 3.1415927410125732 - + @@ -516,7 +516,7 @@ - +
0
@@ -530,7 +530,7 @@ 3.1415927410125732 - + diff --git a/protocols/modbus/src/test/resources/protocols/modbus/tcp/Modbus-all-datatypes-little-endian.xml b/protocols/modbus/src/test/resources/protocols/modbus/tcp/Modbus-all-datatypes-little-endian.xml index bdac150afad..7e46dc7933f 100644 --- a/protocols/modbus/src/test/resources/protocols/modbus/tcp/Modbus-all-datatypes-little-endian.xml +++ b/protocols/modbus/src/test/resources/protocols/modbus/tcp/Modbus-all-datatypes-little-endian.xml @@ -101,7 +101,7 @@ - +
0
@@ -109,7 +109,7 @@ BOOL - + @@ -117,12 +117,12 @@ - + OK true - + @@ -201,7 +201,7 @@ - +
0
@@ -212,7 +212,7 @@ true - + @@ -296,7 +296,7 @@ - +
1
@@ -304,7 +304,7 @@ BYTE - + @@ -312,12 +312,12 @@ - + OK 42 - + @@ -396,7 +396,7 @@ - +
1
@@ -407,7 +407,7 @@ 42 - + @@ -491,7 +491,7 @@ - +
2
@@ -499,7 +499,7 @@ WORD - + @@ -507,12 +507,12 @@ - + OK 42424 - + @@ -591,7 +591,7 @@ - +
2
@@ -602,7 +602,7 @@ 42424 - + @@ -686,7 +686,7 @@ - +
3
@@ -694,7 +694,7 @@ DWORD - + @@ -702,12 +702,12 @@ - + OK 4242442424 - + @@ -786,7 +786,7 @@ - +
3
@@ -797,7 +797,7 @@ 4242442424 - + @@ -881,7 +881,7 @@ - +
9
@@ -889,7 +889,7 @@ SINT - + @@ -897,12 +897,12 @@ - + OK -42 - + @@ -981,7 +981,7 @@ - +
9
@@ -992,7 +992,7 @@ -42 - + @@ -1076,7 +1076,7 @@ - +
10
@@ -1084,7 +1084,7 @@ USINT - + @@ -1092,12 +1092,12 @@ - + OK 42 - + @@ -1176,7 +1176,7 @@ - +
10
@@ -1187,7 +1187,7 @@ 42 - + @@ -1271,7 +1271,7 @@ - +
11
@@ -1279,7 +1279,7 @@ INT - + @@ -1287,12 +1287,12 @@ - + OK -2424 - + @@ -1371,7 +1371,7 @@ - +
11
@@ -1382,7 +1382,7 @@ -2424 - + @@ -1466,7 +1466,7 @@ - +
12
@@ -1474,7 +1474,7 @@ UINT - + @@ -1482,12 +1482,12 @@ - + OK 42424 - + @@ -1566,7 +1566,7 @@ - +
12
@@ -1577,7 +1577,7 @@ 42424 - + @@ -1661,7 +1661,7 @@ - +
13
@@ -1669,7 +1669,7 @@ DINT - + @@ -1677,12 +1677,12 @@ - + OK -242442424 - + @@ -1761,7 +1761,7 @@ - +
13
@@ -1772,7 +1772,7 @@ -242442424 - + @@ -1856,7 +1856,7 @@ - +
15
@@ -1864,7 +1864,7 @@ UDINT - + @@ -1872,12 +1872,12 @@ - + OK 4242442424 - + @@ -1956,7 +1956,7 @@ - +
15
@@ -1967,7 +1967,7 @@ 4242442424 - + @@ -2051,7 +2051,7 @@ - +
17
@@ -2059,7 +2059,7 @@ LINT - + @@ -2067,12 +2067,12 @@ - + OK -4242442424242424242 - + @@ -2151,7 +2151,7 @@ - +
17
@@ -2162,7 +2162,7 @@ -4242442424242424242 - + @@ -2246,7 +2246,7 @@ - +
21
@@ -2254,7 +2254,7 @@ ULINT - + @@ -2262,12 +2262,12 @@ - + OK 4242442424242424242 - + @@ -2346,7 +2346,7 @@ - +
21
@@ -2357,7 +2357,7 @@ 4242442424242424242 - + @@ -2441,7 +2441,7 @@ - +
25
@@ -2449,7 +2449,7 @@ REAL - + @@ -2457,12 +2457,12 @@ - + OK 3.1415929794311523 - + @@ -2541,7 +2541,7 @@ - +
25
@@ -2552,7 +2552,7 @@ 3.1415929794311523 - + @@ -2636,7 +2636,7 @@ - +
27
@@ -2644,7 +2644,7 @@ LREAL - + @@ -2652,12 +2652,12 @@ - + OK 2.71828182846 - + @@ -2736,7 +2736,7 @@ - +
27
@@ -2747,7 +2747,7 @@ 2.71828182846 - + diff --git a/protocols/modbus/src/test/resources/protocols/modbus/tcp/Modbus-all-datatypes.xml b/protocols/modbus/src/test/resources/protocols/modbus/tcp/Modbus-all-datatypes.xml index 5b335dfe8a5..938027aff6f 100644 --- a/protocols/modbus/src/test/resources/protocols/modbus/tcp/Modbus-all-datatypes.xml +++ b/protocols/modbus/src/test/resources/protocols/modbus/tcp/Modbus-all-datatypes.xml @@ -95,7 +95,7 @@ - +
0
@@ -103,7 +103,7 @@ BOOL - + @@ -111,12 +111,12 @@ - + OK true - + @@ -195,7 +195,7 @@ - +
0
@@ -206,7 +206,7 @@ true - + @@ -290,7 +290,7 @@ - +
1
@@ -298,7 +298,7 @@ BYTE - + @@ -306,12 +306,12 @@ - + OK 42 - + @@ -390,7 +390,7 @@ - +
1
@@ -401,7 +401,7 @@ 42 - + @@ -485,7 +485,7 @@ - +
2
@@ -493,7 +493,7 @@ WORD - + @@ -501,12 +501,12 @@ - + OK 42424 - + @@ -585,7 +585,7 @@ - +
2
@@ -596,7 +596,7 @@ 42424 - + @@ -680,7 +680,7 @@ - +
3
@@ -688,7 +688,7 @@ DWORD - + @@ -696,12 +696,12 @@ - + OK 4242442424 - + @@ -780,7 +780,7 @@ - +
3
@@ -791,7 +791,7 @@ 4242442424 - + @@ -875,7 +875,7 @@ - +
9
@@ -883,7 +883,7 @@ SINT - + @@ -891,12 +891,12 @@ - + OK -42 - + @@ -975,7 +975,7 @@ - +
9
@@ -986,7 +986,7 @@ -42 - + @@ -1070,7 +1070,7 @@ - +
10
@@ -1078,7 +1078,7 @@ USINT - + @@ -1086,12 +1086,12 @@ - + OK 42 - + @@ -1170,7 +1170,7 @@ - +
10
@@ -1181,7 +1181,7 @@ 42 - + @@ -1265,7 +1265,7 @@ - +
11
@@ -1273,7 +1273,7 @@ INT - + @@ -1281,12 +1281,12 @@ - + OK -2424 - + @@ -1365,7 +1365,7 @@ - +
11
@@ -1376,7 +1376,7 @@ -2424 - + @@ -1460,7 +1460,7 @@ - +
12
@@ -1468,7 +1468,7 @@ UINT - + @@ -1476,12 +1476,12 @@ - + OK 42424 - + @@ -1560,7 +1560,7 @@ - +
12
@@ -1571,7 +1571,7 @@ 42424 - + @@ -1655,7 +1655,7 @@ - +
13
@@ -1663,7 +1663,7 @@ DINT - + @@ -1671,12 +1671,12 @@ - + OK -242442424 - + @@ -1755,7 +1755,7 @@ - +
13
@@ -1766,7 +1766,7 @@ -242442424 - + @@ -1850,7 +1850,7 @@ - +
15
@@ -1858,7 +1858,7 @@ UDINT - + @@ -1866,12 +1866,12 @@ - + OK 4242442424 - + @@ -1950,7 +1950,7 @@ - +
15
@@ -1961,7 +1961,7 @@ 4242442424 - + @@ -2045,7 +2045,7 @@ - +
17
@@ -2053,7 +2053,7 @@ LINT - + @@ -2061,12 +2061,12 @@ - + OK -4242442424242424242 - + @@ -2145,7 +2145,7 @@ - +
17
@@ -2156,7 +2156,7 @@ -4242442424242424242 - + @@ -2240,7 +2240,7 @@ - +
21
@@ -2248,7 +2248,7 @@ ULINT - + @@ -2256,12 +2256,12 @@ - + OK 4242442424242424242 - + @@ -2340,7 +2340,7 @@ - +
21
@@ -2351,7 +2351,7 @@ 4242442424242424242 - + @@ -2435,7 +2435,7 @@ - +
25
@@ -2443,7 +2443,7 @@ REAL - + @@ -2451,12 +2451,12 @@ - + OK 3.1415929794311523 - + @@ -2535,7 +2535,7 @@ - +
25
@@ -2546,7 +2546,7 @@ 3.1415929794311523 - + @@ -2630,7 +2630,7 @@ - +
27
@@ -2638,7 +2638,7 @@ LREAL - + @@ -2646,12 +2646,12 @@ - + OK 2.71828182846 - + @@ -2730,7 +2730,7 @@ - +
27
@@ -2741,7 +2741,7 @@ 2.71828182846 - +