Skip to content

fix(fo): max of dynamic array#1323

Closed
BulotF wants to merge 2 commits into
v2-mainfrom
fix/minmax-dynamic-array
Closed

fix(fo): max of dynamic array#1323
BulotF wants to merge 2 commits into
v2-mainfrom
fix/minmax-dynamic-array

Conversation

@BulotF

@BulotF BulotF commented Aug 22, 2025

Copy link
Copy Markdown

Prise en compte du cas esa-2025-a00 :
Un maximum est indiqué pour les tableaux dynamiques personnalisés, mais il ne faut pas en tenir compte.

Moralité : le maximum indiqué dans le DDI n'est à utiliser que lorsque le tableau dynamique n'a pas de variable externe associée (les 2 tableaux liés aux restructurations l'utilisent bien, eux)

Prise en compte du cas esa-2025-a00 :
Un maximum est indiqué pour les tableaux dynamiques personnalisés, mais il ne faut pas en tenir compte.

Moralité : le maximum indiqué dans le DDI n'est à utiliser que lorsque le tableau dynamique n'a pas de variable externe associée
(les 2 tableaux liés aux restructurations l'utilisent bien, eux)

@nsenave nsenave left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BulotF Possible / pertinent de rajouter de la doc ou un commentaire pour garder la raison d'être de cette condition dans le code ?

@nsenave nsenave changed the title Update models.xsl fix(fo): max of dynamic array Sep 11, 2025
laurentC35 added a commit to InseeFr/Eno-Xml that referenced this pull request Sep 17, 2025
PR InseeFr/Eno#1323

Prise en compte du cas esa-2025-a00 :
Un maximum est indiqué pour les tableaux dynamiques personnalisés, mais il ne faut pas en tenir compte.

Moralité : le maximum indiqué dans le DDI n'est à utiliser que lorsque le tableau dynamique n'a pas de variable externe associée (les 2 tableaux liés aux restructurations l'utilisent bien, eux)

Co-Authored-By: François Bulot <11525050+BulotF@users.noreply.github.com>
@laurentC35

Copy link
Copy Markdown
Contributor

Moved to InseeFr/Eno-Xml#245 (mono-repo)

@laurentC35 laurentC35 closed this Sep 17, 2025
@nsenave nsenave deleted the fix/minmax-dynamic-array branch October 15, 2025 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants