Skip to content
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.

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

fix: sysManager.LookupTreeItem is not working with the latest 4026 beta #115

Closed
wants to merge 1 commit into from

Conversation

iadonkey
Copy link
Contributor

@iadonkey iadonkey commented Jan 30, 2024

todo:

  • create and use property for the library manager
  • use the property instead of code duplication, there are at least 3 more places where the libManager is looked up like this

@iadonkey iadonkey requested a review from seehma January 30, 2024 13:27
@iadonkey
Copy link
Contributor Author

iadonkey commented Jan 30, 2024

I think we should ask Beckhoff if this is intendet or not ... ITcSysManager16::LookupTreeItem behaves differently as before since the latest beta

var plc = _plc.Object as dynamic;

// Code that was working in the previous beta and TC3.1.4024.xx
// var sysManager = plc.SystemManager as ITcSysManager2;
// var libManager = sysManager.LookupTreeItem(plc.PathName + "^References") as ITcPlcLibraryManager;

// Now we have todo
var libManagerItem = plc.LookupChild("References");
var libManager = libManagerItem as ITcPlcLibraryManager;

@iadonkey iadonkey force-pushed the feature/support_tc_4026_3 branch from 091fbbe to 3e4c770 Compare January 30, 2024 13:33
@iadonkey
Copy link
Contributor Author

already merged fix with #120

@iadonkey iadonkey closed this Feb 14, 2024
@iadonkey iadonkey deleted the feature/support_tc_4026_3 branch February 14, 2024 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant