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

Effet de bord des modifs faites sur Entity / IEntity #270

Open
rducom opened this issue Oct 30, 2018 · 6 comments
Open

Effet de bord des modifs faites sur Entity / IEntity #270

rducom opened this issue Oct 30, 2018 · 6 comments

Comments

@rducom
Copy link
Contributor

rducom commented Oct 30, 2018

image

Si pas d'URL setter : KO

@nfaugout-lucca
Copy link
Contributor

Là ce que je comprends c'est que ton candidate contient une propriété Url et que du coup le patcher essaie d'écrire cette valeur, alors que Url est en lecture seule puisque c'est l'entité elle-même qui expose son Url, pas le monde extérieur qui le lui impose.

Si c'est ça alors y'a pas de "BUG", juste une mauvaise utilisation.

@rducom
Copy link
Contributor Author

rducom commented Oct 30, 2018

J'ai ici utilisé IEntityBase<T>, qui n'a plus se setter (introduit dans cette PR #245).

@rducom
Copy link
Contributor Author

rducom commented Oct 30, 2018

Cette problématique sautera avec l'externalisation du champ Url des entités

@Poltuu
Copy link
Contributor

Poltuu commented Oct 31, 2018

le front pousse "url" dans son payload ?
fonctionnelement, c raisonnable de planter si c le cas, mais c bien un breaking change

@rducom
Copy link
Contributor Author

rducom commented Oct 31, 2018

non là on pète sur un simple appel en Get depuis l'exterieur sans paramètres spécifiques

@Poltuu
Copy link
Contributor

Poltuu commented Nov 13, 2018

@rducom on a réglé ce pb non ?

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

No branches or pull requests

3 participants