search

assemblee-virtuelle / archipelago

Fostering interconnections between communities by creating synergies between their platforms
Apache License 2.0
14 stars 6 forks source link

Le nouveau DereferenceMixin ne fonctionne pas #173

Closed srosset81 closed 3 months ago

srosset81 commented 4 months ago

Suite au déploiement de https://github.com/assemblee-virtuelle/archipelago/pull/159, lorsque j'essaie de GET un utilisateur, j'ai cette erreur:

Capture d’écran du 2024-03-26 12-02-50

On voit que l'erreur est générée par le DereferenceMixin, qui doit appeler ldp.resource.get avec un resourceUri vide.

En enlevant le DerefenceMixin des containers /users et /organizations, ça marche à nouveau.

mguihal commented 4 months ago

Je me permets de demander pourquoi la PR #159 a été mergée si celle-ci n'était pas fonctionnelle ? N'y-a-t'il pas eu des tests fait avant le merge ?

Là on se retrouve dans une situation inconfortable où une fonctionnalité est KO sur master. Au lieu de créer un ticket de bug a-posteriori, j'aurais plutôt tendance à questionner si on doit revert les derniers changements en attendant une version fonctionnelle...

simonLouvet commented 4 months ago

Le bug venait bien d'un cas nominal non traité (pas de relations réifiées sur le prédicat). Les tests automatisés sont sur semapps et pas sur archipelago. Ca pourrait être pertinent d'en ajouter, mais personne n'a la bande passante à ma connaissance

srosset81 commented 4 months ago

Le bug venait bien d'un cas nominal non traité (pas de relations réifiées sur le prédicat). Les tests automatisés sont sur semapps et pas sur archipelago. Ca pourrait être pertinent d'en ajouter, mais personne n'a la bande passante à ma connaissance

Il aurait quand même fallu tester au moins avec les données de l'AV.