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

Gegenereerde .NET code veroorzaakt compiler warning door NaamPersoon/Naam constructie #850

Open
MelvLee opened this issue Nov 24, 2021 · 4 comments
Labels
enhancement New feature or request v2.0

Comments

@MelvLee
Copy link
Collaborator

MelvLee commented Nov 24, 2021

Deze issue is gemeld door @KayodeBakker

Overerving van de Naam component door de NaamPersoon component veroorzaakt in .NET gegenereerde code de volgende compiler warning: 'NaamPersoon.InOnderzoek' hides inherited member 'Naam.InOnderzoek'.

In standaard situaties wil je bij overerving dat de kind component (in dit geval NaamPersoon) alle properties van het ouder component (in dit geval Naam) krijgt. Als aan het kind component een property (in dit geval InOnderzoek) wordt toegevoegd met dezelfde naam als een parent property, dan is parent property niet meer te bevragen. In dit specifiek geval is dat geen issue omdat het type (NaamPersoonInOnderzoek) van de NaamPersoon.InOnderzoek het type (NaamInOnderzoek) van Naam.InOnderzoek overerft, waardoor er geen informatie verloren gaat.

Een nette oplossing hiervoor is om een NaamBase component te introduceren zonder de InOnderzoek property. Deze kan dan door zowel Naam als door NaamPersoon wordt overerft met hun eigen InOnderzoek property

@MelvLee MelvLee added the enhancement New feature or request label Nov 24, 2021
@JohanBoer JohanBoer added this to Project Backlog in Scrum Board BRP bevragen Nov 29, 2021
@JohanBoer JohanBoer moved this from Project Backlog to In progress in Scrum Board BRP bevragen Nov 29, 2021
@fsamwel fsamwel added the v2.0 label Dec 2, 2021
@KayodeBakker
Copy link
Collaborator

@fsamwel @MelvLee
ik zie nu dat er een v2.0 tag erbij is gekomen. Ik neem aan dat de wijziging van Melvin dat naar develop is gegaan, ook naar master komt als onderdeel van v1.3.0?

@MelvLee
Copy link
Collaborator Author

MelvLee commented Dec 6, 2021

@fsamwel @MelvLee ik zie nu dat er een v2.0 tag erbij is gekomen. Ik neem aan dat de wijziging van Melvin dat naar develop is gegaan, ook naar master komt als onderdeel van v1.3.0?

Ja, ik wacht tot pull request #859 wordt ge-approved. Dan kan ik van de wijzigingen die hiermee te maken hebben een p.r. maken voor de master

@JohanBoer
Copy link
Collaborator

Volgens mij is dit issue zowel in de master als in develop opgelost

@JohanBoer JohanBoer moved this from In progress to Review in Scrum Board BRP bevragen Jan 31, 2022
@KayodeBakker
Copy link
Collaborator

Eens

@JohanBoer JohanBoer moved this from Review to Done in Scrum Board BRP bevragen Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v2.0
Projects
Development

No branches or pull requests

4 participants