Project

General

Profile

Anomalie #4512

Mise à jour de SPIP 2.1 à SPIP 3.3 : migration des logos inéfective

Added by RealET 🔸 8 months ago. Updated 6 days ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
06/25/2020
Due date:
% Done:

0%

Resolution:
fixed
Navigateur:

Description

Bonjour,

À force de tracer le problème, j'ai fini par trouver que l'extension du fichier placé dans IMG/logos/ est perdue pour tester sa présence.
Ce qui perd l'extension est ici : https://git.spip.net/spip/medias/src/branch/master/action/ajouter_documents.php#L377

Pourquoi ?
Parce que c'est https://git.spip.net/spip/medias/src/branch/master/medias_administrations.php#L134 qui change le nom du champ de media (SPIP 2.1) à media_defaut

Sur suggestion de marcimat sur IRC, j'ai testé en déplaçant le code de mise à jour des logos dans l'upgrade de Medias
Ça marche nickel.

Ayant testé ça avec une installation SVN, voici les diffs au format git patch (ToirtoiseSVN m'a proposé ce format).

svn10000.php.patch View - Suppression de l'upgrade des logos dans le core (817 Bytes) RealET 🔸, 06/25/2020 07:54 PM

medias.patch View - Upgrade des logos dans Medias (1.41 KB) RealET 🔸, 06/25/2020 07:54 PM

History

#1 Updated by RealET 🔸 8 months ago

Si besoin, voici la trace :
#0 deplacer_fichier_upload(../IMG/logo/arton4786.png, ../IMG/logo/arton4786.) 
called at [ecrire\inc\documents.php:158] 

#1 copier_document(, arton4786.png, ../IMG/logo/arton4786.png, logo) 
called at [plugins-dist\medias\action\ajouter_documents.php:436] 

#2 fixer_fichier_upload(Array ([tmp_name] => ../IMG/logo/arton4786.png,[name] => arton4786.png,[mode] => logoon), logoon) called at [plugins-dist\medias\action\ajouter_documents.php:161] 

#3 action_ajouter_un_document_dist(new, Array ([tmp_name] => ../IMG/logo/arton4786.png,[name] => arton4786.png,[mode] => logoon), article, 4786, logoon) 
called at [plugins-dist\medias\action\ajouter_documents.php:54] 

#4 action_ajouter_documents_dist(new, Array ([0] => Array ([tmp_name] => ../IMG/logo/arton4786.png,[name] => arton4786.png,[mode] => logoon)), article, 4786, logoon) 
called at [ecrire\action\editer_logo.php:134] 

#5 logo_modifier(article, 4786, on, Array ([tmp_name] => ../IMG/logo/arton4786.png,[name] => arton4786.png,[mode] => logoon)) called at [ecrire\action\editer_logo.php:185] 

#6 logo_migrer_en_base(article, 1593099288) called at [ecrire\maj\svn10000.php:710]

#2 Updated by RealET 🔸 about 1 month ago

Bonjour,

Qu'est-ce qui empêche l'intégration de ce patch (mis à part que les numéros de version base ont changés) ?

#3 Updated by cedric - 6 days ago

  • Status changed from Nouveau to Fermé
  • Resolution set to fixed

intégré par https://git.spip.net/spip/medias/commit/477e97d2df5f4e128e769e470d261398edd6b816 et https://git.spip.net/spip/spip/commit/3ffd9169d30bce276e3e6d003c4d2f863bbecbc3
ce qui en plus a le bon gout de rejouer la migration vu qu'il y avait un bug sur les logos de site qui continaient à se mettre à la racine IMG/
Comme ça on va les remigrer

Also available in: Atom PDF