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

[bugfix] link for subitem not working for knxprod without modules #47

Closed
wants to merge 1 commit into from

Conversation

olterion
Copy link
Contributor

@olterion olterion commented Sep 4, 2024

For knxprod without modules the link in e.g. ParameterRef pointing to the Parameter is not working. (null pointer exception)

For knxprod without modules the link in e.g. ParameterRef pointing to the Parameter is not working. (null pointer exception)
@thewhobox
Copy link
Member

Module should be null if the item is from the "Main" Module, eg. the direct application.
I think there is a issue before this.
Can you hand me an example ae-manu where this happens?

@thewhobox
Copy link
Member

Is this related to pr #48 ?
Does this still happen without the changes from the other pr?

@olterion
Copy link
Contributor Author

olterion commented Sep 5, 2024

This PR is not only related to #48. There is also this issue, but there is a very simple way to repoduce the error with actual release v1.8.4:

Projekt->Neu->Leer
Applikation->BCU->MV-0012
Applikation Tab->Allgemeines->ParamterRefs automatisch erstellen->deactivate
ParameterTypes->Hinzufügen
Parameter->Hinzufügen->Type->Dummy PT
ParameterRefs->Hinzufügen->Parameter->dummy
Click on link to Parameter -> ERROR "An unhandled exeption occurred: Object reference not set to an instance of an object."

Here is a ae-manu file, where I made this steps:
Parameter_link_error.zip

@thewhobox
Copy link
Member

I tested it. Its fixed with pr #48

@thewhobox thewhobox closed this Sep 10, 2024
@olterion olterion deleted the bugfix_link_modules branch September 10, 2024 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants