fix(plc4j/canopen): Fixed a bug introduced by my recent refactoring

apache · Feb 7, 2022 · 5b81e70 · 5b81e70
1 parent 8ebd924
commit 5b81e70
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.../java/org/apache/plc4x/java/canopen/api/conversation/canopen/SDODownloadConversation.java b/.../java/org/apache/plc4x/java/canopen/api/conversation/canopen/SDODownloadConversation.java
@@ -41,7 +41,7 @@ public SDODownloadConversation(CANConversation delegate, int nodeId, int answerN
         this.indexAddress = indexAddress;
 
         try {
-            WriteBufferByteBased writeBuffer = new WriteBufferByteBased(DataItem.getLengthInBytes(value, type, null));
+            WriteBufferByteBased writeBuffer = new WriteBufferByteBased(DataItem.getLengthInBytes(value, type, null), ByteOrder.LITTLE_ENDIAN);
             DataItem.staticSerialize(writeBuffer, value, type, null, ByteOrder.LITTLE_ENDIAN);
             data = writeBuffer.getData();
         } catch (SerializationException e) {

diff --git a/...rs/canopen/src/main/java/org/apache/plc4x/java/canopen/protocol/CANOpenProtocolLogic.java b/...rs/canopen/src/main/java/org/apache/plc4x/java/canopen/protocol/CANOpenProtocolLogic.java
@@ -220,7 +220,7 @@ private void writeInternally(DefaultPlcWriteRequest writeRequest, CANOpenPDOFiel
         try {
             String fieldName = writeRequest.getFieldNames().iterator().next();
 
-            WriteBufferByteBased writeBuffer = new WriteBufferByteBased(DataItem.getLengthInBytes(writeValue, field.getCanOpenDataType(), writeValue.getLength()));
+            WriteBufferByteBased writeBuffer = new WriteBufferByteBased(DataItem.getLengthInBytes(writeValue, field.getCanOpenDataType(), writeValue.getLength()), ByteOrder.LITTLE_ENDIAN);
             DataItem.staticSerialize(writeBuffer, writeValue, field.getCanOpenDataType(), writeValue.getLength(), ByteOrder.LITTLE_ENDIAN);
             final CANOpenPDOPayload payload = new CANOpenPDOPayload(new CANOpenPDO(writeBuffer.getData()));
             context.sendToWire(new CANOpenFrame((short) field.getNodeId(), field.getService(), payload));