-
Notifications
You must be signed in to change notification settings - Fork 31
In development we have had the little conflict that we didn't want to do stuff the user wouldn't expect a function to do.
BUT not setting the Parent (and the parents parents) in the PlcProgram.Browse makes the browsed resources "not browsable" therefore maybe this should be changed...
All reactions
Replies: 3 comments
Only set the Parents when ApiPlcProgramData is provided as parameter:
ApiPlcProgramData
- Pain point: ApiPlcProgramData will be set differently in different functions
All reactions
0 replies
Generally create new ApiPlcProgramData parents from the string:
image
- Pain point: ApiPlcProgramData will only contain the name of the parent (always!)
All reactions
0 replies
Generally doing this would be bad since Arrays need special consideration and this would lead to a "basic method" doing way more than it should be doing, therefore keep functionality like this in specific handlers => ApiPlcProgramHandler:
image
- PlcProgram.Browse cannot be performed on childrens that have been browsed by default after PlcProgram.Browse call
All reactions
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment