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

FR : Update & GetById( ) #347

Open
nfaugout-lucca opened this issue Aug 23, 2019 · 0 comments
Open

FR : Update & GetById( ) #347

nfaugout-lucca opened this issue Aug 23, 2019 · 0 comments
Assignees
Labels

Comments

@nfaugout-lucca
Copy link
Contributor

Quand on fait un Update, comme on ne refait pas de GetById après le commit, les includes ne sont pas joués sur des objets référencés qui ont changé au cours du PUT.

Par ex sur un user d'une LE 1, si on fait un PUT pour changer sa LE par 2, alors l'include initial aura bien chargé sa LE 1, mais la LE 2 ne sera pas chargée, donc dans la réponse de l'objet User, la LE sera NULL (alors que LEid sera bien égal à 2).

Revenir à un comportement où on refait un GetById( ) post commit me semble une bonne approche, car outre le fait qu'elle règle ce pb d'includes, elle permet surtout d'avoir avec plus de certitude l'état de l'objet tel qu'il est enregistré en base. Si par ex une chaîne est tronquée silencieusement, là on le verra, pas avec l'approche actuelle.

Il suffit de modifier l'AppController pour lui faire faire un GetById( ) après le SaveChanges( )

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

No branches or pull requests

3 participants