Distinction nom réel / nom de fichier des uploads

[etchegom]

Ajout d'un champ realname sur le modèle UploadedFile. Le champ filename contiendra désormais le nom "sluggifié", le champ realname le nom d'origine.

• Pour les object actuels, on copiera la valeur de filename dans realname dans une migration.
• Pour les nouveaux objets, on complètera les champs au moment du save, afin d'avoir le slug dans filename.

Au niveau de l'UI, l'info du nom de fichier est le plus souvent contenue dans les champs de type text avec du contenu JSON (file_dict). J'ai donc fait deux choses:

• Créer un script pour ajouter l'info realname manquante dans ces champs
• Rendre cette info optionnelles au niveau du template (si realname manquant, on prend filename.

Au moment du download, on replace la valeur de realname dans le nom de l'instance téléchargée.

[github-actions]

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
8320	6920	83%	80%	🟢

New Files

No new covered files...

Modified Files

File	Coverage	Status
conventions/services/convention_generator.py	83%	🟢
conventions/services/utils.py	89%	🟢
conventions/views/conventions.py	63%	🟢
upload/admin.py	100%	🟢
upload/models.py	100%	🟢
upload/services.py	76%	🟢
upload/views.py	62%	🟢
TOTAL	82%	🟢

updated for commit: e85a8f1 by action🐍

[kolok]

@etchegom
où est-ce qu'on joue avec le realname et filename pour uploader les fichiers en utilisant la version slugifié et les downloader avec le nom initial ?

[etchegom]

A sortir de cette migration, on devrait le faire passer dans un shell en oneshot.

[etchegom]

@etchegom où est-ce qu'on joue avec le realname et filename pour uploader les fichiers en utilisant la version slugifié et les downloader avec le nom initial ?

J'ai fait plusieurs changements, notamment:

• fallback sur le filename si le realname est vide, coté template
• prise en compte du realnam au moment du download

[kolok]

Je veux bien qu'on fasse une pair-review pour celle-ci

[etchegom]

On affiche realname si il existe, filename sinon.
Tant qu'on n'a pas passé le script de mise à jour des champs "text & files", ou bien dans le cas où on en aurait oublié, on se prémunit de tout problème de display ici.

[kolok]

Un effet de bord avec le CSRF token à investiguer

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud