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

JsonWriter - cas limite #37

Open
RaphaelFOURQUET opened this issue Mar 9, 2013 · 8 comments
Open

JsonWriter - cas limite #37

RaphaelFOURQUET opened this issue Mar 9, 2013 · 8 comments

Comments

@RaphaelFOURQUET
Copy link
Collaborator

@FlorentMouysset @nawalouldamer @JackDanny @medi24 @smailGithub
Un petit bug peut parvenir dans ma classe lorsque deux composantes d'une array ne se suivent pas.

explication :
si on a, a un niveau donné une balise < question > alors toutes les autres balises < question > de ce niveau doivent être à la suite, non séparé par une autre comme :
< question > .... ...< /question >
< autre balise de même niveau > ... </ .. >
< question > ..... < /question >
Je pense que ce cas ne peut pas arriver, donc pour le moment je laisse tel quel, si vous avez des exemples où ce cas se produit donnez les moi car ce serait vraiment dur à gérer.

De plus autre question,en MoodleXML une balise quiz ne peut contenir QUE des balises questions? sinon cela remet en cause le parseur de départ de florent étant donné qu'il me transmet seulement la liste des questions pour la suite du traitement.

Ces deux cas sont les seuls bugs possibles restants dans mon code à ma connaissance.

@ghost ghost assigned RaphaelFOURQUET Mar 9, 2013
@FlorentMouysset
Copy link
Collaborator

Je confirme l’existence du bug au niveau du parser XML qui ne peut que retourner une liste de Moodle XML.
Néanmoins une correction est toujours possible.

@JackDanny
Copy link
Owner

tu veux dire une liste de questions Moodle XML?

@FlorentMouysset
Copy link
Collaborator

Le bug est avéré. @RaphaelFOURQUET je demande un correctif. Merci
Bon courage

@RaphaelFOURQUET
Copy link
Collaborator Author

De plus maintenant, il m'affiche quelquefois des \n comme du texte sans les traduire en saut de ligne, je regarde tout ça si j'ai le temps.

RaphaelFOURQUET added a commit that referenced this issue Mar 11, 2013
@RaphaelFOURQUET
Copy link
Collaborator Author

Cas limite restant : actuellement je ne sais pas traduire en json quelquechose du type :
< balise attribut="name" >texte< /balise > avec pérsence d'attribut et de texte directement sans sous balises.

Car il faut associer le texte à un nom en Json.. :
balise : {
"attribut" : "name",
???? : "text"
}
J'espère que ce cas n'arrive pas, mais je n'ai croisé aucune règle qui l'interdit.
La solution la plus efficace serait de traduire si j'ai le temps par:
balise : {
"attribut" : "name",
"text" : null
}
Comme pour un autre cas que florent m'a aidé à résoudre.

Sinon la présence dans mon resultat de caractère \n non interprétée est également génante :
"text": "\nUne application non distribuée peut implémentée MVC.\n "

@RaphaelFOURQUET
Copy link
Collaborator Author

La présence de \n non souhaitée sera réglée lors de mon prochain push.

RaphaelFOURQUET added a commit that referenced this issue Mar 12, 2013
@RaphaelFOURQUET
Copy link
Collaborator Author

Il me reste à régler le dernier soucis et à faire du refactoring.

@FlorentMouysset
Copy link
Collaborator

Il est avéré que l'exemple moodle contient des erreurs.
Il est donc nécessaire de les corriger.

Détail :
Il n'y a pas de double balise "shuffleanswers".

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

No branches or pull requests

3 participants