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.

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

TraceExplorer: getting the correct node to focus when node is part of a lambda expression #2468

Closed
mgronover opened this issue Aug 13, 2024 · 2 comments
Closed

TraceExplorer: getting the correct node to focus when node is part of a lambda expression #2468

mgronover opened this issue Aug 13, 2024 · 2 comments
Assignees
@mgronover

Comments

@mgronover
Copy link
Member

Also see:

IETS3/iets3.opensource#1032
@mgronover mgronover self-assigned this Aug 13, 2024
@mgronover mgronover changed the title TraceExplorer getting the correct node to focus when node is part of a lambda expression TraceExplorer: getting the correct node to focus when node is part of a lambda expression Aug 13, 2024
mgronover added a commit that referenced this issue Aug 14, 2024
@mgronover
TraceExplorer: add interface for retrieving mapped nodes (#2468)
42c17f1
mgronover added a commit that referenced this issue Aug 14, 2024
@mgronover
TraceExplorer: updated CHANGELOG.md (#2468)
f0de704
@mgronover mgronover mentioned this issue Aug 14, 2024
Merged
mgronover added a commit that referenced this issue Aug 14, 2024
@mgronover
TraceExplorer: fixed CHANGELOG.md (#2468)
9a5e356
mgronover added a commit that referenced this issue Aug 14, 2024
@mgronover
Merge pull request #2469 from mbeddr/feature/2468-TraceExplorer-NodeM…
1e0aa9d 
…apping-for-Lambdas

TraceExplorer: add interface for retrieving mapped nodes (#2468)
@mgronover
Copy link
Member Author

  • fix read access for node mapper in trace explorer.

mgronover added a commit that referenced this issue Aug 19, 2024
@mgronover
TraceExplorer: fix missing read access for node mapper (#2468)
46622ad
@mgronover mgronover mentioned this issue Aug 19, 2024
Merged
mgronover added a commit that referenced this issue Aug 19, 2024
@mgronover
Merge pull request #2481 from mbeddr/bugfix/2468-TraceExplorerRequire…
73ca550 
…sReadAccessForNodeMapping

TraceExplorer: fix missing read access for node mapper (#2468)
mgronover added a commit that referenced this issue Aug 20, 2024
@mgronover
TraceExplorer: weakens too restricted interface again (#2468)
7797745
@mgronover mgronover mentioned this issue Aug 20, 2024
Merged
mgronover added a commit that referenced this issue Aug 20, 2024
@mgronover
Merge pull request #2484 from mbeddr/bugfix/2468-TraceExplorerRequire…
b7eefab 
…sReadAccessForNodeMapping

TraceExplorer: weakens too restricted interface again (#2468)
mgronover added a commit that referenced this issue Aug 20, 2024
@mgronover
TraceExplorer: Fixed NPE (#2468)
c070edb
@mgronover mgronover mentioned this issue Aug 20, 2024
Merged
mgronover added a commit that referenced this issue Aug 20, 2024
@mgronover
TraceExplorer: Harmonised TraceTab interface (#2468)
e6693e3
mgronover added a commit that referenced this issue Aug 21, 2024
@mgronover
Merge pull request #2487 from mbeddr/bugfix/2468-TraceExplorerRequire…
e09bdbf 
…sReadAccessForNodeMapping

TraceExplorer: Fixed NPE (#2468)
mgronover added a commit that referenced this issue Aug 21, 2024
@mgronover
TraceExplorer: Extended ITraceRecord interface and fixed method (#2468)
b4afa9d
@mgronover
Copy link
Member Author

Another problem is, that a trace record stores the nodes but the interface only allows to get the Reference of it within the model. This of cause may result in returning null because in case of a lambda expression the traced node is only a copy of the original without a model.

This can be resolved by introducing the required method to the interface ITraceRecord returning the node instead of the node reference.

@mgronover mgronover mentioned this issue Aug 21, 2024
Merged
mgronover added a commit that referenced this issue Aug 21, 2024
@mgronover
TraceExplorer: Fixed missing method in DefaultTraceRecord (#2468)
f9d14c9
mgronover added a commit that referenced this issue Aug 21, 2024
@mgronover
TraceExplorer: Updated CHANGELOG.md (#2468)
733190b
mgronover added a commit that referenced this issue Aug 21, 2024
@mgronover
Merge pull request #2490 from mbeddr/bugfix/2468-TraceExplorerRequire…
7fb7218 
…sReadAccessForNodeMapping

TraceExplorer: Extended ITraceRecord interface and fixed method (#2468)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mgronover mgronover

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mgronover