plantage de la fonction upgrade_logo_objet() lors de la mise à jour d'un site vers la version 3.3 #4423

Closed
opened 3 years ago by cy.altern · 6 comments
Owner

la mise à jour de la BDD d'un SPIP 3.2 ayant le plugin spip-logo-svg activé (https://contrib.spip.net/logo-svg) plante avec le message:

MAJ 24351 .
Fatal error:  Uncaught Error: Call to undefined function logo_migrer_en_base() 
ligne 711 maj/svn10000.php

cette fonction est fournie par le fichier ecrire/action/editer_logo.php inclu à la ligne précédente:

include_spip('action/editer_logo');

mais puisque le plugin logo-svg fourni lui-aussi un fichier action/editer_logo.php c'est celui-ci qui est inclu.
Vu qu'il n'a pas la fonction logo_migrer_en_base() cela provoque le plantage.

NB1: Tcharlss fait remarquer que même si le plugin logo-svg n'est pas compatible 3.2, cette surcharge est "jouée" parce que la 3.3 est en statut "dev".
NB2: Marcimat note que ce problème risque de se poser aussi pour 3 autres plugins ayant un fichier editer_logo.php:

  • Roles Documents
  • Logos roles
  • lister_logos

Patch testé OK:
appeler précisément le fichier du core avec la modification suivante ligne 709 de ecrire/maj/svn10000.php

include_spip('ecrire/action/editer_logo');
la mise à jour de la BDD d'un SPIP 3.2 ayant le plugin *spip-logo-svg* activé (https://contrib.spip.net/logo-svg) plante avec le message: <pre> MAJ 24351 . Fatal error: Uncaught Error: Call to undefined function logo_migrer_en_base() ligne 711 maj/svn10000.php </pre> cette fonction est fournie par le fichier *ecrire/action/editer_logo.php* inclu à la ligne précédente: <pre> include_spip('action/editer_logo'); </pre> mais puisque le plugin logo-svg fourni lui-aussi un fichier *action/editer_logo.php* c'est celui-ci qui est inclu. Vu qu'il n'a pas la fonction logo_migrer_en_base() cela provoque le plantage. NB1: Tcharlss fait remarquer que même si le plugin logo-svg n'est pas compatible 3.2, cette surcharge est "jouée" parce que la 3.3 est en statut "dev". NB2: Marcimat note que ce problème risque de se poser aussi pour 3 autres plugins ayant un fichier *editer_logo.php*: - Roles Documents - Logos roles - lister_logos Patch testé OK: appeler précisément le fichier du core avec la modification suivante ligne 709 de *ecrire/maj/svn10000.php* <pre> include_spip('ecrire/action/editer_logo'); </pre>
Poster
Owner

j'aurais bien fait directement un PR plutôt qu'un ticket mais le serveur git est (encore?) en rade...

j'aurais bien fait directement un PR plutôt qu'un ticket mais le serveur git est (encore?) en rade...
Owner

Le serveur git est en maintenance comme indiqué sur la liste (et sur IRC)

Le serveur git est en maintenance comme indiqué sur la liste (et sur IRC)
Poster
Owner

hé oui: désolé, j'ai vu ton message une fois le commentaire posté...

hé oui: désolé, j'ai vu ton message une fois le commentaire posté...
Poster
Owner

le PR pour le patch proposé: SPIP/spip#13

le PR pour le patch proposé: https://git.spip.net/SPIP/spip/pulls/13
Poster
Owner

réglé par 1f5764b35d
Statut changé à Fermé

réglé par https://git.spip.net/SPIP/spip/commit/1f5764b35dce1ce520c942488d78cee6bb34432d **Statut changé à Fermé**
Poster
Owner
There is no content yet.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.