bug chargement ftp

pour le chargement ftp, si on a plusieurs sous dossiers dans upload, exemple :

upload |-- ski |-- kayak |-- pédalo

Si on sélectionne : "Tous le dossier ski" alors toutes les images des 3 sous dossiers sont téléchargées

Dans la valeur de l'option, pour les options du type "Tous le dossier ..." il manque un "/" à la fin. cf la fonction pref_files

Le plus simple serait d'ajouter ici : https://git.spip.net/spip/medias/src/branch/master/formulaires/joindre_document.php#L446 le slash, et ca fonctionne.

@tofulm a référencé ce ticket depuis une révision proposition de correction de #4915

Version cible ?

Je trouve tellement bizarre que ce bug ne sorte que maintenant !! c'est en spip 4.1 @cy.altern doit faire un test en windows pour voir ce que ca donne.

@b_b a ajouté le label bug

@b_b a ajouté ce ticket au jalon spip-4.1

Avec un serveur apache sous Windows je confirme que :

• le bogue existe avec SPIP 4.2 et 4.1 / PHP 8.0 ou 8.1.
• le patch de https://git.spip.net/spip/medias/commit/5dcbbede9ece49329ba1785250bb0ec9beec8f44 est OK

@cy.altern a référencé ce ticket depuis une révision fix: au chargement de documents depuis tmp/upload (source "le serveur") ne charger que le sous-dossier demandé (et non pas tous les sous-dossiers)

je ne comprends pas où est passé le commit 5dcbbede9e alors je reprend avec une branche et une PR...

@cy.altern a fait référence à ce ticket depuis !4916 fix: au chargement de documents depuis tmp/upload (source "le serveur") ne charger que le sous-dossier demandé (et non pas tous les sous-dossiers)

je ne comprends pas où est passé le commit 5dcbbede9e alors je reprend avec une branche et une PR...

Ben dans la branche https://git.spip.net/spip/medias/src/branch/issue_4925 comme indiqué dans le commit :)

Et donc il faudrait supprimer la branche de @tofulm puisqu'il ne l'a jamais envoyé en tant que PR.

Et donc il faudrait supprimer la branche de @tofulm puisqu'il ne l'a jamais envoyé en tant que PR.

C'était volontaire @b_b, avant de faire un PR je voulais avoir d'autres avis, je n'étais pas certain que ma correction ne casse pas sur d'autres environnements.

C'était volontaire @b_b, avant de faire un PR je voulais avoir d'autres avis, je n'étais pas certain que ma correction ne casse pas sur d'autres environnements.

Ha ben dans ce cas @cy.altern t'as devancé :p

C'est étrange, cette antériorité, il me semble l'avoir bippé pour qu'il regarde sur sous windows s'il reproduisait. Mais bon ce n'est pas l'essentiel. je vais donc supprimer ma branche issue_4925

C'est étrange, cette antériorité, il me semble l'avoir bippé pour qu'il regarde sur sous windows s'il reproduisait. Mais bon ce n'est pas l'essentiel. je vais donc supprimer ma branche issue_4925

Quand je disais "devancé" c'était à propos de l'envoi de la PR, pas de la branche qui va avec :)

@cy.altern a référencé ce ticket depuis une révision fix: au chargement de documents depuis tmp/upload (source "le serveur") ne charger que le sous-dossier demandé (et non pas tous les sous-dossiers)

@cy.altern a fermé ce ticket

https://discuter.spip.net/t/ajouter-dossiers-images-spip-4-1-9/168601/2

Est-ce qu’il n’eut pas fallu le reporter pour SPIP 4.1 ? (c’est correctif sécu maintenant uniquement), mais je m’étonne que ça n’ait pas été fait sur le coup ?

Ha merci @marcimat je n'arrivais pas à remettre la main sur ce ticket pour le poster dans le fil sur discuter.

@cy.altern devrait pouvoir réparer son oubli en faisant le report, même si c'est hors carde sécu maintenant, au moins ça permettra aux gens qui sont encore en 4.1 de bénéficier de ce correctif qui était bien déclaré pour la 4.1 :p