chore: passe de rector UP_TO_PHP_73

action/associer_album.php

@@ -39,7 +39,7 @@ function action_associer_album_dist($arg = null) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 	}
-	list($id_album, $objet, $id_objet) = preg_split('/\W/', $arg);
+	[$id_album, $objet, $id_objet] = preg_split('/\W/', $arg);
 
 	if ($id_album = intval($id_album)) {
 		include_spip('action/editer_liens');

action/desordonner_liens_albums.php

@@ -39,7 +39,7 @@ function action_desordonner_liens_albums_dist(?string $arg = null) {
 		$arg = $securiser_action();
 	}
 	$data = explode('-', $arg);
-	list($objet, $objet_lie, $id_objet_lie, $champ_rang) = $data;
+	[$objet, $objet_lie, $id_objet_lie, $champ_rang] = $data;
 
 	// Objet source possédant la table de liens et éventuellement la colonne portant le rang
 	$objet = objet_type($objet);

action/dissocier_album.php

@@ -41,7 +41,7 @@ function action_dissocier_album_dist($arg = null) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 	}
-	list($album, $objet, $id_objet) = explode('/', $arg);
+	[$album, $objet, $id_objet] = explode('/', $arg);
 
 	// si l'identifiant de l'objet est négatif, vérifier qu'il correspondant à celui du visiteur,
 	// (cas d'un album lié à un objet pas encore enregistré en base).

action/instituer_album.php

@@ -38,7 +38,7 @@ function action_instituer_album_dist($arg = null) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 	}
-	list($id_album, $statut) = preg_split('/\W/', $arg);
+	[$id_album, $statut] = preg_split('/\W/', $arg);
 	if (!$statut) {
 		$statut = _request('statut_nouv'); // cas POST
 	}

action/ordonner_liens_albums.php

@@ -65,7 +65,7 @@ function action_ordonner_liens_albums_dist() {
 		return envoyer_json_erreur(_T('medias:erreur_autorisation') . ' ' . _T('medias:erreur_deplacement_impossible'));
 	}
 
-	list($cle_objet, $table_liens) = objet_associable($objet);
+	[$cle_objet, $table_liens] = objet_associable($objet);
 
 	$success = $errors = [];
 
@@ -116,7 +116,7 @@ function action_ordonner_liens_albums_dist() {
 
 function ordonner_liens_albums_json_envoi($data) {
 	header('Content-Type: application/json; charset=' . $GLOBALS['meta']['charset']);
-	echo json_encode($data);
+	echo json_encode($data, JSON_THROW_ON_ERROR);
 }
 
 function ordonner_liens_albums_json_erreur($msg) {

action/supprimer_album.php

@@ -41,8 +41,8 @@ function action_supprimer_album_dist($arg = null) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 	}
-	list($id_album, $supprimer_orphelins) = preg_split('/\W/', $arg);
-	$supprimer_docs_orphelins = ($supprimer_docs_orphelins == 'orphelins') ? true : false;
+	[$id_album, $supprimer_docs_orphelins] = preg_split('/\W/', $arg);
+	$supprimer_docs_orphelins = (($supprimer_docs_orphelins === 'orphelins') ? true : false);
 
 	// suppression
 	if ($id_album = intval($id_album)) {

action/vider_album.php

@@ -56,8 +56,8 @@ function action_vider_album_dist($arg = null) {
 		$arg = $securiser_action();
 	}
 
-	list($id_album, $supprimer_orphelins) = preg_split('/\W/', $arg);
-	$supprimer_orphelins = ($supprimer_orphelins == 'orphelins') ? true : false;
+	[$id_album, $supprimer_orphelins] = preg_split('/\W/', $arg);
+	$supprimer_orphelins = (($supprimer_orphelins === 'orphelins') ? true : false);
 
 	if ($id_album = intval($id_album)) {
 		include_spip('inc/albums');

albums_autorisations.php

@@ -432,7 +432,7 @@ function autoriser_album_vider_dist($faire, $type, $id, $qui, $options) {
  */
 function tester_album($test, $args) {
 	$ok = false;
-	list($faire, $type, $id, $qui, $options) = $args;
+	[$faire, $type, $id, $qui, $options] = $args;
 
 	// Perf : une statique car ces tests sont mutualisés entre les autorisations,
 	// donc appelés potentiellement plusieurs fois dans le même hit.

formulaires/ajouter_album.php

@@ -87,7 +87,7 @@ function formulaires_ajouter_album_charger_dist($objet = '', $id_objet = 0, $red
 		// Valeurs du formulaire d'édition d'un album
 		$charger_editer_album = charger_fonction('charger', 'formulaires/editer_album');
 		$valeurs_editer_album = $charger_editer_album('new', $redirection, $associer_objet, $lier_trad);
-		if (count($valeurs_editer_album)) {
+		if (is_countable($valeurs_editer_album) ? count($valeurs_editer_album) : 0) {
 			$valeurs = array_merge($valeurs, $valeurs_editer_album);
 		}
 
@@ -327,7 +327,7 @@ function js_ajouter_albums(string $ajaxbloc, array $params = [], array $ids = []
 		$callback = "function(){ jQuery('${divs}').animateAppend(); }";
 		$params['callback'] = $callback;
 	}
-	$params = json_encode($params);
+	$params = json_encode($params, JSON_THROW_ON_ERROR);
 	$js = "if (window.jQuery) jQuery(function(){ajaxReload('$ajaxbloc',$params);});";
 	$js = "<script type=\"text/javascript\">$js</script>";
 

formulaires/editer_album.php

@@ -74,7 +74,7 @@ function formulaires_editer_album_charger_dist($id_album = 'new', $retour = '',
 	if (
 		!intval($id_album)
 		and $associer_objet
-		and list($objet, $id_objet) = explode('|', $associer_objet)
+		and [$objet, $id_objet] = explode('|', $associer_objet)
 	) {
 		$valeurs['statut'] = 'publie';
 		include_spip('inc/config');
@@ -152,7 +152,7 @@ function formulaires_editer_album_traiter_dist($id_album = 'new', $retour = '',
 
 	// un lien a prendre en compte ?
 	if ($associer_objet and $id_album = $res['id_album']) {
-		if (list($objet, $id_objet) = explode('|', $associer_objet)) {
+		if ([$objet, $id_objet] = explode('|', $associer_objet)) {
 			include_spip('action/editer_liens');
 			objet_associer(['album' => $id_album], [$objet => $id_objet]);
 			if (isset($res['redirect'])) {

formulaires/migrer_albums.php

@@ -116,6 +116,7 @@ function formulaires_migrer_albums_traiter_dist() {
  * @return int nombre d'articles migrés
  */
 function albums_migrer_articles($where_articles, $where_mots, $refuser, $lier_rubriques) {
+	$id_album = null;
 	include_spip('action/editer_objet');
 	include_spip('action/editer_liens');
 
@@ -127,7 +128,7 @@ function albums_migrer_articles($where_articles, $where_mots, $refuser, $lier_ru
 		// y a-t-il deja un album associe ?
 		$liens = objet_trouver_liens(['album' => '*'], ['article' => $row['id_article']]);
 		if (
-			!count($liens)
+			!(is_countable($liens) ? count($liens) : 0)
 			and $id_album = objet_inserer('album')
 		) {
 			// associer tout de suite à l'article
@@ -176,7 +177,7 @@ function albums_migrer_articles($where_articles, $where_mots, $refuser, $lier_ru
 				'id_objet=' . intval($row['id_article'])
 			);
 
-			if (count($docs)) {
+			if (is_countable($docs) ? count($docs) : 0) {
 				$insert = [];
 				foreach ($docs as $doc) {
 					$insert[] = ['id_document' => $doc['id_document'], 'objet' => 'album','id_objet' => $id_album];
@@ -191,7 +192,7 @@ function albums_migrer_articles($where_articles, $where_mots, $refuser, $lier_ru
 					ON (M.id_mot=L.id_mot AND L.objet="article")',
 				'id_objet=' . intval($row['id_article']) . ' AND (' . $where_mots . ')'
 			);
-			if (count($mots)) {
+			if (is_countable($mots) ? count($mots) : 0) {
 				$insert = [];
 				foreach ($mots as $mot) {
 					$insert[] = ['id_mot' => $mot['id_mot'],'objet' => 'album','id_objet' => $id_album];
@@ -207,7 +208,7 @@ function albums_migrer_articles($where_articles, $where_mots, $refuser, $lier_ru
 
 		// refuser l'article si option demandee
 		// meme si c'est un article migre un coup avant
-		if (count($liens) or $id_album) {
+		if (is_countable($liens) ? count($liens) : 0 or $id_album) {
 			if ($refuser) {
 				objet_modifier('article', $row['id_article'], ['statut' => 'refuse']);
 			}

inc/albums.php

@@ -95,14 +95,14 @@ function vider_albums($ids_albums, $supprimer_docs_orphelins = false) {
 		if (
 			$id_album = intval($id_album)
 			and autoriser('modifier', 'album', $id_album)
-			and count(objet_trouver_liens(['document' => '*'], ['album' => $id_album]))
+			and is_countable(objet_trouver_liens(['document' => '*'], ['album' => $id_album])) ? count(objet_trouver_liens(['document' => '*'], ['album' => $id_album])) : 0
 		) {
 			$dissocier_document = charger_fonction('dissocier_document', 'action');
 			$modes_documents = ['I/image','I/document','D/document']; // cf. @note
 			foreach ($modes_documents as $mode) {
 				$dissocier_document("${id_album}-album-${mode}-${supprimer_docs_orphelins}");
 				// dès que l'album est vide, on arrête
-				if (!count(objet_trouver_liens(['document' => '*'], ['album' => $id_album]))) {
+				if (!(is_countable(objet_trouver_liens(['document' => '*'], ['album' => $id_album])) ? count(objet_trouver_liens(['document' => '*'], ['album' => $id_album])) : 0)) {
 					break;
 				}
 			}

modeles/album_fonctions.php

@@ -251,7 +251,7 @@ function calculer_album_params(int $id_album, ?array $contexte = [], ?string $pa
 
 	// On fait qu'une seule fois le travail par modèle
 	static $memory;
-	$hash = hash('md5', $id_album . json_encode($contexte)); // le plus rapide en php 8.1 = xxh3
+	$hash = hash('md5', $id_album . json_encode($contexte, JSON_THROW_ON_ERROR)); // le plus rapide en php 8.1 = xxh3
 	if (isset($memory[$hash])) {
 		return ($param ? ($memory[$hash][$param] ?? null) : $memory[$hash]);
 	}