Remplacer les caractères spéciaux dans les noms des fichiers envoyés

Fix #119 (closed)

ping @lebardix

@maieul a ajouté 2 révisions : be0bbcbf b7732a80

@rastapopoulos a évalué le commit b7732a80 :

diff --git a/inc/formidable_fichiers.php b/inc/formidable_fichiers.php
--- a/inc/formidable_fichiers.php
+++ b/inc/formidable_fichiers.php
@@ -579,0 +583,4 @@
+ * @param string $nom
+ * @return string $nom
+**/
+function formidable_standardiser_nom_fichier(string $nom): string {

Est-ce que SPIP n'aurait pas déjà une fonction de ce genre qui s'assure d'avoir un nom de fichier "passe partout", plutôt que refaire notre propre tambouille en plugin à maintenir ?

@rastapopoulos a évalué cette PR (b7732a80)

@maieul a évalué le commit b7732a80 :

diff --git a/inc/formidable_fichiers.php b/inc/formidable_fichiers.php
--- a/inc/formidable_fichiers.php
+++ b/inc/formidable_fichiers.php
@@ -579,0 +583,4 @@
+ * @param string $nom
+ * @return string $nom
+**/
+function formidable_standardiser_nom_fichier(string $nom): string {

Et bien j'ai cherché et je n'ai pas trouvé. Le code que j'ai mis est honteusement inspiré de https://git.spip.net/spip/spip/src/branch/master/ecrire/inc/documents.php#L146

@rastapopoulos a évalué le commit b7732a80 :

diff --git a/inc/formidable_fichiers.php b/inc/formidable_fichiers.php
--- a/inc/formidable_fichiers.php
+++ b/inc/formidable_fichiers.php
@@ -579,0 +583,4 @@
+ * @param string $nom
+ * @return string $nom
+**/
+function formidable_standardiser_nom_fichier(string $nom): string {

Ah ouais c'est une fonction presque générique, sauf qu'elle impose le dossier final d'arrivée… un peu couillon dommage.

@maieul a évalué le commit b7732a80 :

diff --git a/inc/formidable_fichiers.php b/inc/formidable_fichiers.php
--- a/inc/formidable_fichiers.php
+++ b/inc/formidable_fichiers.php
@@ -579,0 +583,4 @@
+ * @param string $nom
+ * @return string $nom
+**/
+function formidable_standardiser_nom_fichier(string $nom): string {

Du coup on valide tel quel et tant pis on verra peut être dans le futur à brancher formidable sur cette fonction generique ?

@rastapopoulos a évalué le commit b7732a80 :

diff --git a/inc/formidable_fichiers.php b/inc/formidable_fichiers.php
--- a/inc/formidable_fichiers.php
+++ b/inc/formidable_fichiers.php
@@ -579,0 +583,4 @@
+ * @param string $nom
+ * @return string $nom
+**/
+function formidable_standardiser_nom_fichier(string $nom): string {

Ben ouais du coup, pas d'autre solution pour l'instant.

@maieul a fait référence à cette PR depuis un commentaire de #119 Probleme avec les images (jpg,png,?) comportant dans leur nom un apostrophe - ' -

@maieul a fermé cette PR

intégré en 2c353b08