cmseaton42 / node-ethernet-ip

A Lightweight Ethernet/IP API written to interface with Rockwell ControlLogix/CompactLogix Controllers.
MIT License
265 stars 106 forks source link

I added the ability to read strings by modifying 'parseReadMessageResponse(data)' method. I add this to the case/ switch: case 672: this.controller_value = data.toString('ascii', 8); break; #24

Open tjusticeus opened 6 years ago

tjusticeus commented 6 years ago

Current Behavior

Expected Behavior

Possible Solution (Optional)

Context

Steps to Reproduce (for bugs only)

1. 2. 3. 4.

Your Environment

jhenson29 commented 6 years ago

L83...High power! We're actually actively working on integrating string read/write in a more general way along with UDTs right now. Come join us on gitter for more discussion!

austin557 commented 5 years ago

@tjusticeus How exactly were you able to read strings? I really need the ability to just read strings but haven't been able to figure it out yet.

tjusticeus commented 5 years ago

It has been awhile since i looked at this. I will have to refresh my memory. Get back to you soon.

Sent from Yahoo Mail on Android

On Thu, Jul 18, 2019 at 5:02 PM, austin557notifications@github.com wrote:
@tjusticeus How exactly were you able to read strings? I really need the ability to just read strings but haven't been able to figure it out yet.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

tjusticeus commented 5 years ago

go to ethernet-ip/src/tag/index.js

go to method 'parseReadMessageResponse(data)' //Line 304

under the 'break' of case BOOL, I added another case

case 672: this.controller_value = data.toString('ascii', 8); break;

austin557 commented 5 years ago

@tjusticeus Thanks for the help!

RubenFarfanL commented 4 years ago

any know how to write string?