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

Le POST et PUT en Turtle ne fonctionne plus #995

Closed
srosset81 opened this issue Jun 27, 2022 · 3 comments
Closed

Le POST et PUT en Turtle ne fonctionne plus #995

srosset81 opened this issue Jun 27, 2022 · 3 comments
Labels
bug Something isn't working feature ldp

Comments

@srosset81
Copy link
Contributor

C'est pourtant nécessaire si on veut être compatible LDP.

Voir discussions #987 (comment)

Il faudra sans doute revoir le code fait par Niko afin d'utiliser à nouveau resource directement (sans resource.body)

Ajouter des tests pour éviter des régressions.

@srosset81 srosset81 added bug Something isn't working ldp labels Jun 27, 2022
@nikoPLP
Copy link
Contributor

nikoPLP commented Jun 27, 2022

Il n'y a plus rien dans le code qui pourrait faire amrcher du Turtle dans les POST et PUT.
je ne sais pas a quel moment le code a disparu.
Quand vous aurez décider de quoi faire, en effet il faudra harmoniser ldp.resource.create et peut etre aussi le PATCh sur container, qui utilisent tous les deux le resource.body

@srosset81
Copy link
Contributor Author

srosset81 commented Jun 27, 2022

Il n'y a plus rien dans le code qui pourrait faire amrcher du Turtle dans les POST et PUT.

Faut pas dramatiser non plus 😄 Le problème est apparu sans doute quand j'ai séparé ldp.container.post et ldp.resource.create. Je crois avoir refactoré les actions put et patch à ce moment, en assumant avoir un objet. Je pense pas que c'est un gros travail de revenir à ce qui marchait avant.

@srosset81
Copy link
Contributor Author

Closed in favor of #1226

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature ldp
Projects
None yet
Development

No branches or pull requests

2 participants