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

Longueur des noms de fichier à l'exportation #358

Closed
4 of 5 tasks
EmmanuelDemey opened this issue Jun 23, 2023 · 13 comments · Fixed by #514
Closed
4 of 5 tasks

Longueur des noms de fichier à l'exportation #358

EmmanuelDemey opened this issue Jun 23, 2023 · 13 comments · Fixed by #514
Assignees

Comments

@EmmanuelDemey
Copy link
Collaborator

EmmanuelDemey commented Jun 23, 2023

Les limites de nombre de caractères dans les noms de fichier et dans la longueur du chemin pour accéder à un fichier font que certains fichiers d’exportation ne peuvent pas être ouverts (notamment lorsqu’il s’agit de zip).

Il faudrait limiter la longueur des noms de fichiers résultant d’exportations à 200 caractères.

  •  Un concept
  • Une liste concept
  • Une collection
  • Une liste collection
  • Un Rapport avec ses documents
@EmmanuelDemey
Copy link
Collaborator Author

Vérifier si les caractères interdits sont gérés (faire attention au support cross Système d'exploitation)

@Emilia-Vittek
Copy link

Pas de problème à l'ouverture du fichier compressé avec 7-zip. En revanche l'ouverture avec l'explorateur de Windows pose problème, avec les messages suivants :

  • si le nom de fichier est long (test avec un nom de 123 caractères, espaces inclus), le message est : " Windows ne peut pas effectuer l'extraction. Le chemin de destination est trop long. Renommez le fichier compressé et réessayez. "
  • si le nom de fichier est court (test avec un nom de 25 caractères, espaces inclus), le message est : " Windows ne peut pas effectuer l'extraction. Impossible de créer le fichier de destination.".

En regardant de plus près, il se trouve que le nom de fichier inclut le caractère ":" (deux points) que l'explorateur Windows ne sait pas gérer dans les noms de fichiers et dossiers.

Cela laisse penser que c'est un "mauvais" message d'erreur qui nous a poussés vers une demande non adaptée, mais non inutile pour autant. Toutefois, il faut pouvoir bannir des noms de fichiers générés les caractères non supportés par les OS tels que Windows, Mac OS et Linux.
Pour Windows ces caractères sont \ / : * ? " < > |
Quels sont-ils pour Mac OS et Linux ?

@EmmanuelDemey
Copy link
Collaborator Author

@Emilia-Vittek pour le premier point quel limite je met en nom de fichier ? Par ce que cela depend de ou tu met le fichier dans ton arborescence. L'endroit ou tu mets le fichier compte également dans le calcul de la taille de la destination.

@Emilia-Vittek
Copy link

@EmmanuelDemey sur la longueur, elle dépend effectivement de l'arborescence, mais sur Windows 11 il y a de la marge. Nous demandons toujours une limite à 200 caractères, stp.

@EmmanuelDemey
Copy link
Collaborator Author

@Emilia-Vittek je viens de refaire une PR en supprimant les caractères accentués et les caractères de ponctuation.

@Emilia-Vittek
Copy link

@EmmanuelDemey Le nom est bien ramené à 200 caractères, mais comme c'est la fin qui est tronquée, cela supprime l'extension, ce qui ne permet pas d'ouvrir le fichier directement.
Qu'est-il possible de faire, stp ? Merci.

@Emilia-Vittek
Copy link

@EmmanuelDemey La nuit portant conseil, elle m'a poussée à faire de nouveaux tests aujourd'hui. Voici un récapitulatif de ce que j'ai constaté.

Les noms sont toujours ramenés à 200 caractères mais :

  • concept : c'est Ok ;
  • liste de concepts : Ok ;
  • collection : il y a un problème à l'exportation en odt seconde langue (retour code erreur 500 et le fichier n'est pas créé) ;
  • liste de collections : il y a un problème à l'exportation en odt seconde langue (retour code erreur 500 et le fichier n'est pas créé) ;
  • rapport avec ses documents : le rapport est exporté et le nom trop long du document est bien tronqué, mais cela supprime l'extension ce qui fait qu'on ne peut pas ouvrir le fichier directement (cf. première remarque, message précédent).

@Emilia-Vittek
Copy link

@EmmanuelDemey En fait, mercredi dernier (i. e. quand j'ai refait les tests) il y a eu des problèmes de saturation de l'espace. J'ai donc recommencé ce matin (vendredi 22/09/2023). Au final, pour la longueur des noms de fichier à l'exportation, les noms sont toujours ramenés à 200 caractères mais :

  • **concept :** c'est Ok ;
    
  • **liste de concepts** : Ok ;
    
  • **collection :** c'est Ok ;
    
  • **liste de collections** : c'est Ok ;
    
  • **rapport avec ses documents** : le rapport est exporté et le nom trop long du document est bien tronqué, **mais cela supprime l'extension** ce qui fait qu'on ne peut pas ouvrir le fichier directement (cf. première remarque, premier message de recette).
    

J'ai par ailleurs détecté un problème dans l'exportation des collections et des listes de collection : l'exportation en seconde langue ne se fait pas quand le label en seconde langue n'est pas renseigné et ce n'est pas un champ obligatoire. Je crée une carte "à creuser" car nous devons d'abord instruire le sujet côté métier.

Toutes mes excuses pour les revirements et un grand merci.

@EmmanuelDemey
Copy link
Collaborator Author

@Emilia-Vittek je viens de faire le fix pour le document.
Peux tu créer un nouveau ticket pour le bug que tu as détecté. J ele prendrai en compte au prochain sprint.

@Emilia-Vittek
Copy link

@EmmanuelDemey
Merci pour la correction. Je teste dès que cela passe en recette.
La carte est créée pour un prochain sprint.

@Emilia-Vittek
Copy link

@EmmanuelDemey
Je viens de "boucler" cette recette et il me reste encore un problème :

  • Ok pour "concept", "liste de concepts", "collection" et "liste de collections" ;
  • il reste un problème pour l'exportation du rapport qualité : sur une série dont le nom comportait 229 caractères, le nom du fichier du rapport comporte 224 caractères qui se décomposent en 16 caractères poutr la mention "Rapport qualité ", 204 carcatères par "troncature du nom de la série" et 4 caractères pour l'extension précédée d'un point (".odt").

Désolée.

@EmmanuelDemey
Copy link
Collaborator Author

@Emilia-Vittek Je viens de reproduire le problème.
Par contre je préfère créer un nouveau ticket afin de nous faciliter la vie.
Je bouge ce ticket dans Pret pour la Prod. Et je vais en créer un nouveau pour le bug que tu as détecté.

@EmmanuelDemey
Copy link
Collaborator Author

Voici la nouvelle issue : InseeFr/Bauhaus#503

@EmmanuelDemey EmmanuelDemey linked a pull request Feb 15, 2024 that will close this issue
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 a pull request may close this issue.

3 participants