Skip to content

[BUG] [CRITICAL] TcoDrivesBeckhoff.TcoMultiAxis Loading position set in automat #50

@KrisztianOlah

Description

@KrisztianOlah

Describe the bug
It is currently possible to load TcoMultiAxis' position set while in automat. This can lead to physical crash with the manipulator, if operator loads a position set belonging a different reference. Furthermore, when loading the position set via UX _identifier does not get updated to the new value so it isn't possible to check on the PLC that the change has even occured!

EDIT: It is also not protected by any user permissions!

To Reproduce
1.) Load position set via plc
2.) Check LoadPositionTask._identifier, it is now updated
3.) Load different position set
4.) LoadPositionTask._identifier is unchanged

Expected behavior
The Load Save, Delete and Create buttons should only be available in manual mode. We could also use a public property or Status item which we can check on the PLC as to what is the current loaded position set, so we don't need to load it on every production cycle.

Screenshots

Image Image

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingquestionFurther information is requested

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions