Skip to content

Commit

Permalink
fix(plc4go/eip): fix possible double match in switch
Browse files Browse the repository at this point in the history
  • Loading branch information
sruehl committed Jun 5, 2023
1 parent 85dcb43 commit 096cb0d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions plc4go/internal/eip/Reader.go
Expand Up @@ -198,7 +198,7 @@ func (m *Reader) ToPlc4xReadResponse(response readWriteModel.CipService, readReq
plcValues := map[string]values.PlcValue{}
responseCodes := map[string]apiModel.PlcResponseCode{}
switch response := response.(type) {
case readWriteModel.CipReadResponse: // only 1 tag
case readWriteModel.CipReadResponseExactly: // only 1 tag
cipReadResponse := response
tagName := readRequest.GetTagNames()[0]
tag := readRequest.GetTag(tagName).(EIPPlcTag)
Expand All @@ -215,7 +215,7 @@ func (m *Reader) ToPlc4xReadResponse(response readWriteModel.CipService, readReq
}
plcValues[tagName] = plcValue
responseCodes[tagName] = code
case readWriteModel.MultipleServiceResponse: //Multiple response
case readWriteModel.MultipleServiceResponseExactly: //Multiple response
multipleServiceResponse := response
nb := multipleServiceResponse.GetServiceNb()
arr := make([]readWriteModel.CipService, nb)
Expand Down
4 changes: 2 additions & 2 deletions plc4go/internal/eip/Writer.go
Expand Up @@ -308,12 +308,12 @@ func encodeValue(value apiValues.PlcValue, _type readWriteModel.CIPDataTypeCode,
func (m Writer) ToPlc4xWriteResponse(response readWriteModel.CipService, writeRequest apiModel.PlcWriteRequest) (apiModel.PlcWriteResponse, error) {
responseCodes := map[string]apiModel.PlcResponseCode{}
switch response := response.(type) {
case readWriteModel.CipWriteResponse: // only 1 tag
case readWriteModel.CipWriteResponseExactly: // only 1 tag
cipReadResponse := response
tagName := writeRequest.GetTagNames()[0]
code := decodeResponseCode(cipReadResponse.GetStatus())
responseCodes[tagName] = code
case readWriteModel.MultipleServiceResponse: //Multiple response
case readWriteModel.MultipleServiceResponseExactly: //Multiple response
/* multipleServiceResponse := response
nb := multipleServiceResponse.GetServiceNb()
arr := make([]readWriteModel.CipService, nb)
Expand Down

0 comments on commit 096cb0d

Please sign in to comment.