Spéciale dédicace chankalan : fix suppression/conversion du vieux logo du site. Ça fonctionne, mise à part le rechargement ajax qui ne se fait pas. Éviter une notice également quand on dissocie le document logo du site.

action/dissocier_document_role.php

@@ -148,7 +148,10 @@ function supprimer_lien_document_role($id_document, $objet, $id_objet, $role, $s
 		)
 	);
 
-	if ($check) {
+	if (
+		$check
+		and $objet != 'site_spip' // pas de vérif pour le faux objet utilisé pour le logo du site
+	) {
 		// si demande, on verifie que ses documents vus sont bien lies !
 		$spip_table_objet = table_objet_sql($objet);
 		$table_objet = table_objet($objet);

action/iconifier_logo.php

@@ -32,9 +32,8 @@ function action_iconifier_logo_dist($arg = null) {
 	if (is_null($arg)) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
-		//$arg = $_REQUEST['arg']; // temporaire pour dev
 	}
-	list($objet, $id_objet, $etat) = explode('-', $arg);
+	list($objet, $id_objet, $etat) = explode('/', $arg);
 
 	include_spip('inc/autoriser');
 	if (autoriser('iconifier', $objet, $id_objet)) {

action/supprimer_logo.php

@@ -34,9 +34,8 @@ function action_supprimer_logo_dist($arg = null) {
 	if (is_null($arg)) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
-		//$arg = $_REQUEST['arg']; // temporaire pour dev
 	}
-	list($objet, $id_objet, $etat) = explode('-', $arg);
+	list($objet, $id_objet, $etat) = explode('/', $arg);
 
 	// Suppression
 	include_spip('inc/autoriser');

formulaires/inc-apercu-logo.html

@@ -82,19 +82,19 @@
 			<:roles_documents:bouton_iconifier_logo:>,
 			#URL_ACTION_AUTEUR{
 				iconifier_logo,
-				#ENV{objet}-#ENV{id_objet}-#ENV{logo/6},
+				#ENV{objet}/#ENV{id_objet}/#ENV{logo/6},
 				#SELF
 			},
-			'ajax',
+			'ajax'
 		})]
 		[(#BOUTON_ACTION{
 			<:lien_supprimer:>,
 			#URL_ACTION_AUTEUR{
 				supprimer_logo,
-				#ENV{objet}-#ENV{id_objet}-#ENV{logo/6},
+				#ENV{objet}/#ENV{id_objet}/#ENV{logo/6},
 				#SELF
 			},
-			'ajax supprimer',
+			'ajax supprimer'
 		})]
 	</div>
 	]

paquet.xml

@@ -1,7 +1,7 @@
 <paquet
 	prefix="roles_documents"
 	categorie="edition"
-	version="1.2.21"
+	version="1.2.22"
 	etat="dev"
 	compatibilite="[3.2.0;3.2.*]"
 	logo="prive/themes/spip/images/roles_documents-48.png"