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

SimpleAPI: A Bogus/wrong link to a Data is not detected #5110

Open
fredroy opened this issue Nov 13, 2024 · 1 comment
Open

SimpleAPI: A Bogus/wrong link to a Data is not detected #5110

fredroy opened this issue Nov 13, 2024 · 1 comment
Labels
issue: bug (minor) Bug affecting only some users or with no major impact on the framework

Comments

@fredroy
Copy link
Contributor

fredroy commented Nov 13, 2024

High probability to happen in SofaPython3 too.

Problematic issue coming from #5081

    - there is no message/warning whatsoever if the link is wrong (i.e point to the non-existing data). This is understandable if setting a link a posteriori is a feature;
    - the only way to know the link was wrong is to test if getParent() is null... which would mean ideally that it should be tested every time...
@fredroy fredroy added the issue: bug (minor) Bug affecting only some users or with no major impact on the framework label Nov 13, 2024
@bakpaul
Copy link
Contributor

bakpaul commented Nov 13, 2024

Ideas:

  • add an additional checker when data isLinked and getParent is null
  • Remove lazy link (might be braking for XML) and use strong link (@damienmarchal)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue: bug (minor) Bug affecting only some users or with no major impact on the framework
Projects
None yet
Development

No branches or pull requests

2 participants