New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(plc4go/codegen): DefaultPlcWriteRequest interface conversion, cau… #335

Merged

sruehl merged 1 commit into apache:develop from hongjinlin:develop

Mar 22, 2022

Contributor

hongjinlin commented Mar 17, 2022

…se it not implement

Description: When I try to write more than one data, the program panic.

... writeRequest, err := connection.WriteRequestBuilder().AddQuery("field1", "400005:UINT", uint16(9900)).AddQuery("field2", "400006:UINT", uint16(9988)).Build() ...

Error message:

(1) panic: interface conversion: model.DefaultPlcWriteRequest is not interceptors.WriterExposer: missing method GetWriter
(2)panic: interface conversion: model.DefaultPlcWriteRequest is not interceptors.WriteRequestInterceptorExposer: missing method GetWriteRequestInterceptor

Result: So I implemented them, and multiple writes to the data were successful!


          fix(plc4go/codegen): DefaultPlcWriteRequest interface conversion, cau…

c8d80c2

…se it not implement

sruehl merged commit cc013c6 into apache:develop

Contributor

sruehl commented Mar 22, 2022

thanks for the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment