Projet

Général

Profil

Development #22409

cmis: format "natif" de fichier Publik non respecté

Ajouté par Emmanuel Cazenave il y a environ 6 ans. Mis à jour il y a presque 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
09 mars 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Raté ici #21751 : le connecteur attend un foo['name'] au lieu d'un foo['filename'].


Fichiers

Révisions associées

Révision 5f792313 (diff)
Ajouté par Emmanuel Cazenave il y a environ 6 ans

cmis: use 'filename' parameter for wcs compliance (#22409)

Use 'data' keyword for the returned data.

Historique

#1

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

Et je change très légèrement les tests fonctionnels pour pouvoir les tourner sur un serveur CMIS apache-chemistry.

#2

Mis à jour par Thomas Noël il y a environ 6 ans

Pour être propre, ça serait mieux d'avoir deux patches, un qui corrige effectivement le name->filename et l'autre qui accorde l'affaire avec CMIS apache-chemistry.

Aussi, dans le patch qui transforme name en filename, rendre également la réponse un peu plus "publik-compliant" en renvoyant un {"data": ...}, qui formera un couple {"err": 0/1, "data": ...} dans la réponse du endpoint. C'est à dire :

-    return {'properties': doc.properties}
+    return {'data':{'properties': doc.properties}}

#4

Mis à jour par Thomas Noël il y a environ 6 ans

Ack

#5

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

  • Statut changé de Nouveau à Résolu (à déployer)
commit 5f792313c1f1c766e031b4da7a5f911162ef1739
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Tue Mar 13 12:00:06 2018 +0100

    cmis: use 'filename' parameter for wcs compliance (#22409)

    Use 'data' keyword for the returned data.
commit 5c4842e9a110ac0976fb19c91a2c5a1bd85876fe
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>  
Date:   Tue Mar 13 12:01:52 2018 +0100                                 

    cmis: change functionnal tests for apache-chemistry server compliance

#6

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#7

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Statut changé de Solution déployée à Fermé

Formats disponibles : Atom PDF