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

Base.CopyTo() does not work well with list properties #2790

Open
ewoutkramer opened this issue May 23, 2024 · 1 comment
Open

Base.CopyTo() does not work well with list properties #2790

ewoutkramer opened this issue May 23, 2024 · 1 comment
Assignees
Labels

Comments

@ewoutkramer
Copy link
Member

The generated CopyTo() function does not handle lists well. Since the properties always return a list (an empty list if the element is not set), the current test that tests the list against null to see if it needs to be copied is wrong, and should use Any() instead.

@mmsmits
Copy link
Member

mmsmits commented May 29, 2024

You have to change the code generator for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

No branches or pull requests

2 participants