spip / SPIP

ecrire/action/tester.php

Indentation +86 added lines, -86 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -29,100 +29,100 @@ 
  * redirige sur l'image ainsi créée (sinon sur une image d'echec).
  **/
 function action_tester_dist() {
-	$arg = _request('arg');
+    $arg = _request('arg');
 
-	// verifier les formats acceptes par GD
-	if ($arg === 'gd2') {
-		$gd_formats = [];
+    // verifier les formats acceptes par GD
+    if ($arg === 'gd2') {
+        $gd_formats = [];
 
-		if (function_exists('imagetypes')) {
-			if (imagetypes() & IMG_GIF) {
-				$gd_formats[] = 'gif';
-			}
-			if (imagetypes() & IMG_JPG) {
-				$gd_formats[] = 'jpg';
-			}
-			if (imagetypes() & IMG_PNG) {
-				$gd_formats[] = 'png';
-			}
-			if (imagetypes() & IMG_WEBP) {
-				$gd_formats[] = 'webp';
-			}
-		}
+        if (function_exists('imagetypes')) {
+            if (imagetypes() & IMG_GIF) {
+                $gd_formats[] = 'gif';
+            }
+            if (imagetypes() & IMG_JPG) {
+                $gd_formats[] = 'jpg';
+            }
+            if (imagetypes() & IMG_PNG) {
+                $gd_formats[] = 'png';
+            }
+            if (imagetypes() & IMG_WEBP) {
+                $gd_formats[] = 'webp';
+            }
+        }
 
-		$gd_formats = implode(',', $gd_formats);
-		ecrire_meta('gd_formats_read', $gd_formats);
-		ecrire_meta('gd_formats', $gd_formats);
-	} elseif ($arg == 'netpbm') {
-		// verifier les formats netpbm
-		if (!defined('_PNMSCALE_COMMAND')) {
-			define('_PNMSCALE_COMMAND', 'pnmscale');
-		} // Securite : mes_options.php peut preciser le chemin absolu
-		if (_PNMSCALE_COMMAND == '') {
-			return;
-		}
-		$netpbm_formats = [];
+        $gd_formats = implode(',', $gd_formats);
+        ecrire_meta('gd_formats_read', $gd_formats);
+        ecrire_meta('gd_formats', $gd_formats);
+    } elseif ($arg == 'netpbm') {
+        // verifier les formats netpbm
+        if (!defined('_PNMSCALE_COMMAND')) {
+            define('_PNMSCALE_COMMAND', 'pnmscale');
+        } // Securite : mes_options.php peut preciser le chemin absolu
+        if (_PNMSCALE_COMMAND == '') {
+            return;
+        }
+        $netpbm_formats = [];
 
-		$jpegtopnm_command = str_replace(
-			'pnmscale',
-			'jpegtopnm',
-			(string) _PNMSCALE_COMMAND
-		);
-		$pnmtojpeg_command = str_replace(
-			'pnmscale',
-			'pnmtojpeg',
-			(string) _PNMSCALE_COMMAND
-		);
+        $jpegtopnm_command = str_replace(
+            'pnmscale',
+            'jpegtopnm',
+            (string) _PNMSCALE_COMMAND
+        );
+        $pnmtojpeg_command = str_replace(
+            'pnmscale',
+            'pnmtojpeg',
+            (string) _PNMSCALE_COMMAND
+        );
 
-		$vignette = _ROOT_IMG_PACK . 'test.jpg';
-		$dest = _DIR_VAR . 'test-jpg.jpg';
-		$commande = "$jpegtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
-		spip_logger()->info($commande);
-		exec($commande);
-		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
-			$netpbm_formats[] = 'jpg';
-		}
-		$giftopnm_command = str_replace('pnmscale', 'giftopnm', (string) _PNMSCALE_COMMAND);
-		$pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', (string) _PNMSCALE_COMMAND);
-		$vignette = _ROOT_IMG_PACK . 'test.gif';
-		$dest = _DIR_VAR . 'test-gif.jpg';
-		$commande = "$giftopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
-		spip_logger()->info($commande);
-		exec($commande);
-		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
-			$netpbm_formats[] = 'gif';
-		}
+        $vignette = _ROOT_IMG_PACK . 'test.jpg';
+        $dest = _DIR_VAR . 'test-jpg.jpg';
+        $commande = "$jpegtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+        spip_logger()->info($commande);
+        exec($commande);
+        if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
+            $netpbm_formats[] = 'jpg';
+        }
+        $giftopnm_command = str_replace('pnmscale', 'giftopnm', (string) _PNMSCALE_COMMAND);
+        $pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', (string) _PNMSCALE_COMMAND);
+        $vignette = _ROOT_IMG_PACK . 'test.gif';
+        $dest = _DIR_VAR . 'test-gif.jpg';
+        $commande = "$giftopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+        spip_logger()->info($commande);
+        exec($commande);
+        if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
+            $netpbm_formats[] = 'gif';
+        }
 
-		$pngtopnm_command = str_replace('pnmscale', 'pngtopnm', (string) _PNMSCALE_COMMAND);
-		$vignette = _ROOT_IMG_PACK . 'test.png';
-		$dest = _DIR_VAR . 'test-gif.jpg';
-		$commande = "$pngtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
-		spip_logger()->info($commande);
-		exec($commande);
-		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
-			$netpbm_formats[] = 'png';
-		}
+        $pngtopnm_command = str_replace('pnmscale', 'pngtopnm', (string) _PNMSCALE_COMMAND);
+        $vignette = _ROOT_IMG_PACK . 'test.png';
+        $dest = _DIR_VAR . 'test-gif.jpg';
+        $commande = "$pngtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+        spip_logger()->info($commande);
+        exec($commande);
+        if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
+            $netpbm_formats[] = 'png';
+        }
 
-		ecrire_meta('netpbm_formats', implode(',', $netpbm_formats ?: []));
-	}
+        ecrire_meta('netpbm_formats', implode(',', $netpbm_formats ?: []));
+    }
 
-	// et maintenant envoyer la vignette de tests
-	if (in_array($arg, ['gd2', 'imagick', 'convert', 'netpbm'])) {
-		include_spip('inc/filtres');
-		include_spip('inc/filtres_images_mini');
-		$taille_preview = 150;
-		$image = _image_valeurs_trans(_DIR_IMG_PACK . 'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
+    // et maintenant envoyer la vignette de tests
+    if (in_array($arg, ['gd2', 'imagick', 'convert', 'netpbm'])) {
+        include_spip('inc/filtres');
+        include_spip('inc/filtres_images_mini');
+        $taille_preview = 150;
+        $image = _image_valeurs_trans(_DIR_IMG_PACK . 'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
 
-		$image['fichier_dest'] = _DIR_VAR . "test_$arg";
+        $image['fichier_dest'] = _DIR_VAR . "test_$arg";
 
-		if (
-			($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true))
-			&& $preview['width'] * $preview['height'] > 0
-		) {
-			redirige_par_entete($preview['fichier']);
-		}
-	}
+        if (
+            ($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true))
+            && $preview['width'] * $preview['height'] > 0
+        ) {
+            redirige_par_entete($preview['fichier']);
+        }
+    }
 
-	# image echec
-	redirige_par_entete(chemin_image('echec-reducteur-xx.svg'));
+    # image echec
+    redirige_par_entete(chemin_image('echec-reducteur-xx.svg'));
 }

Spacing +11 added lines, -11 removed lines

@@ -74,9 +74,9 @@ 
 			(string) _PNMSCALE_COMMAND
 		);
 
-		$vignette = _ROOT_IMG_PACK . 'test.jpg';
-		$dest = _DIR_VAR . 'test-jpg.jpg';
-		$commande = "$jpegtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+		$vignette = _ROOT_IMG_PACK.'test.jpg';
+		$dest = _DIR_VAR.'test-jpg.jpg';
+		$commande = "$jpegtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_logger()->info($commande);
 		exec($commande);
 		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
@@ -84,9 +84,9 @@ 
 		}
 		$giftopnm_command = str_replace('pnmscale', 'giftopnm', (string) _PNMSCALE_COMMAND);
 		$pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', (string) _PNMSCALE_COMMAND);
-		$vignette = _ROOT_IMG_PACK . 'test.gif';
-		$dest = _DIR_VAR . 'test-gif.jpg';
-		$commande = "$giftopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+		$vignette = _ROOT_IMG_PACK.'test.gif';
+		$dest = _DIR_VAR.'test-gif.jpg';
+		$commande = "$giftopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_logger()->info($commande);
 		exec($commande);
 		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
@@ -94,9 +94,9 @@ 
 		}
 
 		$pngtopnm_command = str_replace('pnmscale', 'pngtopnm', (string) _PNMSCALE_COMMAND);
-		$vignette = _ROOT_IMG_PACK . 'test.png';
-		$dest = _DIR_VAR . 'test-gif.jpg';
-		$commande = "$pngtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+		$vignette = _ROOT_IMG_PACK.'test.png';
+		$dest = _DIR_VAR.'test-gif.jpg';
+		$commande = "$pngtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_logger()->info($commande);
 		exec($commande);
 		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
@@ -111,9 +111,9 @@ 
 		include_spip('inc/filtres');
 		include_spip('inc/filtres_images_mini');
 		$taille_preview = 150;
-		$image = _image_valeurs_trans(_DIR_IMG_PACK . 'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
+		$image = _image_valeurs_trans(_DIR_IMG_PACK.'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
 
-		$image['fichier_dest'] = _DIR_VAR . "test_$arg";
+		$image['fichier_dest'] = _DIR_VAR."test_$arg";
 
 		if (
 			($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true))

ecrire/action/editer_logo.php

Indentation +231 added lines, -231 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -29,36 +29,36 @@ 
  *     `on` ou `off`
  */
 function logo_supprimer($objet, $id_objet, $etat) {
-	$chercher_logo = charger_fonction('chercher_logo', 'inc');
-	$objet = objet_type($objet);
-	$primary = id_table_objet($objet);
-	include_spip('inc/chercher_logo');
-
-	// existe-t-il deja un logo ?
-	$logo = $chercher_logo($id_objet, $primary, $etat);
-	if ($logo) {
-		# TODO : deprecated, a supprimer -> anciens logos IMG/artonxx.png pas en base
-		if ((is_countable($logo) ? count($logo) : 0) < 6) {
-			spip_logger('logo')->info('Supprimer ancien logo ' . json_encode($logo, JSON_THROW_ON_ERROR));
-			spip_unlink($logo[0]);
-		}
-		elseif (
-			($doc = $logo[5])
-			&& isset($doc['id_document'])
-			&& ($id_document = $doc['id_document'])
-		) {
-			include_spip('action/editer_liens');
-			// supprimer le lien dans la base
-			objet_dissocier(['document' => $id_document], [$objet => $id_objet], ['role' => '*']);
-
-			// verifier si il reste des liens avec d'autres objets et sinon supprimer
-			$liens = objet_trouver_liens(['document' => $id_document], '*');
-			if ($liens === []) {
-				$supprimer_document = charger_fonction('supprimer_document', 'action');
-				$supprimer_document($doc['id_document']);
-			}
-		}
-	}
+    $chercher_logo = charger_fonction('chercher_logo', 'inc');
+    $objet = objet_type($objet);
+    $primary = id_table_objet($objet);
+    include_spip('inc/chercher_logo');
+
+    // existe-t-il deja un logo ?
+    $logo = $chercher_logo($id_objet, $primary, $etat);
+    if ($logo) {
+        # TODO : deprecated, a supprimer -> anciens logos IMG/artonxx.png pas en base
+        if ((is_countable($logo) ? count($logo) : 0) < 6) {
+            spip_logger('logo')->info('Supprimer ancien logo ' . json_encode($logo, JSON_THROW_ON_ERROR));
+            spip_unlink($logo[0]);
+        }
+        elseif (
+            ($doc = $logo[5])
+            && isset($doc['id_document'])
+            && ($id_document = $doc['id_document'])
+        ) {
+            include_spip('action/editer_liens');
+            // supprimer le lien dans la base
+            objet_dissocier(['document' => $id_document], [$objet => $id_objet], ['role' => '*']);
+
+            // verifier si il reste des liens avec d'autres objets et sinon supprimer
+            $liens = objet_trouver_liens(['document' => $id_document], '*');
+            if ($liens === []) {
+                $supprimer_document = charger_fonction('supprimer_document', 'action');
+                $supprimer_document($doc['id_document']);
+            }
+        }
+    }
 }
 
 /**
@@ -75,76 +75,76 @@ 
  *     Erreur, sinon ''
  */
 function logo_modifier($objet, $id_objet, $etat, $source) {
-	$chercher_logo = charger_fonction('chercher_logo', 'inc');
-	$objet = objet_type($objet);
-	$primary = id_table_objet($objet);
-	include_spip('inc/chercher_logo');
-
-	$mode = preg_replace(',\W,', '', $etat);
-	if (!$mode) {
-		spip_logger('logo')->info("logo_modifier : etat $etat invalide");
-
-		return 'etat invalide';
-	}
-	// chercher dans la base
-	$mode_document = 'logo' . $mode;
-
-	include_spip('inc/documents');
-	$erreur = '';
-
-	if (!$source) {
-		spip_logger('logo')->info('spip_image_ajouter : source inconnue');
-
-		return 'source inconnue';
-	}
-
-	// fichier dans upload/
-	if (is_string($source)) {
-		$tmp_name = false;
-		if (file_exists($source)) {
-			$tmp_name = $source;
-		} elseif (file_exists($f = determine_upload() . $source)) {
-			$tmp_name = $f;
-		}
-		if (!$tmp_name) {
-			spip_logger('logo')->info('spip_image_ajouter : source inconnue');
-
-			return 'source inconnue';
-		}
-		$source = [
-			'tmp_name' => $tmp_name,
-			'name' => basename($tmp_name),
-		];
-	} elseif ($erreur = check_upload_error($source['error'], '', true)) {
-		return $erreur;
-	}
-
-	// supprimer le logo éventuel existant
-	// TODO : si un logo existe, le modifier plutot que supprimer + reinserer
-	// (mais il faut gerer le cas ou il est utilise par plusieurs objets, donc pas si simple)
-	// mais de toute facon l'interface actuelle oblige a supprimer + reinserer
-	// @see medias_upgrade_logo_objet()
-	if (empty($GLOBALS['logo_migrer_en_base'])) {
-		logo_supprimer($objet, $id_objet, $etat);
-	}
-
-
-	include_spip('inc/autoriser');
-	$source['mode'] = $mode_document;
-	$ajouter_documents = charger_fonction('ajouter_documents', 'action');
-	autoriser_exception('associerdocuments', $objet, $id_objet);
-	$ajoutes = $ajouter_documents('new', [$source], $objet, $id_objet, $mode_document);
-	autoriser_exception('associerdocuments', $objet, $id_objet, false);
-
-	$id_document = reset($ajoutes);
-
-	if (!is_numeric($id_document)) {
-		$erreur = ($id_document ?: 'Erreur inconnue');
-		spip_logger('logo')->info("Erreur ajout logo : $erreur pour source=" . json_encode($source, JSON_THROW_ON_ERROR));
-		return $erreur;
-	}
-
-	return ''; // tout est bon, pas d'erreur
+    $chercher_logo = charger_fonction('chercher_logo', 'inc');
+    $objet = objet_type($objet);
+    $primary = id_table_objet($objet);
+    include_spip('inc/chercher_logo');
+
+    $mode = preg_replace(',\W,', '', $etat);
+    if (!$mode) {
+        spip_logger('logo')->info("logo_modifier : etat $etat invalide");
+
+        return 'etat invalide';
+    }
+    // chercher dans la base
+    $mode_document = 'logo' . $mode;
+
+    include_spip('inc/documents');
+    $erreur = '';
+
+    if (!$source) {
+        spip_logger('logo')->info('spip_image_ajouter : source inconnue');
+
+        return 'source inconnue';
+    }
+
+    // fichier dans upload/
+    if (is_string($source)) {
+        $tmp_name = false;
+        if (file_exists($source)) {
+            $tmp_name = $source;
+        } elseif (file_exists($f = determine_upload() . $source)) {
+            $tmp_name = $f;
+        }
+        if (!$tmp_name) {
+            spip_logger('logo')->info('spip_image_ajouter : source inconnue');
+
+            return 'source inconnue';
+        }
+        $source = [
+            'tmp_name' => $tmp_name,
+            'name' => basename($tmp_name),
+        ];
+    } elseif ($erreur = check_upload_error($source['error'], '', true)) {
+        return $erreur;
+    }
+
+    // supprimer le logo éventuel existant
+    // TODO : si un logo existe, le modifier plutot que supprimer + reinserer
+    // (mais il faut gerer le cas ou il est utilise par plusieurs objets, donc pas si simple)
+    // mais de toute facon l'interface actuelle oblige a supprimer + reinserer
+    // @see medias_upgrade_logo_objet()
+    if (empty($GLOBALS['logo_migrer_en_base'])) {
+        logo_supprimer($objet, $id_objet, $etat);
+    }
+
+
+    include_spip('inc/autoriser');
+    $source['mode'] = $mode_document;
+    $ajouter_documents = charger_fonction('ajouter_documents', 'action');
+    autoriser_exception('associerdocuments', $objet, $id_objet);
+    $ajoutes = $ajouter_documents('new', [$source], $objet, $id_objet, $mode_document);
+    autoriser_exception('associerdocuments', $objet, $id_objet, false);
+
+    $id_document = reset($ajoutes);
+
+    if (!is_numeric($id_document)) {
+        $erreur = ($id_document ?: 'Erreur inconnue');
+        spip_logger('logo')->info("Erreur ajout logo : $erreur pour source=" . json_encode($source, JSON_THROW_ON_ERROR));
+        return $erreur;
+    }
+
+    return ''; // tout est bon, pas d'erreur
 }
 
 /**
@@ -164,126 +164,126 @@ 
  */
 function logo_migrer_en_base($objet, $time_limit) {
 
-	$dir_logos_erreurs = sous_repertoire(_DIR_IMG, 'logo_erreurs');
-	$dir_logos = sous_repertoire(_DIR_IMG, 'logo');
-	$formats_logos = ['jpg', 'png', 'svg', 'gif'];
-	if (isset($GLOBALS['formats_logos'])) {
-		$formats_logos = $GLOBALS['formats_logos'];
-	}
-
-
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$chercher_logo = charger_fonction('chercher_logo', 'inc');
-	include_spip('inc/chercher_logo');
-	$_id_objet = id_table_objet($objet);
-	$table = table_objet_sql($objet);
-	$type = type_du_logo($_id_objet);
-	$desc = $trouver_table($table);
-
-	// on desactive les revisions
-	$liste_objets_versionnes = $GLOBALS['meta']['objets_versions'] ?? '';
-	unset($GLOBALS['meta']['objets_versions']);
-	// et le signalement des editions
-	$articles_modif = $GLOBALS['meta']['articles_modif'] ?? '';
-	$GLOBALS['meta']['articles_modif'] = 'non';
-
-	foreach (['on', 'off'] as $mode) {
-		$nom_base = $type . $mode;
-		$dir = (defined('_DIR_LOGOS') ? _DIR_LOGOS : _DIR_IMG);
-
-		$files = glob($dir . $nom_base . '*');
-		// est-ce que c'est une nouvelle tentative de migration ?
-		// dans ce cas les logos sont deja dans IMG/logo/
-		if (!(is_countable($files) ? count($files) : 0)) {
-			$files = glob($dir_logos . $nom_base . '*');
-			if (is_countable($files) ? count($files) : 0) {
-				// mais il faut verifier si ils ont pas deja ete migres pour tout ou partie
-				$filescheck = [];
-				foreach ($files as $file) {
-					$short = basename(dirname((string) $file)) . DIRECTORY_SEPARATOR . basename((string) $file);
-					$filescheck[$short] = $file;
-				}
-				// trouver ceux deja migres
-				$deja = sql_allfetsel('fichier', 'spip_documents', sql_in('fichier', array_keys($filescheck)) . " AND mode LIKE 'logo%'");
-				if (is_countable($deja) ? count($deja) : 0) {
-					$deja = array_column($deja, 'fichier');
-					$restant = array_diff(array_keys($filescheck), $deja);
-					$files = [];
-					if ($restant !== []) {
-						foreach ($restant as $r) {
-							$files[] = $filescheck[$r];
-						}
-					}
-				}
-				// et si il en reste on peut y aller...
-				// mais il faut modifier $dir qui sert de base dans la suite
-				if (is_countable($files) ? count($files) : 0) {
-					$dir = $dir_logos;
-				}
-			}
-		}
-
-		$count = (is_countable($files) ? count($files) : 0);
-		spip_logger('maj')->notice("logo_migrer_en_base $objet $mode : " . $count . ' logos restant');
-
-		$deja = [];
-		foreach ($files as $file) {
-			$logo = substr((string) $file, strlen($dir . $nom_base));
-			$logo = explode('.', $logo);
-			if (
-				is_numeric($logo[0])
-				&& (($id_objet = (int) $logo[0]) || in_array($objet, ['site', 'rubrique']))
-				&& !isset($deja[$id_objet])
-			) {
-				$logo = $chercher_logo($id_objet, $_id_objet, $mode);
-				// if no logo in base
-				if (!$logo || (is_countable($logo) ? count($logo) : 0) < 6) {
-					foreach ($formats_logos as $format) {
-						if (@file_exists($d = ($dir . ($nom = $nom_base . (int) $id_objet . '.' . $format)))) {
-							if (isset($desc['field']['date_modif'])) {
-								$date_modif = sql_getfetsel('date_modif', $table, "$_id_objet=$id_objet");
-							} else {
-								$date_modif = null;
-							}
-							// s'assurer que le logo a les bon droits au passage (evite un echec en cas de sanitization d'un svg)
-							@chmod($d, _SPIP_CHMOD & 0666);
-							// logo_modifier commence par supprimer le logo existant, donc on le deplace pour pas le perdre
-							@rename($d, $dir_logos . $nom);
-							// et on le declare comme nouveau logo
-							logo_modifier($objet, $id_objet, $mode, $dir_logos . $nom);
-							if ($date_modif) {
-								sql_updateq($table, ['date_modif' => $date_modif], "$_id_objet=$id_objet");
-							}
-							break;
-						}
-					}
-				}
-				$deja[$id_objet] = true;
-			}
-			// si le fichier est encore la on le move : rien a faire ici
-			// (sauf si c'est une re-migration : il est deja dans logo/ donc il bouge pas)
-			if ($dir !== $dir_logos && file_exists($file)) {
-				@rename($file, $dir_logos_erreurs . basename((string) $file));
-			}
-
-			$count--;
-			if ($count % 250 === 0) {
-				spip_logger('maj')->notice("logo_migrer_en_base $objet $mode : " . $count . ' logos restant');
-			}
-
-			if ($time_limit && time() > $time_limit) {
-				effacer_meta('drapeau_edition');
-				return;
-			}
-		}
-	}
-
-	if ($liste_objets_versionnes) {
-		$GLOBALS['meta']['objets_versions'] = $liste_objets_versionnes;
-	}
-	$GLOBALS['meta']['articles_modif'] = $articles_modif;
-
-	effacer_meta('drapeau_edition');
+    $dir_logos_erreurs = sous_repertoire(_DIR_IMG, 'logo_erreurs');
+    $dir_logos = sous_repertoire(_DIR_IMG, 'logo');
+    $formats_logos = ['jpg', 'png', 'svg', 'gif'];
+    if (isset($GLOBALS['formats_logos'])) {
+        $formats_logos = $GLOBALS['formats_logos'];
+    }
+
+
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $chercher_logo = charger_fonction('chercher_logo', 'inc');
+    include_spip('inc/chercher_logo');
+    $_id_objet = id_table_objet($objet);
+    $table = table_objet_sql($objet);
+    $type = type_du_logo($_id_objet);
+    $desc = $trouver_table($table);
+
+    // on desactive les revisions
+    $liste_objets_versionnes = $GLOBALS['meta']['objets_versions'] ?? '';
+    unset($GLOBALS['meta']['objets_versions']);
+    // et le signalement des editions
+    $articles_modif = $GLOBALS['meta']['articles_modif'] ?? '';
+    $GLOBALS['meta']['articles_modif'] = 'non';
+
+    foreach (['on', 'off'] as $mode) {
+        $nom_base = $type . $mode;
+        $dir = (defined('_DIR_LOGOS') ? _DIR_LOGOS : _DIR_IMG);
+
+        $files = glob($dir . $nom_base . '*');
+        // est-ce que c'est une nouvelle tentative de migration ?
+        // dans ce cas les logos sont deja dans IMG/logo/
+        if (!(is_countable($files) ? count($files) : 0)) {
+            $files = glob($dir_logos . $nom_base . '*');
+            if (is_countable($files) ? count($files) : 0) {
+                // mais il faut verifier si ils ont pas deja ete migres pour tout ou partie
+                $filescheck = [];
+                foreach ($files as $file) {
+                    $short = basename(dirname((string) $file)) . DIRECTORY_SEPARATOR . basename((string) $file);
+                    $filescheck[$short] = $file;
+                }
+                // trouver ceux deja migres
+                $deja = sql_allfetsel('fichier', 'spip_documents', sql_in('fichier', array_keys($filescheck)) . " AND mode LIKE 'logo%'");
+                if (is_countable($deja) ? count($deja) : 0) {
+                    $deja = array_column($deja, 'fichier');
+                    $restant = array_diff(array_keys($filescheck), $deja);
+                    $files = [];
+                    if ($restant !== []) {
+                        foreach ($restant as $r) {
+                            $files[] = $filescheck[$r];
+                        }
+                    }
+                }
+                // et si il en reste on peut y aller...
+                // mais il faut modifier $dir qui sert de base dans la suite
+                if (is_countable($files) ? count($files) : 0) {
+                    $dir = $dir_logos;
+                }
+            }
+        }
+
+        $count = (is_countable($files) ? count($files) : 0);
+        spip_logger('maj')->notice("logo_migrer_en_base $objet $mode : " . $count . ' logos restant');
+
+        $deja = [];
+        foreach ($files as $file) {
+            $logo = substr((string) $file, strlen($dir . $nom_base));
+            $logo = explode('.', $logo);
+            if (
+                is_numeric($logo[0])
+                && (($id_objet = (int) $logo[0]) || in_array($objet, ['site', 'rubrique']))
+                && !isset($deja[$id_objet])
+            ) {
+                $logo = $chercher_logo($id_objet, $_id_objet, $mode);
+                // if no logo in base
+                if (!$logo || (is_countable($logo) ? count($logo) : 0) < 6) {
+                    foreach ($formats_logos as $format) {
+                        if (@file_exists($d = ($dir . ($nom = $nom_base . (int) $id_objet . '.' . $format)))) {
+                            if (isset($desc['field']['date_modif'])) {
+                                $date_modif = sql_getfetsel('date_modif', $table, "$_id_objet=$id_objet");
+                            } else {
+                                $date_modif = null;
+                            }
+                            // s'assurer que le logo a les bon droits au passage (evite un echec en cas de sanitization d'un svg)
+                            @chmod($d, _SPIP_CHMOD & 0666);
+                            // logo_modifier commence par supprimer le logo existant, donc on le deplace pour pas le perdre
+                            @rename($d, $dir_logos . $nom);
+                            // et on le declare comme nouveau logo
+                            logo_modifier($objet, $id_objet, $mode, $dir_logos . $nom);
+                            if ($date_modif) {
+                                sql_updateq($table, ['date_modif' => $date_modif], "$_id_objet=$id_objet");
+                            }
+                            break;
+                        }
+                    }
+                }
+                $deja[$id_objet] = true;
+            }
+            // si le fichier est encore la on le move : rien a faire ici
+            // (sauf si c'est une re-migration : il est deja dans logo/ donc il bouge pas)
+            if ($dir !== $dir_logos && file_exists($file)) {
+                @rename($file, $dir_logos_erreurs . basename((string) $file));
+            }
+
+            $count--;
+            if ($count % 250 === 0) {
+                spip_logger('maj')->notice("logo_migrer_en_base $objet $mode : " . $count . ' logos restant');
+            }
+
+            if ($time_limit && time() > $time_limit) {
+                effacer_meta('drapeau_edition');
+                return;
+            }
+        }
+    }
+
+    if ($liste_objets_versionnes) {
+        $GLOBALS['meta']['objets_versions'] = $liste_objets_versionnes;
+    }
+    $GLOBALS['meta']['articles_modif'] = $articles_modif;
+
+    effacer_meta('drapeau_edition');
 }
 
 
@@ -304,14 +304,14 @@ 
  * @deprecated 4.0 MAIS NE PAS SUPPRIMER CAR SERT POUR L'UPGRADE des logos et leur mise en base
  **/
 function type_du_logo($_id_objet) {
-	if ((debug_backtrace(0, 2)[1]['function'] ?? '') !== 'logo_migrer_en_base') {
-		trigger_deprecation('spip', '4.0', 'Using "%s" is deprecated', __FUNCTION__);
-	}
-	$legacy_tables_logos = [
-		'id_article' => 'art',
-		'id_auteur' => 'aut',
-		'id_rubrique' => 'rub',
-		'id_groupe' => 'groupe',
-	];
-	return $legacy_tables_logos[$_id_objet] ?? objet_type(preg_replace(',^id_,', '', $_id_objet));
+    if ((debug_backtrace(0, 2)[1]['function'] ?? '') !== 'logo_migrer_en_base') {
+        trigger_deprecation('spip', '4.0', 'Using "%s" is deprecated', __FUNCTION__);
+    }
+    $legacy_tables_logos = [
+        'id_article' => 'art',
+        'id_auteur' => 'aut',
+        'id_rubrique' => 'rub',
+        'id_groupe' => 'groupe',
+    ];
+    return $legacy_tables_logos[$_id_objet] ?? objet_type(preg_replace(',^id_,', '', $_id_objet));
 }

Spacing +16 added lines, -16 removed lines

@@ -39,7 +39,7 @@ 
 	if ($logo) {
 		# TODO : deprecated, a supprimer -> anciens logos IMG/artonxx.png pas en base
 		if ((is_countable($logo) ? count($logo) : 0) < 6) {
-			spip_logger('logo')->info('Supprimer ancien logo ' . json_encode($logo, JSON_THROW_ON_ERROR));
+			spip_logger('logo')->info('Supprimer ancien logo '.json_encode($logo, JSON_THROW_ON_ERROR));
 			spip_unlink($logo[0]);
 		}
 		elseif (
@@ -87,7 +87,7 @@ 
 		return 'etat invalide';
 	}
 	// chercher dans la base
-	$mode_document = 'logo' . $mode;
+	$mode_document = 'logo'.$mode;
 
 	include_spip('inc/documents');
 	$erreur = '';
@@ -103,7 +103,7 @@ 
 		$tmp_name = false;
 		if (file_exists($source)) {
 			$tmp_name = $source;
-		} elseif (file_exists($f = determine_upload() . $source)) {
+		} elseif (file_exists($f = determine_upload().$source)) {
 			$tmp_name = $f;
 		}
 		if (!$tmp_name) {
@@ -140,7 +140,7 @@ 
 
 	if (!is_numeric($id_document)) {
 		$erreur = ($id_document ?: 'Erreur inconnue');
-		spip_logger('logo')->info("Erreur ajout logo : $erreur pour source=" . json_encode($source, JSON_THROW_ON_ERROR));
+		spip_logger('logo')->info("Erreur ajout logo : $erreur pour source=".json_encode($source, JSON_THROW_ON_ERROR));
 		return $erreur;
 	}
 
@@ -188,23 +188,23 @@ 
 	$GLOBALS['meta']['articles_modif'] = 'non';
 
 	foreach (['on', 'off'] as $mode) {
-		$nom_base = $type . $mode;
+		$nom_base = $type.$mode;
 		$dir = (defined('_DIR_LOGOS') ? _DIR_LOGOS : _DIR_IMG);
 
-		$files = glob($dir . $nom_base . '*');
+		$files = glob($dir.$nom_base.'*');
 		// est-ce que c'est une nouvelle tentative de migration ?
 		// dans ce cas les logos sont deja dans IMG/logo/
 		if (!(is_countable($files) ? count($files) : 0)) {
-			$files = glob($dir_logos . $nom_base . '*');
+			$files = glob($dir_logos.$nom_base.'*');
 			if (is_countable($files) ? count($files) : 0) {
 				// mais il faut verifier si ils ont pas deja ete migres pour tout ou partie
 				$filescheck = [];
 				foreach ($files as $file) {
-					$short = basename(dirname((string) $file)) . DIRECTORY_SEPARATOR . basename((string) $file);
+					$short = basename(dirname((string) $file)).DIRECTORY_SEPARATOR.basename((string) $file);
 					$filescheck[$short] = $file;
 				}
 				// trouver ceux deja migres
-				$deja = sql_allfetsel('fichier', 'spip_documents', sql_in('fichier', array_keys($filescheck)) . " AND mode LIKE 'logo%'");
+				$deja = sql_allfetsel('fichier', 'spip_documents', sql_in('fichier', array_keys($filescheck))." AND mode LIKE 'logo%'");
 				if (is_countable($deja) ? count($deja) : 0) {
 					$deja = array_column($deja, 'fichier');
 					$restant = array_diff(array_keys($filescheck), $deja);
@@ -224,11 +224,11 @@ 
 		}
 
 		$count = (is_countable($files) ? count($files) : 0);
-		spip_logger('maj')->notice("logo_migrer_en_base $objet $mode : " . $count . ' logos restant');
+		spip_logger('maj')->notice("logo_migrer_en_base $objet $mode : ".$count.' logos restant');
 
 		$deja = [];
 		foreach ($files as $file) {
-			$logo = substr((string) $file, strlen($dir . $nom_base));
+			$logo = substr((string) $file, strlen($dir.$nom_base));
 			$logo = explode('.', $logo);
 			if (
 				is_numeric($logo[0])
@@ -239,7 +239,7 @@ 
 				// if no logo in base
 				if (!$logo || (is_countable($logo) ? count($logo) : 0) < 6) {
 					foreach ($formats_logos as $format) {
-						if (@file_exists($d = ($dir . ($nom = $nom_base . (int) $id_objet . '.' . $format)))) {
+						if (@file_exists($d = ($dir.($nom = $nom_base.(int) $id_objet.'.'.$format)))) {
 							if (isset($desc['field']['date_modif'])) {
 								$date_modif = sql_getfetsel('date_modif', $table, "$_id_objet=$id_objet");
 							} else {
@@ -248,9 +248,9 @@ 
 							// s'assurer que le logo a les bon droits au passage (evite un echec en cas de sanitization d'un svg)
 							@chmod($d, _SPIP_CHMOD & 0666);
 							// logo_modifier commence par supprimer le logo existant, donc on le deplace pour pas le perdre
-							@rename($d, $dir_logos . $nom);
+							@rename($d, $dir_logos.$nom);
 							// et on le declare comme nouveau logo
-							logo_modifier($objet, $id_objet, $mode, $dir_logos . $nom);
+							logo_modifier($objet, $id_objet, $mode, $dir_logos.$nom);
 							if ($date_modif) {
 								sql_updateq($table, ['date_modif' => $date_modif], "$_id_objet=$id_objet");
 							}
@@ -263,12 +263,12 @@ 
 			// si le fichier est encore la on le move : rien a faire ici
 			// (sauf si c'est une re-migration : il est deja dans logo/ donc il bouge pas)
 			if ($dir !== $dir_logos && file_exists($file)) {
-				@rename($file, $dir_logos_erreurs . basename((string) $file));
+				@rename($file, $dir_logos_erreurs.basename((string) $file));
 			}
 
 			$count--;
 			if ($count % 250 === 0) {
-				spip_logger('maj')->notice("logo_migrer_en_base $objet $mode : " . $count . ' logos restant');
+				spip_logger('maj')->notice("logo_migrer_en_base $objet $mode : ".$count.' logos restant');
 			}
 
 			if ($time_limit && time() > $time_limit) {

Braces +1 added lines, -2 removed lines

@@ -41,8 +41,7 @@
 		if ((is_countable($logo) ? count($logo) : 0) < 6) {
 			spip_logger('logo')->info('Supprimer ancien logo ' . json_encode($logo, JSON_THROW_ON_ERROR));
 			spip_unlink($logo[0]);
-		}
-		elseif (
+		} elseif (
 			($doc = $logo[5])
 			&& isset($doc['id_document'])
 			&& ($id_document = $doc['id_document'])

ecrire/action/api_transmettre.php

Indentation +80 added lines, -80 removed lines

@@ -16,92 +16,92 @@
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
 function action_api_transmettre_dist($arg = null) {
 
-	// Obtenir l'argument 'id_auteur/cle/format/fond'
-	if (is_null($arg)) {
-		$arg = _request('arg');
-	}
-
-	$args = explode('/', (string) $arg);
-
-	if (count($args) !== 4) {
-		action_api_transmettre_fail($arg);
-	}
-
-	[$id_auteur, $cle, $format, $fond] = $args;
-	$id_auteur = (int) $id_auteur;
-
-	if (preg_match(',[^\w\\.-],', $format)) {
-		action_api_transmettre_fail("format $format ??");
-	}
-	if (preg_match(',[^\w\\.-],', $fond)) {
-		action_api_transmettre_fail("fond $fond ??");
-	}
-
-	// verifier la cle
-	//[(#ENV{id,0}|securiser_acces{#ENV{cle}, voirstats, #ENV{op}, #ENV{args}}|?{1,0})]
-	//[(#ENV{id,0}|securiser_acces{#ENV{cle}, voirstats, #ENV{op}, #ENV{args}}|?{1,0})]
-
-	$qs = $_SERVER['QUERY_STRING'];
-	// retirer action et arg de la qs
-	$contexte = [];
-	parse_str((string) $qs, $contexte);
-	foreach (array_keys($contexte) as $k) {
-		if (in_array($k, ['action', 'arg', 'var_mode'])) {
-			unset($contexte[$k]);
-		}
-	}
-	$qs = http_build_query($contexte);
-	include_spip('inc/acces');
-	if (!securiser_acces_low_sec((int) $id_auteur, $cle, "transmettre/$format", $fond, $qs)) {
-		// si le autoriser low_sec n'est pas bon, on peut valider l'appel si l'auteur est identifie
-		include_spip('inc/autoriser');
-		$autoriser_type = preg_replace(',\W+,', '', "_{$format}{$fond}");
-		if (
-			!$id_auteur
-			|| empty($GLOBALS['visiteur_session']['id_auteur'])
-			|| $GLOBALS['visiteur_session']['id_auteur'] != $id_auteur
-			|| !autoriser('transmettre', $autoriser_type, $id_auteur)
-		) {
-			action_api_transmettre_fail("auth QS $qs ??");
-		}
-	}
-
-	$contexte['id_auteur'] = $id_auteur;
-
-	$fond = "transmettre/$format/$fond";
-
-	if (!trouver_fond($fond)) {
-		$fond = "prive/$fond";
-	}
-
-	if (!trouver_fond($fond)) {
-		action_api_transmettre_fail("fond $fond ??");
-	}
-
-	$res = recuperer_fond($fond, $contexte, ['raw' => true]);
-	if (!empty($res['entetes'])) {
-		foreach ($res['entetes'] as $h => $v) {
-			header("$h: $v");
-		}
-	}
-
-	$res = ltrim((string) $res['texte']);
-	if (empty($res)) {
-		spip_logger('transmettre')->notice("$arg $qs resultat vide");
-	}
-
-	echo $res;
-	exit();
+    // Obtenir l'argument 'id_auteur/cle/format/fond'
+    if (is_null($arg)) {
+        $arg = _request('arg');
+    }
+
+    $args = explode('/', (string) $arg);
+
+    if (count($args) !== 4) {
+        action_api_transmettre_fail($arg);
+    }
+
+    [$id_auteur, $cle, $format, $fond] = $args;
+    $id_auteur = (int) $id_auteur;
+
+    if (preg_match(',[^\w\\.-],', $format)) {
+        action_api_transmettre_fail("format $format ??");
+    }
+    if (preg_match(',[^\w\\.-],', $fond)) {
+        action_api_transmettre_fail("fond $fond ??");
+    }
+
+    // verifier la cle
+    //[(#ENV{id,0}|securiser_acces{#ENV{cle}, voirstats, #ENV{op}, #ENV{args}}|?{1,0})]
+    //[(#ENV{id,0}|securiser_acces{#ENV{cle}, voirstats, #ENV{op}, #ENV{args}}|?{1,0})]
+
+    $qs = $_SERVER['QUERY_STRING'];
+    // retirer action et arg de la qs
+    $contexte = [];
+    parse_str((string) $qs, $contexte);
+    foreach (array_keys($contexte) as $k) {
+        if (in_array($k, ['action', 'arg', 'var_mode'])) {
+            unset($contexte[$k]);
+        }
+    }
+    $qs = http_build_query($contexte);
+    include_spip('inc/acces');
+    if (!securiser_acces_low_sec((int) $id_auteur, $cle, "transmettre/$format", $fond, $qs)) {
+        // si le autoriser low_sec n'est pas bon, on peut valider l'appel si l'auteur est identifie
+        include_spip('inc/autoriser');
+        $autoriser_type = preg_replace(',\W+,', '', "_{$format}{$fond}");
+        if (
+            !$id_auteur
+            || empty($GLOBALS['visiteur_session']['id_auteur'])
+            || $GLOBALS['visiteur_session']['id_auteur'] != $id_auteur
+            || !autoriser('transmettre', $autoriser_type, $id_auteur)
+        ) {
+            action_api_transmettre_fail("auth QS $qs ??");
+        }
+    }
+
+    $contexte['id_auteur'] = $id_auteur;
+
+    $fond = "transmettre/$format/$fond";
+
+    if (!trouver_fond($fond)) {
+        $fond = "prive/$fond";
+    }
+
+    if (!trouver_fond($fond)) {
+        action_api_transmettre_fail("fond $fond ??");
+    }
+
+    $res = recuperer_fond($fond, $contexte, ['raw' => true]);
+    if (!empty($res['entetes'])) {
+        foreach ($res['entetes'] as $h => $v) {
+            header("$h: $v");
+        }
+    }
+
+    $res = ltrim((string) $res['texte']);
+    if (empty($res)) {
+        spip_logger('transmettre')->notice("$arg $qs resultat vide");
+    }
+
+    echo $res;
+    exit();
 }
 
 function action_api_transmettre_fail($arg): never {
-	include_spip('inc/minipres');
-	echo minipres(_T('info_acces_interdit'), $arg);
-	exit;
+    include_spip('inc/minipres');
+    echo minipres(_T('info_acces_interdit'), $arg);
+    exit;
 }

ecrire/action/referencer_traduction.php

Indentation +45 added lines, -45 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -40,54 +40,54 @@ 
  */
 function action_referencer_traduction_dist($objet, $id_objet, $id_trad) {
 
-	// ne rien faire si id_trad est ambigu
-	if (!is_numeric($id_trad)) {
-		return false;
-	}
+    // ne rien faire si id_trad est ambigu
+    if (!is_numeric($id_trad)) {
+        return false;
+    }
 
-	$table_objet_sql = table_objet_sql($objet);
-	$id_table_objet = id_table_objet($objet);
+    $table_objet_sql = table_objet_sql($objet);
+    $id_table_objet = id_table_objet($objet);
 
-	// on a fourni un id_trad : affectation ou modification du groupe de trad
-	if ($id_trad) {
-		// selectionner l'objet cible, qui doit etre different de nous-meme,
-		// et quitter s'il n'existe pas
-		$id_lier = sql_getfetsel(
-			'id_trad',
-			$table_objet_sql,
-			"$id_table_objet=" . (int) $id_trad . " AND NOT($id_table_objet=" . (int) $id_objet . ')'
-		);
-		if ($id_lier === null) {
-			spip_logger()->info("echec lien de trad vers objet $objet/$id_objet incorrect ($id_trad)");
+    // on a fourni un id_trad : affectation ou modification du groupe de trad
+    if ($id_trad) {
+        // selectionner l'objet cible, qui doit etre different de nous-meme,
+        // et quitter s'il n'existe pas
+        $id_lier = sql_getfetsel(
+            'id_trad',
+            $table_objet_sql,
+            "$id_table_objet=" . (int) $id_trad . " AND NOT($id_table_objet=" . (int) $id_objet . ')'
+        );
+        if ($id_lier === null) {
+            spip_logger()->info("echec lien de trad vers objet $objet/$id_objet incorrect ($id_trad)");
 
-			return false;
-		}
+            return false;
+        }
 
-		// $id_lier est le numero du groupe de traduction
-		// Si l'objet vise n'est pas deja traduit, son identifiant devient
-		// le nouvel id_trad de ce nouveau groupe et on l'affecte aux deux
-		// objets
-		if ($id_lier == 0) {
-			sql_updateq($table_objet_sql, ['id_trad' => $id_trad], "$id_table_objet IN ($id_trad, $id_objet)");
-		} // si id_lier = id_objet alors on veut changer la reference de tout le groupe de trad
-		elseif ($id_lier == $id_objet) {
-			sql_updateq($table_objet_sql, ['id_trad' => $id_trad], "id_trad = $id_lier");
-		} // sinon ajouter notre objet dans le groupe
-		else {
-			sql_updateq($table_objet_sql, ['id_trad' => $id_lier], "$id_table_objet=" . (int) $id_objet);
-		}
-	} // on a fourni un id_trad nul : sortir id_objet du groupe de trad
-	else {
-		$old_id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$id_table_objet=" . (int) $id_objet);
-		// supprimer le lien de traduction
-		sql_updateq($table_objet_sql, ['id_trad' => 0], "$id_table_objet=" . (int) $id_objet);
+        // $id_lier est le numero du groupe de traduction
+        // Si l'objet vise n'est pas deja traduit, son identifiant devient
+        // le nouvel id_trad de ce nouveau groupe et on l'affecte aux deux
+        // objets
+        if ($id_lier == 0) {
+            sql_updateq($table_objet_sql, ['id_trad' => $id_trad], "$id_table_objet IN ($id_trad, $id_objet)");
+        } // si id_lier = id_objet alors on veut changer la reference de tout le groupe de trad
+        elseif ($id_lier == $id_objet) {
+            sql_updateq($table_objet_sql, ['id_trad' => $id_trad], "id_trad = $id_lier");
+        } // sinon ajouter notre objet dans le groupe
+        else {
+            sql_updateq($table_objet_sql, ['id_trad' => $id_lier], "$id_table_objet=" . (int) $id_objet);
+        }
+    } // on a fourni un id_trad nul : sortir id_objet du groupe de trad
+    else {
+        $old_id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$id_table_objet=" . (int) $id_objet);
+        // supprimer le lien de traduction
+        sql_updateq($table_objet_sql, ['id_trad' => 0], "$id_table_objet=" . (int) $id_objet);
 
-		// Verifier si l'ancien groupe ne comporte plus qu'un seul objet. Alors mettre a zero.
-		$cpt = sql_countsel($table_objet_sql, 'id_trad=' . (int) $old_id_trad);
-		if ($cpt == 1) {
-			sql_updateq($table_objet_sql, ['id_trad' => 0], 'id_trad=' . (int) $old_id_trad);
-		}
-	}
+        // Verifier si l'ancien groupe ne comporte plus qu'un seul objet. Alors mettre a zero.
+        $cpt = sql_countsel($table_objet_sql, 'id_trad=' . (int) $old_id_trad);
+        if ($cpt == 1) {
+            sql_updateq($table_objet_sql, ['id_trad' => 0], 'id_trad=' . (int) $old_id_trad);
+        }
+    }
 
-	return true;
+    return true;
 }

Spacing +6 added lines, -6 removed lines

@@ -55,7 +55,7 @@ 
 		$id_lier = sql_getfetsel(
 			'id_trad',
 			$table_objet_sql,
-			"$id_table_objet=" . (int) $id_trad . " AND NOT($id_table_objet=" . (int) $id_objet . ')'
+			"$id_table_objet=".(int) $id_trad." AND NOT($id_table_objet=".(int) $id_objet.')'
 		);
 		if ($id_lier === null) {
 			spip_logger()->info("echec lien de trad vers objet $objet/$id_objet incorrect ($id_trad)");
@@ -74,18 +74,18 @@ 
 			sql_updateq($table_objet_sql, ['id_trad' => $id_trad], "id_trad = $id_lier");
 		} // sinon ajouter notre objet dans le groupe
 		else {
-			sql_updateq($table_objet_sql, ['id_trad' => $id_lier], "$id_table_objet=" . (int) $id_objet);
+			sql_updateq($table_objet_sql, ['id_trad' => $id_lier], "$id_table_objet=".(int) $id_objet);
 		}
 	} // on a fourni un id_trad nul : sortir id_objet du groupe de trad
 	else {
-		$old_id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$id_table_objet=" . (int) $id_objet);
+		$old_id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$id_table_objet=".(int) $id_objet);
 		// supprimer le lien de traduction
-		sql_updateq($table_objet_sql, ['id_trad' => 0], "$id_table_objet=" . (int) $id_objet);
+		sql_updateq($table_objet_sql, ['id_trad' => 0], "$id_table_objet=".(int) $id_objet);
 
 		// Verifier si l'ancien groupe ne comporte plus qu'un seul objet. Alors mettre a zero.
-		$cpt = sql_countsel($table_objet_sql, 'id_trad=' . (int) $old_id_trad);
+		$cpt = sql_countsel($table_objet_sql, 'id_trad='.(int) $old_id_trad);
 		if ($cpt == 1) {
-			sql_updateq($table_objet_sql, ['id_trad' => 0], 'id_trad=' . (int) $old_id_trad);
+			sql_updateq($table_objet_sql, ['id_trad' => 0], 'id_trad='.(int) $old_id_trad);
 		}
 	}
 

ecrire/action/session.php

Indentation +12 added lines, -12 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -31,16 +31,16 @@ 
  *   Envoyer en réponse : json contenant toutes les variables publiques de la session
  **/
 function action_session_dist() {
-	if (
-		($var = _request('var'))
-		&& preg_match(',^[a-z_0-9-]+$,i', (string) $var)
-		&& $_SERVER['REQUEST_METHOD'] == 'POST'
-	) {
-		include_spip('inc/session');
-		session_set('session_' . $var, $val = _request('val'));
-		#spip_logger('autosave')->info("autosave:$var:$val");
-	}
+    if (
+        ($var = _request('var'))
+        && preg_match(',^[a-z_0-9-]+$,i', (string) $var)
+        && $_SERVER['REQUEST_METHOD'] == 'POST'
+    ) {
+        include_spip('inc/session');
+        session_set('session_' . $var, $val = _request('val'));
+        #spip_logger('autosave')->info("autosave:$var:$val");
+    }
 
-	# TODO: mode lecture de session ; n'afficher que ce qu'il faut
-	#echo json_encode($GLOBALS['visiteur_session']);
+    # TODO: mode lecture de session ; n'afficher que ce qu'il faut
+    #echo json_encode($GLOBALS['visiteur_session']);
 }

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 		&& $_SERVER['REQUEST_METHOD'] == 'POST'
 	) {
 		include_spip('inc/session');
-		session_set('session_' . $var, $val = _request('val'));
+		session_set('session_'.$var, $val = _request('val'));
 		#spip_logger('autosave')->info("autosave:$var:$val");
 	}
 

ecrire/action/editer_objet.php

Indentation +558 added lines, -558 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -32,36 +32,36 @@ 
  */
 function action_editer_objet_dist($id = null, $objet = null, $set = null) {
 
-	// appel direct depuis une url avec arg = "objet/id"
-	if (is_null($id) || is_null($objet)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-		[$objet, $id] = array_pad(explode('/', (string) $arg, 2), 2, null);
-	}
-
-	// appel incorrect ou depuis une url erronnée interdit
-	if (is_null($id) || is_null($objet)) {
-		include_spip('inc/minipres');
-		echo minipres(_T('info_acces_interdit'));
-		die();
-	}
-
-	// si id n'est pas un nombre, c'est une creation
-	// mais on verifie qu'on a toutes les donnees qu'il faut.
-	if (!$id = (int) $id) {
-		// on ne sait pas si un parent existe mais on essaye
-		$id_parent = _request('id_parent');
-		$id = objet_inserer($objet, $id_parent);
-	}
-
-	if (!($id = (int) $id) > 0) {
-		return [$id, _L('echec enregistrement en base')];
-	}
-
-	// Enregistre l'envoi dans la BD
-	$err = objet_modifier($objet, $id, $set);
-
-	return [$id, $err];
+    // appel direct depuis une url avec arg = "objet/id"
+    if (is_null($id) || is_null($objet)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+        [$objet, $id] = array_pad(explode('/', (string) $arg, 2), 2, null);
+    }
+
+    // appel incorrect ou depuis une url erronnée interdit
+    if (is_null($id) || is_null($objet)) {
+        include_spip('inc/minipres');
+        echo minipres(_T('info_acces_interdit'));
+        die();
+    }
+
+    // si id n'est pas un nombre, c'est une creation
+    // mais on verifie qu'on a toutes les donnees qu'il faut.
+    if (!$id = (int) $id) {
+        // on ne sait pas si un parent existe mais on essaye
+        $id_parent = _request('id_parent');
+        $id = objet_inserer($objet, $id_parent);
+    }
+
+    if (!($id = (int) $id) > 0) {
+        return [$id, _L('echec enregistrement en base')];
+    }
+
+    // Enregistre l'envoi dans la BD
+    $err = objet_modifier($objet, $id, $set);
+
+    return [$id, $err];
 }
 
 /**
@@ -90,83 +90,83 @@ 
  *   - En cas d'erreur : (string) message d'erreur
  */
 function objet_modifier($objet, $id, $set = null) {
-	if (($t = objet_type($objet)) !== $objet) {
-		spip_logger('editer')->notice("objet_modifier: appel avec type $objet invalide au lieu de $t");
-		$objet = $t;
-	}
-	if (
-		include_spip('action/editer_' . $objet) && function_exists($modifier = $objet . '_modifier')
-	) {
-		return $modifier($id, $set);
-	}
-
-	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_sql);
-	if (!$desc || !isset($desc['field'])) {
-		spip_logger('editer')->error("Objet $objet inconnu dans objet_modifier");
-
-		return _L("Erreur objet $objet inconnu");
-	}
-	include_spip('inc/modifier');
-
-	$champ_date = '';
-	if (isset($desc['date']) && $desc['date']) {
-		$champ_date = $desc['date'];
-	} elseif (isset($desc['field']['date'])) {
-		$champ_date = 'date';
-	}
-
-	$include_list = array_keys($desc['field']);
-	// on ne traite pas la cle primaire par defaut, notamment car
-	// sur une creation, id_x vaut 'oui', et serait enregistre en id_x=0 dans la base
-	$include_list = array_diff($include_list, [$desc['key']['PRIMARY KEY']]);
-
-	if (isset($desc['champs_editables']) && is_array($desc['champs_editables'])) {
-		$include_list = $desc['champs_editables'];
-	}
-	$c = collecter_requests(
-		// include list
-		$include_list,
-		// exclude list
-		[$champ_date, 'statut', 'id_parent', 'id_secteur'],
-		// donnees eventuellement fournies
-		$set
-	);
-
-	// Si l'objet est publie, invalider les caches et demander sa reindexation
-	if (objet_test_si_publie($objet, $id)) {
-		$invalideur = "id='$objet/$id'";
-		$indexation = true;
-	} else {
-		$invalideur = '';
-		$indexation = false;
-	}
-
-	if (
-		$err = objet_modifier_champs(
-			$objet,
-			$id,
-			[
-			'data' => $set,
-			'nonvide' => '',
-			'invalideur' => $invalideur,
-			'indexation' => $indexation,
-			// champ a mettre a date('Y-m-d H:i:s') s'il y a modif
-			'date_modif' => (isset($desc['field']['date_modif']) ? 'date_modif' : '')
-			],
-			$c
-		)
-	) {
-		return $err;
-	}
-
-	// Modification de statut, changement de rubrique ?
-	// FIXME: Ici lorsqu'un $set est passé, la fonction collecter_requests() retourne tout
-	//         le tableau $set hors liste d’exclusion, mais du coup on a possiblement des champs en trop.
-	$c = collecter_requests([$champ_date, 'statut', 'id_parent'], [], $set);
-
-	return objet_instituer($objet, $id, $c);
+    if (($t = objet_type($objet)) !== $objet) {
+        spip_logger('editer')->notice("objet_modifier: appel avec type $objet invalide au lieu de $t");
+        $objet = $t;
+    }
+    if (
+        include_spip('action/editer_' . $objet) && function_exists($modifier = $objet . '_modifier')
+    ) {
+        return $modifier($id, $set);
+    }
+
+    $table_sql = table_objet_sql($objet);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_sql);
+    if (!$desc || !isset($desc['field'])) {
+        spip_logger('editer')->error("Objet $objet inconnu dans objet_modifier");
+
+        return _L("Erreur objet $objet inconnu");
+    }
+    include_spip('inc/modifier');
+
+    $champ_date = '';
+    if (isset($desc['date']) && $desc['date']) {
+        $champ_date = $desc['date'];
+    } elseif (isset($desc['field']['date'])) {
+        $champ_date = 'date';
+    }
+
+    $include_list = array_keys($desc['field']);
+    // on ne traite pas la cle primaire par defaut, notamment car
+    // sur une creation, id_x vaut 'oui', et serait enregistre en id_x=0 dans la base
+    $include_list = array_diff($include_list, [$desc['key']['PRIMARY KEY']]);
+
+    if (isset($desc['champs_editables']) && is_array($desc['champs_editables'])) {
+        $include_list = $desc['champs_editables'];
+    }
+    $c = collecter_requests(
+        // include list
+        $include_list,
+        // exclude list
+        [$champ_date, 'statut', 'id_parent', 'id_secteur'],
+        // donnees eventuellement fournies
+        $set
+    );
+
+    // Si l'objet est publie, invalider les caches et demander sa reindexation
+    if (objet_test_si_publie($objet, $id)) {
+        $invalideur = "id='$objet/$id'";
+        $indexation = true;
+    } else {
+        $invalideur = '';
+        $indexation = false;
+    }
+
+    if (
+        $err = objet_modifier_champs(
+            $objet,
+            $id,
+            [
+            'data' => $set,
+            'nonvide' => '',
+            'invalideur' => $invalideur,
+            'indexation' => $indexation,
+            // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
+            'date_modif' => (isset($desc['field']['date_modif']) ? 'date_modif' : '')
+            ],
+            $c
+        )
+    ) {
+        return $err;
+    }
+
+    // Modification de statut, changement de rubrique ?
+    // FIXME: Ici lorsqu'un $set est passé, la fonction collecter_requests() retourne tout
+    //         le tableau $set hors liste d’exclusion, mais du coup on a possiblement des champs en trop.
+    $c = collecter_requests([$champ_date, 'statut', 'id_parent'], [], $set);
+
+    return objet_instituer($objet, $id, $c);
 }
 
 /**
@@ -197,154 +197,154 @@ 
  *     - (int) 0 si le type d'objet n'existe pas ou que la table est mal déclarée
  */
 function objet_inserer($objet, $id_parent = null, $set = null) {
-	$d = null;
-	if (($t = objet_type($objet)) !== $objet) {
-		spip_logger('editer')->notice("objet_inserer: appel avec type $objet invalide au lieu de $t");
-		$objet = $t;
-	}
-	if (
-		include_spip('action/editer_' . $objet)
-		&& function_exists($inserer = $objet . '_inserer')
-	) {
-		return $inserer($id_parent, $set);
-	}
-
-	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_sql);
-	if (!$desc || !isset($desc['field'])) {
-		return 0;
-	}
-
-	$lang_rub = '';
-	$champs = [];
-	if (isset($desc['field']['id_rubrique'])) {
-		// Si id_rubrique vaut 0 ou n'est pas definie, creer l'objet
-		// dans la premiere rubrique racine
-		if (!$id_rubrique = (int) $id_parent) {
-			$row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0', '', '0+titre,titre', '1');
-			$id_rubrique = $row['id_rubrique'];
-		} else {
-			$row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
-		}
-
-		$champs['id_rubrique'] = $id_rubrique;
-		if (isset($desc['field']['id_secteur'])) {
-			$champs['id_secteur'] = $row['id_secteur'];
-		}
-		$lang_rub = $row['lang'];
-	}
-
-	// La langue a la creation : si les liens de traduction sont autorises
-	// dans les rubriques, on essaie avec la langue de l'auteur,
-	// ou a defaut celle de la rubrique
-	// Sinon c'est la langue de la rubrique qui est choisie + heritee
-	if (
-		isset($desc['field']['lang']) && !empty($GLOBALS['meta']['multi_objets']) && in_array(
-			$table_sql,
-			explode(',', (string) $GLOBALS['meta']['multi_objets'])
-		)
-	) {
-		lang_select($GLOBALS['visiteur_session']['lang'] ?? '');
-		if (
-			in_array(
-				$GLOBALS['spip_lang'],
-				explode(',', (string) $GLOBALS['meta']['langues_multilingue'])
-			)
-		) {
-			$champs['lang'] = $GLOBALS['spip_lang'];
-			if (isset($desc['field']['langue_choisie'])) {
-				$champs['langue_choisie'] = 'oui';
-			}
-		}
-	} elseif (isset($desc['field']['lang']) && isset($desc['field']['langue_choisie'])) {
-		$champs['lang'] = ($lang_rub ?: $GLOBALS['meta']['langue_site']);
-		$champs['langue_choisie'] = 'non';
-	}
-
-	if (isset($desc['field']['statut'])) {
-		if (isset($desc['statut_textes_instituer'])) {
-			$cles_statut = array_keys($desc['statut_textes_instituer']);
-			$champs['statut'] = reset($cles_statut);
-		} else {
-			$champs['statut'] = 'prepa';
-		}
-	}
-
-
-	if (isset($desc['date']) && ($d = $desc['date']) || isset($desc['field'][$d = 'date'])) {
-		$champs[$d] = date('Y-m-d H:i:s');
-	}
-
-	if ($set) {
-		$champs = array_merge($champs, $set);
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_insertion',
-		[
-			'args' => [
-				'table' => $table_sql,
-				'id_parent' => $id_parent,
-			],
-			'data' => $champs
-		]
-	);
-
-	$id = sql_insertq($table_sql, $champs);
-
-	if ($id) {
-		// controler si le serveur n'a pas renvoye une erreur
-		// et associer l'auteur sinon
-		// si la table n'a pas deja un champ id_auteur
-		// et si le form n'a pas poste un id_auteur (meme vide, ce qui sert a annuler cette auto association)
-		if ($id > 0 && !isset($desc['field']['id_auteur'])) {
-			$id_auteur = ((is_null(_request('id_auteur')) && isset($GLOBALS['visiteur_session']['id_auteur']))
-				? $GLOBALS['visiteur_session']['id_auteur']
-				: _request('id_auteur'));
-			if ($id_auteur) {
-				include_spip('action/editer_auteur');
-				auteur_associer($id_auteur, [$objet => $id]);
-			}
-		}
-
-		pipeline(
-			'post_insertion',
-			[
-				'args' => [
-					'table' => $table_sql,
-					'id_parent' => $id_parent,
-					'id_objet' => $id,
-				],
-				'data' => $champs
-			]
-		);
-	}
-
-	// Appeler une notification
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			"{$objet}_inserer",
-			$id,
-			[
-				'id_parent' => $id_parent,
-				'champs' => $champs,
-			]
-		);
-		$notifications(
-			'objet_inserer',
-			$id,
-			[
-				'objet' => $objet,
-				'id_objet' => $id,
-				'id_parent' => $id_parent,
-				'champs' => $champs,
-			]
-		);
-	}
-
-	return $id;
+    $d = null;
+    if (($t = objet_type($objet)) !== $objet) {
+        spip_logger('editer')->notice("objet_inserer: appel avec type $objet invalide au lieu de $t");
+        $objet = $t;
+    }
+    if (
+        include_spip('action/editer_' . $objet)
+        && function_exists($inserer = $objet . '_inserer')
+    ) {
+        return $inserer($id_parent, $set);
+    }
+
+    $table_sql = table_objet_sql($objet);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_sql);
+    if (!$desc || !isset($desc['field'])) {
+        return 0;
+    }
+
+    $lang_rub = '';
+    $champs = [];
+    if (isset($desc['field']['id_rubrique'])) {
+        // Si id_rubrique vaut 0 ou n'est pas definie, creer l'objet
+        // dans la premiere rubrique racine
+        if (!$id_rubrique = (int) $id_parent) {
+            $row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0', '', '0+titre,titre', '1');
+            $id_rubrique = $row['id_rubrique'];
+        } else {
+            $row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
+        }
+
+        $champs['id_rubrique'] = $id_rubrique;
+        if (isset($desc['field']['id_secteur'])) {
+            $champs['id_secteur'] = $row['id_secteur'];
+        }
+        $lang_rub = $row['lang'];
+    }
+
+    // La langue a la creation : si les liens de traduction sont autorises
+    // dans les rubriques, on essaie avec la langue de l'auteur,
+    // ou a defaut celle de la rubrique
+    // Sinon c'est la langue de la rubrique qui est choisie + heritee
+    if (
+        isset($desc['field']['lang']) && !empty($GLOBALS['meta']['multi_objets']) && in_array(
+            $table_sql,
+            explode(',', (string) $GLOBALS['meta']['multi_objets'])
+        )
+    ) {
+        lang_select($GLOBALS['visiteur_session']['lang'] ?? '');
+        if (
+            in_array(
+                $GLOBALS['spip_lang'],
+                explode(',', (string) $GLOBALS['meta']['langues_multilingue'])
+            )
+        ) {
+            $champs['lang'] = $GLOBALS['spip_lang'];
+            if (isset($desc['field']['langue_choisie'])) {
+                $champs['langue_choisie'] = 'oui';
+            }
+        }
+    } elseif (isset($desc['field']['lang']) && isset($desc['field']['langue_choisie'])) {
+        $champs['lang'] = ($lang_rub ?: $GLOBALS['meta']['langue_site']);
+        $champs['langue_choisie'] = 'non';
+    }
+
+    if (isset($desc['field']['statut'])) {
+        if (isset($desc['statut_textes_instituer'])) {
+            $cles_statut = array_keys($desc['statut_textes_instituer']);
+            $champs['statut'] = reset($cles_statut);
+        } else {
+            $champs['statut'] = 'prepa';
+        }
+    }
+
+
+    if (isset($desc['date']) && ($d = $desc['date']) || isset($desc['field'][$d = 'date'])) {
+        $champs[$d] = date('Y-m-d H:i:s');
+    }
+
+    if ($set) {
+        $champs = array_merge($champs, $set);
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_insertion',
+        [
+            'args' => [
+                'table' => $table_sql,
+                'id_parent' => $id_parent,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    $id = sql_insertq($table_sql, $champs);
+
+    if ($id) {
+        // controler si le serveur n'a pas renvoye une erreur
+        // et associer l'auteur sinon
+        // si la table n'a pas deja un champ id_auteur
+        // et si le form n'a pas poste un id_auteur (meme vide, ce qui sert a annuler cette auto association)
+        if ($id > 0 && !isset($desc['field']['id_auteur'])) {
+            $id_auteur = ((is_null(_request('id_auteur')) && isset($GLOBALS['visiteur_session']['id_auteur']))
+                ? $GLOBALS['visiteur_session']['id_auteur']
+                : _request('id_auteur'));
+            if ($id_auteur) {
+                include_spip('action/editer_auteur');
+                auteur_associer($id_auteur, [$objet => $id]);
+            }
+        }
+
+        pipeline(
+            'post_insertion',
+            [
+                'args' => [
+                    'table' => $table_sql,
+                    'id_parent' => $id_parent,
+                    'id_objet' => $id,
+                ],
+                'data' => $champs
+            ]
+        );
+    }
+
+    // Appeler une notification
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            "{$objet}_inserer",
+            $id,
+            [
+                'id_parent' => $id_parent,
+                'champs' => $champs,
+            ]
+        );
+        $notifications(
+            'objet_inserer',
+            $id,
+            [
+                'objet' => $objet,
+                'id_objet' => $id,
+                'id_parent' => $id_parent,
+                'champs' => $champs,
+            ]
+        );
+    }
+
+    return $id;
 }
 
 
@@ -361,139 +361,139 @@ 
  * @return string
  */
 function objet_instituer($objet, $id, $c, $calcul_rub = true) {
-	if (($t = objet_type($objet)) !== $objet) {
-		spip_logger('editer')->notice("objet_instituer: appel avec type $objet invalide au lieu de $t");
-		$objet = $t;
-	}
-	if (
-		include_spip('action/editer_' . $objet) && function_exists($instituer = $objet . '_instituer')
-	) {
-		return $instituer($id, $c, $calcul_rub);
-	}
-
-	$table_sql = table_objet_sql($objet);
-	$table_objet = table_objet($objet);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_sql);
-	if (!$desc || !isset($desc['field'])) {
-		return _L("Impossible d'instituer $objet : non connu en base");
-	}
-
-	include_spip('inc/autoriser');
-	include_spip('inc/rubriques');
-	include_spip('inc/modifier');
-
-	$sel = [];
-	$sel[] = (isset($desc['field']['statut']) ? 'statut' : "'' as statut");
-
-	$champ_date = '';
-	if (isset($desc['date']) && $desc['date']) {
-		$champ_date = $desc['date'];
-	} elseif (isset($desc['field']['date'])) {
-		$champ_date = 'date';
-	}
-
-	$sel[] = ($champ_date ? "$champ_date as date" : "'' as date");
-	$sel[] = (isset($desc['field']['id_rubrique']) ? 'id_rubrique' : '0 as id_rubrique');
-
-	$row = sql_fetsel($sel, $table_sql, id_table_objet($objet) . '=' . (int) $id);
-
-	$id_rubrique = $row['id_rubrique'];
-	$statut_ancien = $statut = $row['statut'];
-	$date_ancienne = $date = $row['date'];
-	$champs = [];
-
-	$d = ($date && isset($c[$champ_date])) ? $c[$champ_date] : null;
-	$s = (isset($desc['field']['statut']) && isset($c['statut'])) ? $c['statut'] : $statut;
-
-	// cf autorisations dans inc/instituer_objet
-	if ($s != $statut || $d && $d != $date) {
-		if (
-			$id_rubrique ?
-			autoriser('publierdans', 'rubrique', $id_rubrique)
-			:
-			autoriser('instituer', $objet, $id, null, ['statut' => $s])
-		) {
-			$statut = $champs['statut'] = $s;
-		} else {
-			if ($s != 'publie' && autoriser('modifier', $objet, $id)) {
-				$statut = $champs['statut'] = $s;
-			} else {
-				spip_logger('editer')->notice("editer_objet $objet #$id refus " . json_encode($c, JSON_THROW_ON_ERROR));
-			}
-		}
-
-		// En cas de publication, fixer la date a "maintenant"
-		// sauf si $c commande autre chose
-		// ou si l'objet est deja date dans le futur
-		// En cas de proposition d'un objet (mais pas depublication), idem
-		if (
-			$champ_date
-			&& (
-				$champs['statut'] == 'publie'
-				|| $champs['statut'] == 'prop' && !in_array($statut_ancien, ['publie', 'prop'])
-				|| $d
-			)
-		) {
-			$date = ($d || strtotime((string) ($d = $date)) > time()) ? $d : date('Y-m-d H:i:s');
-			$champs[$champ_date] = $date;
-		}
-	}
-
-	// Verifier que la rubrique demandee existe et est differente
-	// de la rubrique actuelle
-	if (
-		$id_rubrique
-		&& isset($c['id_parent'])
-		&& ($id_parent = $c['id_parent'])
-		&& $id_parent != $id_rubrique
-		&& sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . (int) $id_parent)
-	) {
-		$champs['id_rubrique'] = $id_parent;
-
-		// si l'objet etait publie
-		// et que le demandeur n'est pas admin de la rubrique
-		// repasser l'objet en statut 'propose'.
-		if (
-			$statut == 'publie'
-			&& !autoriser('publierdans', 'rubrique', $id_rubrique)
-		) {
-			$champs['statut'] = 'prop';
-		}
-	}
-
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_edition',
-		[
-			'args' => [
-				'table' => $table_sql,
-				'table_objet' => $table_objet,
-				'spip_table_objet' => $table_sql,
-				'objet' => $objet,
-				'id_objet' => $id,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-				'date_ancienne' => $date_ancienne,
-				'id_parent_ancien' => $id_rubrique,
-			],
-			'data' => $champs
-		]
-	);
-
-	if (!(is_countable($champs) ? count($champs) : 0)) {
-		return '';
-	}
-
-	// Envoyer les modifs.
-	objet_editer_heritage($objet, $id, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
-
-	// Invalider les caches
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='$objet/$id'");
-
-	/*
+    if (($t = objet_type($objet)) !== $objet) {
+        spip_logger('editer')->notice("objet_instituer: appel avec type $objet invalide au lieu de $t");
+        $objet = $t;
+    }
+    if (
+        include_spip('action/editer_' . $objet) && function_exists($instituer = $objet . '_instituer')
+    ) {
+        return $instituer($id, $c, $calcul_rub);
+    }
+
+    $table_sql = table_objet_sql($objet);
+    $table_objet = table_objet($objet);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_sql);
+    if (!$desc || !isset($desc['field'])) {
+        return _L("Impossible d'instituer $objet : non connu en base");
+    }
+
+    include_spip('inc/autoriser');
+    include_spip('inc/rubriques');
+    include_spip('inc/modifier');
+
+    $sel = [];
+    $sel[] = (isset($desc['field']['statut']) ? 'statut' : "'' as statut");
+
+    $champ_date = '';
+    if (isset($desc['date']) && $desc['date']) {
+        $champ_date = $desc['date'];
+    } elseif (isset($desc['field']['date'])) {
+        $champ_date = 'date';
+    }
+
+    $sel[] = ($champ_date ? "$champ_date as date" : "'' as date");
+    $sel[] = (isset($desc['field']['id_rubrique']) ? 'id_rubrique' : '0 as id_rubrique');
+
+    $row = sql_fetsel($sel, $table_sql, id_table_objet($objet) . '=' . (int) $id);
+
+    $id_rubrique = $row['id_rubrique'];
+    $statut_ancien = $statut = $row['statut'];
+    $date_ancienne = $date = $row['date'];
+    $champs = [];
+
+    $d = ($date && isset($c[$champ_date])) ? $c[$champ_date] : null;
+    $s = (isset($desc['field']['statut']) && isset($c['statut'])) ? $c['statut'] : $statut;
+
+    // cf autorisations dans inc/instituer_objet
+    if ($s != $statut || $d && $d != $date) {
+        if (
+            $id_rubrique ?
+            autoriser('publierdans', 'rubrique', $id_rubrique)
+            :
+            autoriser('instituer', $objet, $id, null, ['statut' => $s])
+        ) {
+            $statut = $champs['statut'] = $s;
+        } else {
+            if ($s != 'publie' && autoriser('modifier', $objet, $id)) {
+                $statut = $champs['statut'] = $s;
+            } else {
+                spip_logger('editer')->notice("editer_objet $objet #$id refus " . json_encode($c, JSON_THROW_ON_ERROR));
+            }
+        }
+
+        // En cas de publication, fixer la date a "maintenant"
+        // sauf si $c commande autre chose
+        // ou si l'objet est deja date dans le futur
+        // En cas de proposition d'un objet (mais pas depublication), idem
+        if (
+            $champ_date
+            && (
+                $champs['statut'] == 'publie'
+                || $champs['statut'] == 'prop' && !in_array($statut_ancien, ['publie', 'prop'])
+                || $d
+            )
+        ) {
+            $date = ($d || strtotime((string) ($d = $date)) > time()) ? $d : date('Y-m-d H:i:s');
+            $champs[$champ_date] = $date;
+        }
+    }
+
+    // Verifier que la rubrique demandee existe et est differente
+    // de la rubrique actuelle
+    if (
+        $id_rubrique
+        && isset($c['id_parent'])
+        && ($id_parent = $c['id_parent'])
+        && $id_parent != $id_rubrique
+        && sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . (int) $id_parent)
+    ) {
+        $champs['id_rubrique'] = $id_parent;
+
+        // si l'objet etait publie
+        // et que le demandeur n'est pas admin de la rubrique
+        // repasser l'objet en statut 'propose'.
+        if (
+            $statut == 'publie'
+            && !autoriser('publierdans', 'rubrique', $id_rubrique)
+        ) {
+            $champs['statut'] = 'prop';
+        }
+    }
+
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_edition',
+        [
+            'args' => [
+                'table' => $table_sql,
+                'table_objet' => $table_objet,
+                'spip_table_objet' => $table_sql,
+                'objet' => $objet,
+                'id_objet' => $id,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+                'date_ancienne' => $date_ancienne,
+                'id_parent_ancien' => $id_rubrique,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    if (!(is_countable($champs) ? count($champs) : 0)) {
+        return '';
+    }
+
+    // Envoyer les modifs.
+    objet_editer_heritage($objet, $id, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
+
+    // Invalider les caches
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='$objet/$id'");
+
+    /*
 	if ($date) {
 		$t = strtotime($date);
 		$p = @$GLOBALS['meta']['date_prochain_postdate'];
@@ -502,63 +502,63 @@ 
 		}
 	}*/
 
-	// Pipeline
-	pipeline(
-		'post_edition',
-		[
-			'args' => [
-				'table' => $table_sql,
-				'table_objet' => $table_objet,
-				'spip_table_objet' => $table_sql,
-				'objet' => $objet,
-				'id_objet' => $id,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-				'date_ancienne' => $date_ancienne,
-				'id_parent_ancien' => $id_rubrique,
-			],
-			'data' => $champs
-		]
-	);
-
-	// Notifications
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			"{$objet}_instituer",
-			$id,
-			[
-				'statut' => $statut,
-				'statut_ancien' => $statut_ancien,
-				'date' => $date,
-				'date_ancienne' => $date_ancienne,
-				'id_parent_ancien' => $id_rubrique,
-				'champs' => $champs,
-			]
-		);
-		$notifications(
-			'objet_instituer',
-			$id,
-			[
-				'objet' => $objet,
-				'id_objet' => $id,
-				'statut' => $statut,
-				'statut_ancien' => $statut_ancien,
-				'date' => $date,
-				'date_ancienne' => $date_ancienne,
-				'id_parent_ancien' => $id_rubrique,
-				'champs' => $champs,
-			]
-		);
-
-		// Rétro-compat
-		$notifications(
-			"instituer$objet",
-			$id,
-			['statut' => $statut, 'statut_ancien' => $statut_ancien, 'date' => $date, 'date_ancienne' => $date_ancienne]
-		);
-	}
-
-	return ''; // pas d'erreur
+    // Pipeline
+    pipeline(
+        'post_edition',
+        [
+            'args' => [
+                'table' => $table_sql,
+                'table_objet' => $table_objet,
+                'spip_table_objet' => $table_sql,
+                'objet' => $objet,
+                'id_objet' => $id,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+                'date_ancienne' => $date_ancienne,
+                'id_parent_ancien' => $id_rubrique,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    // Notifications
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            "{$objet}_instituer",
+            $id,
+            [
+                'statut' => $statut,
+                'statut_ancien' => $statut_ancien,
+                'date' => $date,
+                'date_ancienne' => $date_ancienne,
+                'id_parent_ancien' => $id_rubrique,
+                'champs' => $champs,
+            ]
+        );
+        $notifications(
+            'objet_instituer',
+            $id,
+            [
+                'objet' => $objet,
+                'id_objet' => $id,
+                'statut' => $statut,
+                'statut_ancien' => $statut_ancien,
+                'date' => $date,
+                'date_ancienne' => $date_ancienne,
+                'id_parent_ancien' => $id_rubrique,
+                'champs' => $champs,
+            ]
+        );
+
+        // Rétro-compat
+        $notifications(
+            "instituer$objet",
+            $id,
+            ['statut' => $statut, 'statut_ancien' => $statut_ancien, 'date' => $date, 'date_ancienne' => $date_ancienne]
+        );
+    }
+
+    return ''; // pas d'erreur
 }
 
 /**
@@ -573,51 +573,51 @@ 
  * @return void
  */
 function objet_editer_heritage($objet, $id, $id_rubrique, $statut, $champs, $cond = true) {
-	$table_sql = table_objet_sql($objet);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_sql);
-
-	// Si on deplace l'objet
-	// changer aussi son secteur et sa langue (si heritee)
-	if (isset($champs['id_rubrique'])) {
-		$row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
-		$langue = $row_rub['lang'];
-
-		if (isset($desc['field']['id_secteur'])) {
-			$champs['id_secteur'] = $row_rub['id_secteur'];
-		}
-
-		if (
-			isset($desc['field']['lang'])
-			&& isset($desc['field']['langue_choisie'])
-			&& sql_fetsel(
-				'1',
-				$table_sql,
-				id_table_objet($objet) . '=' . (int) $id . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
-			)
-		) {
-			$champs['lang'] = $langue;
-		}
-	}
-
-	if (!$champs) {
-		return;
-	}
-	sql_updateq($table_sql, $champs, id_table_objet($objet) . '=' . (int) $id);
-
-	// Changer le statut des rubriques concernees
-	if ($cond) {
-		include_spip('inc/rubriques');
-		//$postdate = ($GLOBALS['meta']["post_dates"] == "non" AND isset($champs['date']) AND (strtotime($champs['date']) < time()))?$champs['date']:false;
-		$postdate = false;
-		// On rajoute les infos de l'objet
-		$infos = [
-			'objet' => $objet,
-			'id_objet' => $id,
-			'statut_ancien' => $statut,
-		];
-		calculer_rubriques_if($id_rubrique, $champs, $infos, $postdate);
-	}
+    $table_sql = table_objet_sql($objet);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_sql);
+
+    // Si on deplace l'objet
+    // changer aussi son secteur et sa langue (si heritee)
+    if (isset($champs['id_rubrique'])) {
+        $row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
+        $langue = $row_rub['lang'];
+
+        if (isset($desc['field']['id_secteur'])) {
+            $champs['id_secteur'] = $row_rub['id_secteur'];
+        }
+
+        if (
+            isset($desc['field']['lang'])
+            && isset($desc['field']['langue_choisie'])
+            && sql_fetsel(
+                '1',
+                $table_sql,
+                id_table_objet($objet) . '=' . (int) $id . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
+            )
+        ) {
+            $champs['lang'] = $langue;
+        }
+    }
+
+    if (!$champs) {
+        return;
+    }
+    sql_updateq($table_sql, $champs, id_table_objet($objet) . '=' . (int) $id);
+
+    // Changer le statut des rubriques concernees
+    if ($cond) {
+        include_spip('inc/rubriques');
+        //$postdate = ($GLOBALS['meta']["post_dates"] == "non" AND isset($champs['date']) AND (strtotime($champs['date']) < time()))?$champs['date']:false;
+        $postdate = false;
+        // On rajoute les infos de l'objet
+        $infos = [
+            'objet' => $objet,
+            'id_objet' => $id,
+            'statut_ancien' => $statut,
+        ];
+        calculer_rubriques_if($id_rubrique, $champs, $infos, $postdate);
+    }
 }
 
 
@@ -646,71 +646,71 @@ 
  *     string|int : valeur du champ demande pour l'objet demande
  */
 function objet_lire($objet, $valeur_id, $options = []) {
-	if (($t = objet_type($objet)) !== $objet) {
-		spip_logger('editer')->notice("objet_lire: appel avec type $objet invalide au lieu de $t");
-		$objet = $t;
-	}
-
-	// tableau du cache des descriptions et des id d'objet (au sens id_xxx).
-	// Les tableaux sont toujours indexés par le trio [objet][cle][valeur_cle]
-	static $descriptions = [];
-
-	// On détermine le nom du champ id de la table.
-	include_spip('base/objets');
-	$primary = id_table_objet($objet);
-
-	// On détermine l'id à utiliser.
-	$champ_id = (empty($options['champ_id']) ? $primary : $options['champ_id']);
-
-	// Si l'objet n'a pas encore été stocké, il faut récupérer sa description complète.
-	if (
-		!isset($descriptions[$objet][$champ_id][$valeur_id])
-		|| isset($options['force']) && $options['force']
-	) {
-		// Il est possible pour un type d'objet de fournir une fonction de lecture de tous les champs d'un objet.
-		if (
-			include_spip('action/editer_' . $objet)
-			&& function_exists($lire = "{$objet}_lire_champs")
-		) {
-			$valeurs = $lire($objet, $valeur_id, $champ_id);
-		} else {
-			// On récupère la table SQL à partir du type d'objet.
-			$table = table_objet_sql($objet);
-
-			// La condition est appliquée sur le champ désigné par l'utilisateur.
-			$where = [
-				$champ_id . '=' . sql_quote($valeur_id)
-			];
-
-			// Acquisition de tous les champs de l'objet : si l'accès SQL retourne une erreur on renvoie un tableau vide.
-			$valeurs = sql_fetsel('*', $table, $where);
-		}
-
-		if (!$valeurs) {
-			$valeurs = false;
-		}
-
-		$descriptions[$objet][$champ_id][$valeur_id] = $valeurs;
-
-		if ($champ_id !== $primary && isset($valeurs[$primary])) {
-			$descriptions[$objet][$primary][$valeurs[$primary]] = $valeurs;
-			$descriptions[$objet][$champ_id][$valeur_id] = &$descriptions[$objet][$primary][$valeurs[$primary]];
-		}
-	}
-
-	$retour = $descriptions[$objet][$champ_id][$valeur_id];
-
-	// On ne retourne maintenant que les champs demandés.
-	// - on détermine les informations à renvoyer.
-	if ($retour && !empty($options['champs'])) {
-		$champs = $options['champs'];
-		// Extraction des seules informations demandées.
-		// -- si on demande une information unique on renvoie la valeur simple, sinon on renvoie un tableau.
-		// -- si une information n'est pas un champ valide elle n'est pas renvoyée sans renvoyer d'erreur.
-		$retour = is_array($champs)
-			? array_intersect_key($retour, array_flip($champs))
-			: $retour[$champs] ?? false;
-	}
-
-	return $retour;
+    if (($t = objet_type($objet)) !== $objet) {
+        spip_logger('editer')->notice("objet_lire: appel avec type $objet invalide au lieu de $t");
+        $objet = $t;
+    }
+
+    // tableau du cache des descriptions et des id d'objet (au sens id_xxx).
+    // Les tableaux sont toujours indexés par le trio [objet][cle][valeur_cle]
+    static $descriptions = [];
+
+    // On détermine le nom du champ id de la table.
+    include_spip('base/objets');
+    $primary = id_table_objet($objet);
+
+    // On détermine l'id à utiliser.
+    $champ_id = (empty($options['champ_id']) ? $primary : $options['champ_id']);
+
+    // Si l'objet n'a pas encore été stocké, il faut récupérer sa description complète.
+    if (
+        !isset($descriptions[$objet][$champ_id][$valeur_id])
+        || isset($options['force']) && $options['force']
+    ) {
+        // Il est possible pour un type d'objet de fournir une fonction de lecture de tous les champs d'un objet.
+        if (
+            include_spip('action/editer_' . $objet)
+            && function_exists($lire = "{$objet}_lire_champs")
+        ) {
+            $valeurs = $lire($objet, $valeur_id, $champ_id);
+        } else {
+            // On récupère la table SQL à partir du type d'objet.
+            $table = table_objet_sql($objet);
+
+            // La condition est appliquée sur le champ désigné par l'utilisateur.
+            $where = [
+                $champ_id . '=' . sql_quote($valeur_id)
+            ];
+
+            // Acquisition de tous les champs de l'objet : si l'accès SQL retourne une erreur on renvoie un tableau vide.
+            $valeurs = sql_fetsel('*', $table, $where);
+        }
+
+        if (!$valeurs) {
+            $valeurs = false;
+        }
+
+        $descriptions[$objet][$champ_id][$valeur_id] = $valeurs;
+
+        if ($champ_id !== $primary && isset($valeurs[$primary])) {
+            $descriptions[$objet][$primary][$valeurs[$primary]] = $valeurs;
+            $descriptions[$objet][$champ_id][$valeur_id] = &$descriptions[$objet][$primary][$valeurs[$primary]];
+        }
+    }
+
+    $retour = $descriptions[$objet][$champ_id][$valeur_id];
+
+    // On ne retourne maintenant que les champs demandés.
+    // - on détermine les informations à renvoyer.
+    if ($retour && !empty($options['champs'])) {
+        $champs = $options['champs'];
+        // Extraction des seules informations demandées.
+        // -- si on demande une information unique on renvoie la valeur simple, sinon on renvoie un tableau.
+        // -- si une information n'est pas un champ valide elle n'est pas renvoyée sans renvoyer d'erreur.
+        $retour = is_array($champs)
+            ? array_intersect_key($retour, array_flip($champs))
+            : $retour[$champs] ?? false;
+    }
+
+    return $retour;
 }

Spacing +13 added lines, -13 removed lines

@@ -95,7 +95,7 @@ 
 		$objet = $t;
 	}
 	if (
-		include_spip('action/editer_' . $objet) && function_exists($modifier = $objet . '_modifier')
+		include_spip('action/editer_'.$objet) && function_exists($modifier = $objet.'_modifier')
 	) {
 		return $modifier($id, $set);
 	}
@@ -203,8 +203,8 @@ 
 		$objet = $t;
 	}
 	if (
-		include_spip('action/editer_' . $objet)
-		&& function_exists($inserer = $objet . '_inserer')
+		include_spip('action/editer_'.$objet)
+		&& function_exists($inserer = $objet.'_inserer')
 	) {
 		return $inserer($id_parent, $set);
 	}
@@ -225,7 +225,7 @@ 
 			$row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0', '', '0+titre,titre', '1');
 			$id_rubrique = $row['id_rubrique'];
 		} else {
-			$row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
+			$row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique='.(int) $id_rubrique);
 		}
 
 		$champs['id_rubrique'] = $id_rubrique;
@@ -366,7 +366,7 @@ 
 		$objet = $t;
 	}
 	if (
-		include_spip('action/editer_' . $objet) && function_exists($instituer = $objet . '_instituer')
+		include_spip('action/editer_'.$objet) && function_exists($instituer = $objet.'_instituer')
 	) {
 		return $instituer($id, $c, $calcul_rub);
 	}
@@ -396,7 +396,7 @@ 
 	$sel[] = ($champ_date ? "$champ_date as date" : "'' as date");
 	$sel[] = (isset($desc['field']['id_rubrique']) ? 'id_rubrique' : '0 as id_rubrique');
 
-	$row = sql_fetsel($sel, $table_sql, id_table_objet($objet) . '=' . (int) $id);
+	$row = sql_fetsel($sel, $table_sql, id_table_objet($objet).'='.(int) $id);
 
 	$id_rubrique = $row['id_rubrique'];
 	$statut_ancien = $statut = $row['statut'];
@@ -419,7 +419,7 @@ 
 			if ($s != 'publie' && autoriser('modifier', $objet, $id)) {
 				$statut = $champs['statut'] = $s;
 			} else {
-				spip_logger('editer')->notice("editer_objet $objet #$id refus " . json_encode($c, JSON_THROW_ON_ERROR));
+				spip_logger('editer')->notice("editer_objet $objet #$id refus ".json_encode($c, JSON_THROW_ON_ERROR));
 			}
 		}
 
@@ -447,7 +447,7 @@ 
 		&& isset($c['id_parent'])
 		&& ($id_parent = $c['id_parent'])
 		&& $id_parent != $id_rubrique
-		&& sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . (int) $id_parent)
+		&& sql_fetsel('1', 'spip_rubriques', 'id_rubrique='.(int) $id_parent)
 	) {
 		$champs['id_rubrique'] = $id_parent;
 
@@ -580,7 +580,7 @@ 
 	// Si on deplace l'objet
 	// changer aussi son secteur et sa langue (si heritee)
 	if (isset($champs['id_rubrique'])) {
-		$row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
+		$row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique='.sql_quote($champs['id_rubrique']));
 		$langue = $row_rub['lang'];
 
 		if (isset($desc['field']['id_secteur'])) {
@@ -593,7 +593,7 @@ 
 			&& sql_fetsel(
 				'1',
 				$table_sql,
-				id_table_objet($objet) . '=' . (int) $id . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
+				id_table_objet($objet).'='.(int) $id." AND langue_choisie<>'oui' AND lang<>".sql_quote($langue)
 			)
 		) {
 			$champs['lang'] = $langue;
@@ -603,7 +603,7 @@ 
 	if (!$champs) {
 		return;
 	}
-	sql_updateq($table_sql, $champs, id_table_objet($objet) . '=' . (int) $id);
+	sql_updateq($table_sql, $champs, id_table_objet($objet).'='.(int) $id);
 
 	// Changer le statut des rubriques concernees
 	if ($cond) {
@@ -669,7 +669,7 @@ 
 	) {
 		// Il est possible pour un type d'objet de fournir une fonction de lecture de tous les champs d'un objet.
 		if (
-			include_spip('action/editer_' . $objet)
+			include_spip('action/editer_'.$objet)
 			&& function_exists($lire = "{$objet}_lire_champs")
 		) {
 			$valeurs = $lire($objet, $valeur_id, $champ_id);
@@ -679,7 +679,7 @@ 
 
 			// La condition est appliquée sur le champ désigné par l'utilisateur.
 			$where = [
-				$champ_id . '=' . sql_quote($valeur_id)
+				$champ_id.'='.sql_quote($valeur_id)
 			];
 
 			// Acquisition de tous les champs de l'objet : si l'accès SQL retourne une erreur on renvoie un tableau vide.

ecrire/action/auth.php

Indentation +22 added lines, -22 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -24,25 +24,25 @@ 
  */
 function action_auth_dist() {
 
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$arg = $securiser_action();
-
-	if (!preg_match(',^(\w+)[/](.+)$,', (string) $arg, $r)) {
-		spip_logger()->info("action_auth_dist $arg pas compris");
-	} else {
-		$auth_methode = $r[1];
-		$login = $r[2];
-		include_spip('inc/auth');
-		$res = auth_terminer_identifier_login($auth_methode, $login);
-
-		if (is_string($res)) { // Erreur
-			$redirect = _request('redirect');
-			$redirect = parametre_url($redirect, 'var_erreur', $res, '&');
-			include_spip('inc/headers');
-			redirige_par_entete($redirect);
-		}
-
-		// sinon on loge l'auteur identifie, et on finit (redirection automatique)
-		auth_loger($res);
-	}
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $arg = $securiser_action();
+
+    if (!preg_match(',^(\w+)[/](.+)$,', (string) $arg, $r)) {
+        spip_logger()->info("action_auth_dist $arg pas compris");
+    } else {
+        $auth_methode = $r[1];
+        $login = $r[2];
+        include_spip('inc/auth');
+        $res = auth_terminer_identifier_login($auth_methode, $login);
+
+        if (is_string($res)) { // Erreur
+            $redirect = _request('redirect');
+            $redirect = parametre_url($redirect, 'var_erreur', $res, '&');
+            include_spip('inc/headers');
+            redirige_par_entete($redirect);
+        }
+
+        // sinon on loge l'auteur identifie, et on finit (redirection automatique)
+        auth_loger($res);
+    }
 }

ecrire/action/editer_article.php

Indentation +376 added lines, -376 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -38,36 +38,36 @@ 
  *     Liste (identifiant de l'article, texte d'erreur éventuel)
  */
 function action_editer_article_dist($arg = null) {
-	include_spip('inc/autoriser');
-	$err = '';
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
-
-	// si id_article n'est pas un nombre, c'est une creation
-	// mais on verifie qu'on a toutes les donnees qu'il faut.
-	if (!$id_article = (int) $arg) {
-		$id_parent = _request('id_parent');
-		if (!$id_parent) {
-			$err = _L("creation interdite d'un article sans rubrique");
-		} elseif (!autoriser('creerarticledans', 'rubrique', $id_parent)) {
-			$err = _T('info_creerdansrubrique_non_autorise');
-		} else {
-			$id_article = article_inserer($id_parent);
-		}
-	}
-
-	// Enregistre l'envoi dans la BD
-	if ($id_article > 0) {
-		$err = article_modifier($id_article);
-	}
-
-	if ($err) {
-		spip_logger()->error("echec editeur article: $err");
-	}
-
-	return [$id_article, $err];
+    include_spip('inc/autoriser');
+    $err = '';
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
+
+    // si id_article n'est pas un nombre, c'est une creation
+    // mais on verifie qu'on a toutes les donnees qu'il faut.
+    if (!$id_article = (int) $arg) {
+        $id_parent = _request('id_parent');
+        if (!$id_parent) {
+            $err = _L("creation interdite d'un article sans rubrique");
+        } elseif (!autoriser('creerarticledans', 'rubrique', $id_parent)) {
+            $err = _T('info_creerdansrubrique_non_autorise');
+        } else {
+            $id_article = article_inserer($id_parent);
+        }
+    }
+
+    // Enregistre l'envoi dans la BD
+    if ($id_article > 0) {
+        $err = article_modifier($id_article);
+    }
+
+    if ($err) {
+        spip_logger()->error("echec editeur article: $err");
+    }
+
+    return [$id_article, $err];
 }
 
 /**
@@ -89,49 +89,49 @@ 
  */
 function article_modifier($id_article, $set = null) {
 
-	// unifier $texte en cas de texte trop long
-	trop_longs_articles();
-
-	include_spip('inc/modifier');
-	include_spip('inc/filtres');
-	$c = collecter_requests(
-		// include list
-		objet_info('article', 'champs_editables'),
-		// exclude list
-		['date', 'statut', 'id_parent'],
-		// donnees eventuellement fournies
-		$set
-	);
-
-	// Si l'article est publie, invalider les caches et demander sa reindexation
-	$t = sql_getfetsel('statut', 'spip_articles', 'id_article=' . (int) $id_article);
-	$invalideur = $indexation = false;
-	if ($t == 'publie') {
-		$invalideur = "id='article/$id_article'";
-		$indexation = true;
-	}
-
-	if (
-		$err = objet_modifier_champs(
-			'article',
-			$id_article,
-			[
-			'data' => $set,
-			'nonvide' => ['titre' => _T('info_nouvel_article') . ' ' . _T('info_numero_abbreviation') . $id_article],
-			'invalideur' => $invalideur,
-			'indexation' => $indexation,
-			'date_modif' => 'date_modif' // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
-			],
-			$c
-		)
-	) {
-		return $err;
-	}
-
-	// Modification de statut, changement de rubrique ?
-	$c = collecter_requests(['date', 'statut', 'id_parent'], [], $set);
-
-	return article_instituer($id_article, $c);
+    // unifier $texte en cas de texte trop long
+    trop_longs_articles();
+
+    include_spip('inc/modifier');
+    include_spip('inc/filtres');
+    $c = collecter_requests(
+        // include list
+        objet_info('article', 'champs_editables'),
+        // exclude list
+        ['date', 'statut', 'id_parent'],
+        // donnees eventuellement fournies
+        $set
+    );
+
+    // Si l'article est publie, invalider les caches et demander sa reindexation
+    $t = sql_getfetsel('statut', 'spip_articles', 'id_article=' . (int) $id_article);
+    $invalideur = $indexation = false;
+    if ($t == 'publie') {
+        $invalideur = "id='article/$id_article'";
+        $indexation = true;
+    }
+
+    if (
+        $err = objet_modifier_champs(
+            'article',
+            $id_article,
+            [
+            'data' => $set,
+            'nonvide' => ['titre' => _T('info_nouvel_article') . ' ' . _T('info_numero_abbreviation') . $id_article],
+            'invalideur' => $invalideur,
+            'indexation' => $indexation,
+            'date_modif' => 'date_modif' // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
+            ],
+            $c
+        )
+    ) {
+        return $err;
+    }
+
+    // Modification de statut, changement de rubrique ?
+    $c = collecter_requests(['date', 'statut', 'id_parent'], [], $set);
+
+    return article_instituer($id_article, $c);
 }
 
 /**
@@ -167,120 +167,120 @@ 
  */
 function article_inserer($id_rubrique, $set = null) {
 
-	// Si id_rubrique vaut 0 ou n'est pas definie, creer l'article
-	// dans la premiere rubrique racine
-	if (!$id_rubrique = (int) $id_rubrique) {
-		$row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0', '', '0+titre,titre', '1');
-		$id_rubrique = $row['id_rubrique'];
-	} else {
-		$row = sql_fetsel('lang, id_secteur', 'spip_rubriques', "id_rubrique=$id_rubrique");
-	}
-
-	// eviter $id_secteur = NULL (erreur sqlite) si la requete precedente echoue
-	// cas de id_rubrique = -1 par exemple avec plugin "pages"
-	$id_secteur = $row['id_secteur'] ?? 0;
-	$lang_rub = $row['lang'] ?? '';
-
-	$lang = '';
-	$choisie = 'non';
-	// La langue a la creation : si les liens de traduction sont autorises
-	// dans les rubriques, on essaie avec la langue de l'auteur,
-	// ou a defaut celle de la rubrique
-	// Sinon c'est la langue de la rubrique qui est choisie + heritee
-	if (
-		!empty($GLOBALS['meta']['multi_objets']) && in_array(
-			'spip_articles',
-			explode(',', (string) $GLOBALS['meta']['multi_objets'])
-		)
-	) {
-		lang_select($GLOBALS['visiteur_session']['lang'] ?? '');
-		if (
-			in_array(
-				$GLOBALS['spip_lang'],
-				explode(',', (string) $GLOBALS['meta']['langues_multilingue'])
-			)
-		) {
-			$lang = $GLOBALS['spip_lang'];
-			$choisie = 'oui';
-		}
-	}
-
-	if (!$lang) {
-		$choisie = 'non';
-		$lang = $lang_rub ?: $GLOBALS['meta']['langue_site'];
-	}
-
-	$champs = [
-		'id_rubrique' => $id_rubrique,
-		'id_secteur' => $id_secteur,
-		'statut' => 'prepa',
-		'date' => date('Y-m-d H:i:s'),
-		'lang' => $lang,
-		'langue_choisie' => $choisie
-	];
-
-	if ($set) {
-		$champs = array_merge($champs, $set);
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_insertion',
-		[
-			'args' => [
-				'table' => 'spip_articles',
-			],
-			'data' => $champs
-		]
-	);
-
-	$id_article = sql_insertq('spip_articles', $champs);
-
-	// controler si le serveur n'a pas renvoye une erreur
-	if ($id_article > 0) {
-		$id_auteur = ((is_null(_request('id_auteur')) && isset($GLOBALS['visiteur_session']['id_auteur'])) ?
-			$GLOBALS['visiteur_session']['id_auteur']
-			: _request('id_auteur'));
-		if ($id_auteur) {
-			include_spip('action/editer_auteur');
-			auteur_associer($id_auteur, ['article' => $id_article]);
-		}
-	}
-
-	pipeline(
-		'post_insertion',
-		[
-			'args' => [
-				'table' => 'spip_articles',
-				'id_objet' => $id_article
-			],
-			'data' => $champs
-		]
-	);
-
-	// Appeler une notification
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			'article_inserer',
-			$id_article,
-			[
-				'id_parent' => $id_rubrique,
-				'champs' => $champs,
-			]
-		);
-		$notifications(
-			'objet_inserer',
-			$id_article,
-			[
-				'objet' => 'article',
-				'id_objet' => $id_article,
-				'id_parent' => $id_rubrique,
-				'champs' => $champs,
-			]
-		);
-	}
-
-	return $id_article;
+    // Si id_rubrique vaut 0 ou n'est pas definie, creer l'article
+    // dans la premiere rubrique racine
+    if (!$id_rubrique = (int) $id_rubrique) {
+        $row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0', '', '0+titre,titre', '1');
+        $id_rubrique = $row['id_rubrique'];
+    } else {
+        $row = sql_fetsel('lang, id_secteur', 'spip_rubriques', "id_rubrique=$id_rubrique");
+    }
+
+    // eviter $id_secteur = NULL (erreur sqlite) si la requete precedente echoue
+    // cas de id_rubrique = -1 par exemple avec plugin "pages"
+    $id_secteur = $row['id_secteur'] ?? 0;
+    $lang_rub = $row['lang'] ?? '';
+
+    $lang = '';
+    $choisie = 'non';
+    // La langue a la creation : si les liens de traduction sont autorises
+    // dans les rubriques, on essaie avec la langue de l'auteur,
+    // ou a defaut celle de la rubrique
+    // Sinon c'est la langue de la rubrique qui est choisie + heritee
+    if (
+        !empty($GLOBALS['meta']['multi_objets']) && in_array(
+            'spip_articles',
+            explode(',', (string) $GLOBALS['meta']['multi_objets'])
+        )
+    ) {
+        lang_select($GLOBALS['visiteur_session']['lang'] ?? '');
+        if (
+            in_array(
+                $GLOBALS['spip_lang'],
+                explode(',', (string) $GLOBALS['meta']['langues_multilingue'])
+            )
+        ) {
+            $lang = $GLOBALS['spip_lang'];
+            $choisie = 'oui';
+        }
+    }
+
+    if (!$lang) {
+        $choisie = 'non';
+        $lang = $lang_rub ?: $GLOBALS['meta']['langue_site'];
+    }
+
+    $champs = [
+        'id_rubrique' => $id_rubrique,
+        'id_secteur' => $id_secteur,
+        'statut' => 'prepa',
+        'date' => date('Y-m-d H:i:s'),
+        'lang' => $lang,
+        'langue_choisie' => $choisie
+    ];
+
+    if ($set) {
+        $champs = array_merge($champs, $set);
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_insertion',
+        [
+            'args' => [
+                'table' => 'spip_articles',
+            ],
+            'data' => $champs
+        ]
+    );
+
+    $id_article = sql_insertq('spip_articles', $champs);
+
+    // controler si le serveur n'a pas renvoye une erreur
+    if ($id_article > 0) {
+        $id_auteur = ((is_null(_request('id_auteur')) && isset($GLOBALS['visiteur_session']['id_auteur'])) ?
+            $GLOBALS['visiteur_session']['id_auteur']
+            : _request('id_auteur'));
+        if ($id_auteur) {
+            include_spip('action/editer_auteur');
+            auteur_associer($id_auteur, ['article' => $id_article]);
+        }
+    }
+
+    pipeline(
+        'post_insertion',
+        [
+            'args' => [
+                'table' => 'spip_articles',
+                'id_objet' => $id_article
+            ],
+            'data' => $champs
+        ]
+    );
+
+    // Appeler une notification
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            'article_inserer',
+            $id_article,
+            [
+                'id_parent' => $id_rubrique,
+                'champs' => $champs,
+            ]
+        );
+        $notifications(
+            'objet_inserer',
+            $id_article,
+            [
+                'objet' => 'article',
+                'id_objet' => $id_article,
+                'id_parent' => $id_rubrique,
+                'champs' => $champs,
+            ]
+        );
+    }
+
+    return $id_article;
 }
 
 
@@ -308,157 +308,157 @@ 
  */
 function article_instituer($id_article, $c, $calcul_rub = true) {
 
-	include_spip('inc/autoriser');
-	include_spip('inc/rubriques');
-	include_spip('inc/modifier');
-
-	$row = sql_fetsel('statut, date, id_rubrique', 'spip_articles', "id_article=$id_article");
-	$id_rubrique = $row['id_rubrique'];
-	$statut_ancien = $statut = $row['statut'];
-	$date_ancienne = $date = $row['date'];
-	$champs = [];
-
-	$d = $c['date'] ?? null;
-	$s = $c['statut'] ?? $statut;
-
-	// cf autorisations dans inc/instituer_article
-	if ($s != $statut || $d && $d != $date) {
-		if (autoriser('publierdans', 'rubrique', $id_rubrique)) {
-			$statut = $champs['statut'] = $s;
-		} elseif (autoriser('modifier', 'article', $id_article) && $s != 'publie') {
-			$statut = $champs['statut'] = $s;
-		} else {
-			spip_logger()->info("editer_article $id_article refus " . implode(' ', $c));
-		}
-
-		// En cas de publication, fixer la date a "maintenant"
-		// sauf si $c commande autre chose
-		// ou si l'article est deja date dans le futur
-		// En cas de proposition d'un article (mais pas depublication), idem
-		if (
-			$champs['statut'] == 'publie'
-			|| $champs['statut'] == 'prop' && ($d || !in_array($statut_ancien, ['publie', 'prop']))
-		) {
-			$champs['date'] = $d || strtotime((string) ($d = $date)) > time()
-				? $date = $d
-				: ($date = date('Y-m-d H:i:s'));
-		}
-	}
-
-	// Verifier que la rubrique demandee existe et est differente
-	// de la rubrique actuelle
-	if (
-		isset($c['id_parent'])
-		&& ($id_parent = $c['id_parent'])
-		&& $id_parent != $id_rubrique
-		&& sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . (int) $id_parent)
-	) {
-		$champs['id_rubrique'] = $id_parent;
-
-		// si l'article etait publie
-		// et que le demandeur n'est pas admin de la rubrique de destination
-		// repasser l'article en statut 'propose'.
-		if (
-			$statut == 'publie'
-			&& !autoriser('publierdans', 'rubrique', $id_parent)
-		) {
-			$champs['statut'] = 'prop';
-		}
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_edition',
-		[
-			'args' => [
-				'table' => 'spip_articles',
-				'table_objet' => 'articles',
-				'spip_table_objet' => 'spip_articles',
-				'objet' => 'article',
-				'id_objet' => $id_article,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-				'date_ancienne' => $date_ancienne,
-			],
-			'data' => $champs
-		]
-	);
-
-	if (!(is_countable($champs) ? count($champs) : 0)) {
-		return '';
-	}
-
-	// Envoyer les modifs.
-	editer_article_heritage($id_article, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
-
-	// Invalider les caches
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='article/$id_article'");
-
-	if ($date) {
-		$t = strtotime((string) $date);
-		$p = @$GLOBALS['meta']['date_prochain_postdate'];
-		if ($t > time() && (!$p || $t < $p)) {
-			ecrire_meta('date_prochain_postdate', $t);
-		}
-	}
-
-	// Pipeline
-	pipeline(
-		'post_edition',
-		[
-			'args' => [
-				'table' => 'spip_articles',
-				'table_objet' => 'articles',
-				'spip_table_objet' => 'spip_articles',
-				'objet' => 'article',
-				'id_objet' => $id_article,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-				'date_ancienne' => $date_ancienne,
-			],
-			'data' => $champs
-		]
-	);
-
-	// Notifications
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			'article_instituer',
-			$id_article,
-			[
-				'statut' => $statut,
-				'statut_ancien' => $statut_ancien,
-				'date' => $date,
-				'date_ancienne' => $date_ancienne,
-				'id_parent_ancien' => $id_rubrique,
-				'champs' => $champs,
-			]
-		);
-		$notifications(
-			'objet_instituer',
-			$id_article,
-			[
-				'objet' => 'article',
-				'id_objet' => $id_article,
-				'statut' => $statut,
-				'statut_ancien' => $statut_ancien,
-				'date' => $date,
-				'date_ancienne' => $date_ancienne,
-				'id_parent_ancien' => $id_rubrique,
-				'champs' => $champs,
-			]
-		);
-
-		// Rétro-compat
-		$notifications(
-			'instituerarticle',
-			$id_article,
-			['statut' => $statut, 'statut_ancien' => $statut_ancien, 'date' => $date, 'date_ancienne' => $date_ancienne]
-		);
-	}
-
-	return ''; // pas d'erreur
+    include_spip('inc/autoriser');
+    include_spip('inc/rubriques');
+    include_spip('inc/modifier');
+
+    $row = sql_fetsel('statut, date, id_rubrique', 'spip_articles', "id_article=$id_article");
+    $id_rubrique = $row['id_rubrique'];
+    $statut_ancien = $statut = $row['statut'];
+    $date_ancienne = $date = $row['date'];
+    $champs = [];
+
+    $d = $c['date'] ?? null;
+    $s = $c['statut'] ?? $statut;
+
+    // cf autorisations dans inc/instituer_article
+    if ($s != $statut || $d && $d != $date) {
+        if (autoriser('publierdans', 'rubrique', $id_rubrique)) {
+            $statut = $champs['statut'] = $s;
+        } elseif (autoriser('modifier', 'article', $id_article) && $s != 'publie') {
+            $statut = $champs['statut'] = $s;
+        } else {
+            spip_logger()->info("editer_article $id_article refus " . implode(' ', $c));
+        }
+
+        // En cas de publication, fixer la date a "maintenant"
+        // sauf si $c commande autre chose
+        // ou si l'article est deja date dans le futur
+        // En cas de proposition d'un article (mais pas depublication), idem
+        if (
+            $champs['statut'] == 'publie'
+            || $champs['statut'] == 'prop' && ($d || !in_array($statut_ancien, ['publie', 'prop']))
+        ) {
+            $champs['date'] = $d || strtotime((string) ($d = $date)) > time()
+                ? $date = $d
+                : ($date = date('Y-m-d H:i:s'));
+        }
+    }
+
+    // Verifier que la rubrique demandee existe et est differente
+    // de la rubrique actuelle
+    if (
+        isset($c['id_parent'])
+        && ($id_parent = $c['id_parent'])
+        && $id_parent != $id_rubrique
+        && sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . (int) $id_parent)
+    ) {
+        $champs['id_rubrique'] = $id_parent;
+
+        // si l'article etait publie
+        // et que le demandeur n'est pas admin de la rubrique de destination
+        // repasser l'article en statut 'propose'.
+        if (
+            $statut == 'publie'
+            && !autoriser('publierdans', 'rubrique', $id_parent)
+        ) {
+            $champs['statut'] = 'prop';
+        }
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_edition',
+        [
+            'args' => [
+                'table' => 'spip_articles',
+                'table_objet' => 'articles',
+                'spip_table_objet' => 'spip_articles',
+                'objet' => 'article',
+                'id_objet' => $id_article,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+                'date_ancienne' => $date_ancienne,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    if (!(is_countable($champs) ? count($champs) : 0)) {
+        return '';
+    }
+
+    // Envoyer les modifs.
+    editer_article_heritage($id_article, $id_rubrique, $statut_ancien, $champs, $calcul_rub);
+
+    // Invalider les caches
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='article/$id_article'");
+
+    if ($date) {
+        $t = strtotime((string) $date);
+        $p = @$GLOBALS['meta']['date_prochain_postdate'];
+        if ($t > time() && (!$p || $t < $p)) {
+            ecrire_meta('date_prochain_postdate', $t);
+        }
+    }
+
+    // Pipeline
+    pipeline(
+        'post_edition',
+        [
+            'args' => [
+                'table' => 'spip_articles',
+                'table_objet' => 'articles',
+                'spip_table_objet' => 'spip_articles',
+                'objet' => 'article',
+                'id_objet' => $id_article,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+                'date_ancienne' => $date_ancienne,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    // Notifications
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            'article_instituer',
+            $id_article,
+            [
+                'statut' => $statut,
+                'statut_ancien' => $statut_ancien,
+                'date' => $date,
+                'date_ancienne' => $date_ancienne,
+                'id_parent_ancien' => $id_rubrique,
+                'champs' => $champs,
+            ]
+        );
+        $notifications(
+            'objet_instituer',
+            $id_article,
+            [
+                'objet' => 'article',
+                'id_objet' => $id_article,
+                'statut' => $statut,
+                'statut_ancien' => $statut_ancien,
+                'date' => $date,
+                'date_ancienne' => $date_ancienne,
+                'id_parent_ancien' => $id_rubrique,
+                'champs' => $champs,
+            ]
+        );
+
+        // Rétro-compat
+        $notifications(
+            'instituerarticle',
+            $id_article,
+            ['statut' => $statut, 'statut_ancien' => $statut_ancien, 'date' => $date, 'date_ancienne' => $date_ancienne]
+        );
+    }
+
+    return ''; // pas d'erreur
 }
 
 /**
@@ -483,37 +483,37 @@ 
  */
 function editer_article_heritage($id_article, $id_rubrique, $statut, $champs, $cond = true) {
 
-	// Si on deplace l'article
-	//  changer aussi son secteur et sa langue (si heritee)
-	if (isset($champs['id_rubrique'])) {
-		$row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
-
-		$langue = $row_rub['lang'];
-		$champs['id_secteur'] = $row_rub['id_secteur'];
-		if (
-			sql_fetsel(
-				'1',
-				'spip_articles',
-				'id_article=' . (int) $id_article . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
-			)
-		) {
-			$champs['lang'] = $langue;
-		}
-	}
-
-	if (!$champs) {
-		return;
-	}
-
-	sql_updateq('spip_articles', $champs, 'id_article=' . (int) $id_article);
-
-	// Changer le statut des rubriques concernees
-
-	if ($cond) {
-		include_spip('inc/rubriques');
-		$postdate = ($GLOBALS['meta']['post_dates'] == 'non' && isset($champs['date']) && strtotime((string) $champs['date']) < time()) ? $champs['date'] : false;
-		calculer_rubriques_if($id_rubrique, $champs, ['statut_ancien' => $statut], $postdate);
-	}
+    // Si on deplace l'article
+    //  changer aussi son secteur et sa langue (si heritee)
+    if (isset($champs['id_rubrique'])) {
+        $row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
+
+        $langue = $row_rub['lang'];
+        $champs['id_secteur'] = $row_rub['id_secteur'];
+        if (
+            sql_fetsel(
+                '1',
+                'spip_articles',
+                'id_article=' . (int) $id_article . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
+            )
+        ) {
+            $champs['lang'] = $langue;
+        }
+    }
+
+    if (!$champs) {
+        return;
+    }
+
+    sql_updateq('spip_articles', $champs, 'id_article=' . (int) $id_article);
+
+    // Changer le statut des rubriques concernees
+
+    if ($cond) {
+        include_spip('inc/rubriques');
+        $postdate = ($GLOBALS['meta']['post_dates'] == 'non' && isset($champs['date']) && strtotime((string) $champs['date']) < time()) ? $champs['date'] : false;
+        calculer_rubriques_if($id_rubrique, $champs, ['statut_ancien' => $statut], $postdate);
+    }
 }
 
 /**
@@ -522,10 +522,10 @@ 
  * @return void
  */
 function trop_longs_articles() {
-	if (is_array($plus = _request('texte_plus'))) {
-		foreach ($plus as $n => $t) {
-			$plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,", '', (string) $t);
-		}
-		set_request('texte', implode('', $plus) . _request('texte'));
-	}
+    if (is_array($plus = _request('texte_plus'))) {
+        foreach ($plus as $n => $t) {
+            $plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,", '', (string) $t);
+        }
+        set_request('texte', implode('', $plus) . _request('texte'));
+    }
 }

Spacing +8 added lines, -8 removed lines

@@ -104,7 +104,7 @@ 
 	);
 
 	// Si l'article est publie, invalider les caches et demander sa reindexation
-	$t = sql_getfetsel('statut', 'spip_articles', 'id_article=' . (int) $id_article);
+	$t = sql_getfetsel('statut', 'spip_articles', 'id_article='.(int) $id_article);
 	$invalideur = $indexation = false;
 	if ($t == 'publie') {
 		$invalideur = "id='article/$id_article'";
@@ -117,7 +117,7 @@ 
 			$id_article,
 			[
 			'data' => $set,
-			'nonvide' => ['titre' => _T('info_nouvel_article') . ' ' . _T('info_numero_abbreviation') . $id_article],
+			'nonvide' => ['titre' => _T('info_nouvel_article').' '._T('info_numero_abbreviation').$id_article],
 			'invalideur' => $invalideur,
 			'indexation' => $indexation,
 			'date_modif' => 'date_modif' // champ a mettre a date('Y-m-d H:i:s') s'il y a modif
@@ -328,7 +328,7 @@ 
 		} elseif (autoriser('modifier', 'article', $id_article) && $s != 'publie') {
 			$statut = $champs['statut'] = $s;
 		} else {
-			spip_logger()->info("editer_article $id_article refus " . implode(' ', $c));
+			spip_logger()->info("editer_article $id_article refus ".implode(' ', $c));
 		}
 
 		// En cas de publication, fixer la date a "maintenant"
@@ -351,7 +351,7 @@ 
 		isset($c['id_parent'])
 		&& ($id_parent = $c['id_parent'])
 		&& $id_parent != $id_rubrique
-		&& sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . (int) $id_parent)
+		&& sql_fetsel('1', 'spip_rubriques', 'id_rubrique='.(int) $id_parent)
 	) {
 		$champs['id_rubrique'] = $id_parent;
 
@@ -486,7 +486,7 @@ 
 	// Si on deplace l'article
 	//  changer aussi son secteur et sa langue (si heritee)
 	if (isset($champs['id_rubrique'])) {
-		$row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
+		$row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique='.sql_quote($champs['id_rubrique']));
 
 		$langue = $row_rub['lang'];
 		$champs['id_secteur'] = $row_rub['id_secteur'];
@@ -494,7 +494,7 @@ 
 			sql_fetsel(
 				'1',
 				'spip_articles',
-				'id_article=' . (int) $id_article . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
+				'id_article='.(int) $id_article." AND langue_choisie<>'oui' AND lang<>".sql_quote($langue)
 			)
 		) {
 			$champs['lang'] = $langue;
@@ -505,7 +505,7 @@ 
 		return;
 	}
 
-	sql_updateq('spip_articles', $champs, 'id_article=' . (int) $id_article);
+	sql_updateq('spip_articles', $champs, 'id_article='.(int) $id_article);
 
 	// Changer le statut des rubriques concernees
 
@@ -526,6 +526,6 @@ 
 		foreach ($plus as $n => $t) {
 			$plus[$n] = preg_replace(",<!--SPIP-->[\n\r]*,", '', (string) $t);
 		}
-		set_request('texte', implode('', $plus) . _request('texte'));
+		set_request('texte', implode('', $plus)._request('texte'));
 	}
 }

ecrire/action/editer_rubrique.php

Indentation +218 added lines, -218 removed lines

@@ -15,7 +15,7 @@ 
  * @package SPIP\Core\Rubriques\Edition
  */
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/rubriques');
@@ -37,34 +37,34 @@ 
  */
 function action_editer_rubrique_dist($arg = null) {
 
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
-
-	if (!$id_rubrique = (int) $arg) {
-		if ($arg != 'oui') {
-			include_spip('inc/headers');
-			redirige_url_ecrire();
-		}
-		$id_rubrique = rubrique_inserer(_request('id_parent'));
-	}
-
-	$err = rubrique_modifier($id_rubrique);
-
-	if (_request('redirect')) {
-		$redirect = parametre_url(
-			urldecode((string) _request('redirect')),
-			'id_rubrique',
-			$id_rubrique,
-			'&'
-		);
-
-		include_spip('inc/headers');
-		redirige_par_entete($redirect);
-	}
-
-	return [$id_rubrique, $err];
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
+
+    if (!$id_rubrique = (int) $arg) {
+        if ($arg != 'oui') {
+            include_spip('inc/headers');
+            redirige_url_ecrire();
+        }
+        $id_rubrique = rubrique_inserer(_request('id_parent'));
+    }
+
+    $err = rubrique_modifier($id_rubrique);
+
+    if (_request('redirect')) {
+        $redirect = parametre_url(
+            urldecode((string) _request('redirect')),
+            'id_rubrique',
+            $id_rubrique,
+            '&'
+        );
+
+        include_spip('inc/headers');
+        redirige_par_entete($redirect);
+    }
+
+    return [$id_rubrique, $err];
 }
 
 
@@ -79,64 +79,64 @@ 
  *     Identifiant de la rubrique crée
  */
 function rubrique_inserer($id_parent, $set = null) {
-	$champs = [
-		'titre' => _T('item_nouvelle_rubrique'),
-		'id_parent' => (int) $id_parent,
-		'statut' => 'prepa'
-	];
-
-	if ($set) {
-		$champs = array_merge($champs, $set);
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_insertion',
-		[
-			'args' => [
-				'table' => 'spip_rubriques',
-			],
-			'data' => $champs
-		]
-	);
-
-	$id_rubrique = sql_insertq('spip_rubriques', $champs);
-	pipeline(
-		'post_insertion',
-		[
-			'args' => [
-				'table' => 'spip_rubriques',
-				'id_objet' => $id_rubrique
-			],
-			'data' => $champs
-		]
-	);
-	propager_les_secteurs();
-	calculer_langues_rubriques();
-
-	// Appeler une notification
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			'rubrique_inserer',
-			$id_rubrique,
-			[
-				'id_parent' => $id_parent,
-				'champs' => $champs,
-			]
-		);
-		$notifications(
-			'objet_inserer',
-			$id_rubrique,
-			[
-				'objet' => 'rubrique',
-				'id_objet' => $id_rubrique,
-				'id_parent' => $id_parent,
-				'champs' => $champs,
-			]
-		);
-	}
-
-	return $id_rubrique;
+    $champs = [
+        'titre' => _T('item_nouvelle_rubrique'),
+        'id_parent' => (int) $id_parent,
+        'statut' => 'prepa'
+    ];
+
+    if ($set) {
+        $champs = array_merge($champs, $set);
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_insertion',
+        [
+            'args' => [
+                'table' => 'spip_rubriques',
+            ],
+            'data' => $champs
+        ]
+    );
+
+    $id_rubrique = sql_insertq('spip_rubriques', $champs);
+    pipeline(
+        'post_insertion',
+        [
+            'args' => [
+                'table' => 'spip_rubriques',
+                'id_objet' => $id_rubrique
+            ],
+            'data' => $champs
+        ]
+    );
+    propager_les_secteurs();
+    calculer_langues_rubriques();
+
+    // Appeler une notification
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            'rubrique_inserer',
+            $id_rubrique,
+            [
+                'id_parent' => $id_parent,
+                'champs' => $champs,
+            ]
+        );
+        $notifications(
+            'objet_inserer',
+            $id_rubrique,
+            [
+                'objet' => 'rubrique',
+                'id_objet' => $id_rubrique,
+                'id_parent' => $id_parent,
+                'champs' => $champs,
+            ]
+        );
+    }
+
+    return $id_rubrique;
 }
 
 /**
@@ -152,46 +152,46 @@ 
  *     - chaîne : texte d'un message d'erreur
  */
 function rubrique_modifier($id_rubrique, $set = null) {
-	include_spip('inc/autoriser');
-	include_spip('inc/filtres');
-
-	include_spip('inc/modifier');
-	$c = collecter_requests(
-		// include list
-		objet_info('rubrique', 'champs_editables'),
-		// exclude list
-		['id_parent', 'confirme_deplace'],
-		// donnees eventuellement fournies
-		$set
-	);
-
-	if (
-		$err = objet_modifier_champs(
-			'rubrique',
-			$id_rubrique,
-			[
-			'data' => $set,
-			'nonvide' => ['titre' => _T('titre_nouvelle_rubrique') . ' ' . _T('info_numero_abbreviation') . $id_rubrique]
-			],
-			$c
-		)
-	) {
-		return $err;
-	}
-
-	$c = collecter_requests(['id_parent', 'confirme_deplace'], [], $set);
-	// Deplacer la rubrique
-	if (isset($c['id_parent'])) {
-		$err = rubrique_instituer($id_rubrique, $c);
-	}
-
-	// invalider les caches marques de cette rubrique
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='rubrique/$id_rubrique'");
-	// et celui de menu_rubriques
-	effacer_meta('date_calcul_rubriques');
-
-	return $err;
+    include_spip('inc/autoriser');
+    include_spip('inc/filtres');
+
+    include_spip('inc/modifier');
+    $c = collecter_requests(
+        // include list
+        objet_info('rubrique', 'champs_editables'),
+        // exclude list
+        ['id_parent', 'confirme_deplace'],
+        // donnees eventuellement fournies
+        $set
+    );
+
+    if (
+        $err = objet_modifier_champs(
+            'rubrique',
+            $id_rubrique,
+            [
+            'data' => $set,
+            'nonvide' => ['titre' => _T('titre_nouvelle_rubrique') . ' ' . _T('info_numero_abbreviation') . $id_rubrique]
+            ],
+            $c
+        )
+    ) {
+        return $err;
+    }
+
+    $c = collecter_requests(['id_parent', 'confirme_deplace'], [], $set);
+    // Deplacer la rubrique
+    if (isset($c['id_parent'])) {
+        $err = rubrique_instituer($id_rubrique, $c);
+    }
+
+    // invalider les caches marques de cette rubrique
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='rubrique/$id_rubrique'");
+    // et celui de menu_rubriques
+    effacer_meta('date_calcul_rubriques');
+
+    return $err;
 }
 
 /**
@@ -214,29 +214,29 @@ 
  *     false si la confirmation du déplacement n'est pas présente
  */
 function editer_rubrique_breves($id_rubrique, $id_parent, $c = []) {
-	if (!sql_table_exists('spip_breves')) {
-		return true;
-	}
-
-	if (!sql_countsel('spip_breves', "id_rubrique=$id_rubrique")) {
-		return true;
-	}
-
-	if (empty($c['confirme_deplace']) || $c['confirme_deplace'] != 'oui') {
-		return false;
-	}
-
-	if (
-		$id_secteur = sql_getfetsel(
-			'id_secteur',
-			'spip_rubriques',
-			"id_rubrique=$id_parent"
-		)
-	) {
-		sql_updateq('spip_breves', ['id_rubrique' => $id_secteur], "id_rubrique=$id_rubrique");
-	}
-
-	return true;
+    if (!sql_table_exists('spip_breves')) {
+        return true;
+    }
+
+    if (!sql_countsel('spip_breves', "id_rubrique=$id_rubrique")) {
+        return true;
+    }
+
+    if (empty($c['confirme_deplace']) || $c['confirme_deplace'] != 'oui') {
+        return false;
+    }
+
+    if (
+        $id_secteur = sql_getfetsel(
+            'id_secteur',
+            'spip_rubriques',
+            "id_rubrique=$id_parent"
+        )
+    ) {
+        sql_updateq('spip_breves', ['id_rubrique' => $id_secteur], "id_rubrique=$id_rubrique");
+    }
+
+    return true;
 }
 
 
@@ -258,72 +258,72 @@ 
  *     Chaîne : texte du message d'erreur
  */
 function rubrique_instituer($id_rubrique, $c) {
-	// traitement de la rubrique parente
-	// interdiction de deplacer vers ou a partir d'une rubrique
-	// qu'on n'administre pas.
-
-	if (null !== ($id_parent = $c['id_parent'])) {
-		$id_parent = (int) $id_parent;
-		$filles = calcul_branche_in($id_rubrique);
-		if (str_contains(",$id_parent,", (string) ",$filles,")) {
-			spip_logger()->info("La rubrique $id_rubrique ne peut etre fille de sa descendante $id_parent");
-		} else {
-			$s = sql_fetsel('id_parent, statut', 'spip_rubriques', "id_rubrique=$id_rubrique");
-			$old_parent = $s['id_parent'];
-
-			if (
-				!($id_parent != $old_parent
-				&& autoriser('publierdans', 'rubrique', $id_parent)
-				&& autoriser('creerrubriquedans', 'rubrique', $id_parent)
-				&& autoriser('publierdans', 'rubrique', $old_parent)
-				)
-			) {
-				if ($s['statut'] != 'prepa') {
-					spip_logger()->info("deplacement de $id_rubrique vers $id_parent refuse a " . $GLOBALS['visiteur_session']['id_auteur'] . ' ' . $GLOBALS['visiteur_session']['statut']);
-				}
-			} elseif (editer_rubrique_breves($id_rubrique, $id_parent, $c)) {
-				$statut_ancien = $s['statut'];
-				sql_updateq('spip_rubriques', ['id_parent' => $id_parent], "id_rubrique=$id_rubrique");
-
-
-				propager_les_secteurs();
-
-				// Deplacement d'une rubrique publiee ==> chgt general de leur statut
-				if ($statut_ancien == 'publie') {
-					calculer_rubriques_if($old_parent, ['id_rubrique' => $id_parent], ['statut_ancien' => $statut_ancien]);
-				}
-				// Creation ou deplacement d'une rubrique non publiee
-				// invalider le cache de leur menu
-				elseif (!$statut_ancien || $old_parent != $id_parent) {
-					effacer_meta('date_calcul_rubriques');
-				}
-
-				calculer_langues_rubriques();
-
-				// Appeler une notification
-				if ($notifications = charger_fonction('notifications', 'inc')) {
-					$notifications(
-						'rubrique_instituer',
-						$id_rubrique,
-						[
-							'statut_ancien' => $statut_ancien,
-							'id_parent_ancien' => $old_parent,
-						]
-					);
-					$notifications(
-						'objet_instituer',
-						$id_rubrique,
-						[
-							'objet' => 'rubrique',
-							'id_objet' => $id_rubrique,
-							'statut_ancien' => $statut_ancien,
-							'id_parent_ancien' => $old_parent,
-						]
-					);
-				}
-			}
-		}
-	}
-
-	return ''; // pas d'erreur
+    // traitement de la rubrique parente
+    // interdiction de deplacer vers ou a partir d'une rubrique
+    // qu'on n'administre pas.
+
+    if (null !== ($id_parent = $c['id_parent'])) {
+        $id_parent = (int) $id_parent;
+        $filles = calcul_branche_in($id_rubrique);
+        if (str_contains(",$id_parent,", (string) ",$filles,")) {
+            spip_logger()->info("La rubrique $id_rubrique ne peut etre fille de sa descendante $id_parent");
+        } else {
+            $s = sql_fetsel('id_parent, statut', 'spip_rubriques', "id_rubrique=$id_rubrique");
+            $old_parent = $s['id_parent'];
+
+            if (
+                !($id_parent != $old_parent
+                && autoriser('publierdans', 'rubrique', $id_parent)
+                && autoriser('creerrubriquedans', 'rubrique', $id_parent)
+                && autoriser('publierdans', 'rubrique', $old_parent)
+                )
+            ) {
+                if ($s['statut'] != 'prepa') {
+                    spip_logger()->info("deplacement de $id_rubrique vers $id_parent refuse a " . $GLOBALS['visiteur_session']['id_auteur'] . ' ' . $GLOBALS['visiteur_session']['statut']);
+                }
+            } elseif (editer_rubrique_breves($id_rubrique, $id_parent, $c)) {
+                $statut_ancien = $s['statut'];
+                sql_updateq('spip_rubriques', ['id_parent' => $id_parent], "id_rubrique=$id_rubrique");
+
+
+                propager_les_secteurs();
+
+                // Deplacement d'une rubrique publiee ==> chgt general de leur statut
+                if ($statut_ancien == 'publie') {
+                    calculer_rubriques_if($old_parent, ['id_rubrique' => $id_parent], ['statut_ancien' => $statut_ancien]);
+                }
+                // Creation ou deplacement d'une rubrique non publiee
+                // invalider le cache de leur menu
+                elseif (!$statut_ancien || $old_parent != $id_parent) {
+                    effacer_meta('date_calcul_rubriques');
+                }
+
+                calculer_langues_rubriques();
+
+                // Appeler une notification
+                if ($notifications = charger_fonction('notifications', 'inc')) {
+                    $notifications(
+                        'rubrique_instituer',
+                        $id_rubrique,
+                        [
+                            'statut_ancien' => $statut_ancien,
+                            'id_parent_ancien' => $old_parent,
+                        ]
+                    );
+                    $notifications(
+                        'objet_instituer',
+                        $id_rubrique,
+                        [
+                            'objet' => 'rubrique',
+                            'id_objet' => $id_rubrique,
+                            'statut_ancien' => $statut_ancien,
+                            'id_parent_ancien' => $old_parent,
+                        ]
+                    );
+                }
+            }
+        }
+    }
+
+    return ''; // pas d'erreur
 }

Spacing +2 added lines, -2 removed lines

@@ -171,7 +171,7 @@ 
 			$id_rubrique,
 			[
 			'data' => $set,
-			'nonvide' => ['titre' => _T('titre_nouvelle_rubrique') . ' ' . _T('info_numero_abbreviation') . $id_rubrique]
+			'nonvide' => ['titre' => _T('titre_nouvelle_rubrique').' '._T('info_numero_abbreviation').$id_rubrique]
 			],
 			$c
 		)
@@ -279,7 +279,7 @@ 
 				)
 			) {
 				if ($s['statut'] != 'prepa') {
-					spip_logger()->info("deplacement de $id_rubrique vers $id_parent refuse a " . $GLOBALS['visiteur_session']['id_auteur'] . ' ' . $GLOBALS['visiteur_session']['statut']);
+					spip_logger()->info("deplacement de $id_rubrique vers $id_parent refuse a ".$GLOBALS['visiteur_session']['id_auteur'].' '.$GLOBALS['visiteur_session']['statut']);
 				}
 			} elseif (editer_rubrique_breves($id_rubrique, $id_parent, $c)) {
 				$statut_ancien = $s['statut'];