Skip to content

Correction de bug sur `logo_modifier` dans le cas d'une traduction d'un objet.

Maïeul a demandé de fusionner gh-be488ad8/5007/unknown/refs/pull/5007/head vers 4.0

Message du commit

Lorsque l'on créé une traduction pour un article, on duplique le logo. Problème :

  • en SPIP < 4 : le fichier du logo était effectivement bien dupliqué, car on changeait artonxx.ext en artonyy.ext
  • en SPIP 4 : le fichier du logo n'est pas dupliqué, mais il y a une nouvelle entrée en BDD. Conséquence : si on supprime le logo d'une traduction, cela supprime aussi le logo original, car on supprime le fichier.

Ce commit corrige cela en suffixant le logo de la traduction par article-xx.

Évidemment, tout cela n'aura plus de raison d'être le jour où les logos seront pleinement des documents (avec donc possibilité de les mutualiser).

Compléments

  • ping @rastapopoulos et cf spip-contrib-extensions/duplicator#5
  • Je propose la PR sur spip 4.0, car contrairement à d'autres, je ne bosse pas avec les versions de dev. Mais ce serait a priori à reporter en master
  • en complément de ce commit, je vais essayer d'en proposer un dans medias, pour s'assurer qu'on ne puisse pas avoir deux entrées avec le même chemin de fichier, autant sécurisé aussi en amont

Rapports de requête de fusion