Project

General

Profile

Tâche #30194

Scénario #30166: Faire le nécessaire pour que Zéphir 2.8.0 supporte EOLE 2.6.0

Gérer le changement CJSON / JSON sur Zéphir 2.8.0

Added by Gilles Grandgérard about 1 month ago. Updated 25 days ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Start date:
05/29/2020
Due date:
% Done:

100%

Remaining (hours):
0.0

Description

Nous avons identifié que le comportement des librairies CJSON et JSON ne sont pas identiques entre python2.7 et python 3.

Si un caractère '\' apparait dans une variable du fichier Config.eol, il sera chargé par Cjson, mais générera une erreur avec la lib JSon python3.

Sur Zéphir 2.8, il faut :
- s'assurer que l'enregistrement (sur disque) du fichier config.eol soit celui du module (quelque soit la version )
- Au moment du chargement en mémoire, un pré chargement doit être fait, et si version est inférieur à 2.8, patcher le contenu avant de le charger avec 'json.load'
- Au moment de l'écriture, si version est inférieur à 2.8, alors patcher (à l'envers) le contenu avant de l'écrire.

Dans un 1er temps, ne gérer que le caractère '\'.

Référence : https://www.ecma-international.org/publications/files/ECMA-ST/ECMA-404.pdf

Associated revisions

Revision 489c2efa (diff)
Added by Emmanuel GARETTE about 1 month ago

converti le fichier format 'cjson' en format 'json' et inversement (ref #30194)

Revision c4885427 (diff)
Added by Emmanuel GARETTE about 1 month ago

remettre le fichier au format 'cjson' (ref #30194)

Revision 15637509 (diff)
Added by Emmanuel GARETTE about 1 month ago

proposer une fonction générique pour charger un fichier json (ref #30194)

Revision 900213a7 (diff)
Added by Emmanuel GARETTE about 1 month ago

charger les fichiers json via creole (ref #30194)

Revision c84d9215 (diff)
Added by Emmanuel GARETTE 25 days ago

séparation de la fonction json pour utiliser dans les tests (ref #30194)

History

#1 Updated by Gilles Grandgérard about 1 month ago

  • Description updated (diff)

#2 Updated by Emmanuel GARETTE about 1 month ago

  • Status changed from Nouveau to En cours

#3 Updated by Emmanuel GARETTE about 1 month ago

  • Assigned To set to Emmanuel GARETTE

#4 Updated by Emmanuel GARETTE about 1 month ago

  • Status changed from En cours to Résolu

#5 Updated by Emmanuel GARETTE 26 days ago

  • % Done changed from 0 to 100

#6 Updated by Joël Cuissinat 25 days ago

  • Status changed from Résolu to Fermé
  • Remaining (hours) set to 0.0

Ça y'est le test Jenkins ne signale plus d'erreur : https://dev-eole.ac-dijon.fr/jenkins/job/2.8.0/job/test-importation-zephir-2.8.0-amd64/ (job n°59).

Also available in: Atom PDF