[Eno v3] fix page ordering in sequence/question mode

[davdarras]

• page ordering is incorrect in loops
• incorrect paginatedLoop attribute when the loop is paginated
• fix loop ordering in question/sequence mode
• regroup pagination with question regroupments

[AnneHuSKa]

Toujours des soucis sur la génération avec boucles par séquence

[laurentC35]

@AnneHuSKa va trouver des cas de tests

[AnneHuSKa]

@nsenave : je pense que l'issue #772 est un exemple mais je n'arrive pas à trouver un comportement "systématique" ?

[nsenave]

@nsenave : je pense que l'issue #772 est un exemple mais je n'arrive pas à trouver un comportement "systématique" ?

Les issues ont bien l'air liée a priori, je vais regarder ça de près et ajouter une série de tests d'intégration là dessus (-> ça devrait régler les deux issues d'un coup)

[nsenave]

Question sur le caractère "paginé" ou non des boucles (attribut "paginatedLoop" dans Lunatic), quelles sont les valeurs attendues selon les cas ?

	Boucle principale	Boucle liée
Mode QUESTION	false	true
Mode SEQUENCE	false	true

[laurentC35]

• boucle principale en mode pagination par question : on veut pouvoir répondre à cette boucle sur la même page (et min et Max différents) -> false
• boucle principale en mode pagination par séquence : même combat du coup -> false
• boucle liée : on veut répondre question par question -> true
• boucle lié, pagination par séquence, on regroupe toutes les questions d'une même séquence au sein d'une boucle sur la même page, je dirais donc : true

En revanche, j'ai un flou sur un comportement, il me semble que dès qu'on a un attribut itération, la boucle se retrouve toujours paginé.

Je pense qu'il faudrait laisser l'utilisateur, choisir par une case à cocher s'il souhaite que sa boucle soit paginé ou pas.

Il y a des cas par contre, où la pagination par question me semble obligatoire, juste par compréhension de ce qu'il se passe à l'écran 😅

Si @AnneHuSKa ou @romaintailhurat peuvent confirmer ou infirmer 👍

[nsenave]

@laurentC35 l'attribut "iterations" est présent si (et seulement si) on a une boucle liée, donc ça paraît cohérent 👍

J'aime bien l'idée qu'à terme on ait un checkbox ou autre pour définir ça dans pogues

[nsenave]

(pour info) Je suis retourné voir le code Eno v2 :

• En mode question :

<xsl:attribute name="paginatedLoop" select="$isLinkedLoop"/>

i.e. paginée ssi boucle liée source

• En mode séquence :

<xsl:attribute name="paginatedLoop" select="not($parent/@componentType='Sequence')"/>

i.e. paginée ssi définie sur une sous-séquence 🤔 source