You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@plc4x.apache.org by cd...@apache.org on 2018/04/13 14:50:07 UTC

[incubator-plc4x] 01/02: - Fixed an accidental swap of parameters

This is an automated email from the ASF dual-hosted git repository.

cdutz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-plc4x.git

commit b6822ae0c50241c24bf4698bcb89c31cfe082916
Author: Christofer Dutz <ch...@c-ware.de>
AuthorDate: Fri Apr 13 14:50:54 2018 +0200

    - Fixed an accidental swap of parameters
---
 .../java/org/apache/plc4x/java/s7/connection/S7PlcConnection.java     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/plc4j/protocols/s7/src/main/java/org/apache/plc4x/java/s7/connection/S7PlcConnection.java b/plc4j/protocols/s7/src/main/java/org/apache/plc4x/java/s7/connection/S7PlcConnection.java
index 396b4c6..22d0d82 100644
--- a/plc4j/protocols/s7/src/main/java/org/apache/plc4x/java/s7/connection/S7PlcConnection.java
+++ b/plc4j/protocols/s7/src/main/java/org/apache/plc4x/java/s7/connection/S7PlcConnection.java
@@ -123,8 +123,8 @@ public class S7PlcConnection extends AbstractPlcConnection implements PlcReader,
 
     @Override
     protected ChannelHandler getChannelHandler(CompletableFuture<Void> sessionSetupCompleteFuture) {
-        short calledTsapId = S7TsapIdEncoder.encodeS7TsapId(DeviceGroup.OTHERS, rack, slot);
-        short callingTsapId = S7TsapIdEncoder.encodeS7TsapId(DeviceGroup.PG_OR_PC, 0, 0);
+        short calledTsapId = S7TsapIdEncoder.encodeS7TsapId(DeviceGroup.PG_OR_PC, 0, 0);
+        short callingTsapId = S7TsapIdEncoder.encodeS7TsapId(DeviceGroup.OTHERS, rack, slot);
 
         return new ChannelInitializer() {
             @Override

-- 
To stop receiving notification emails like this one, please contact
cdutz@apache.org.