Rector sur Medias avec config PHP 7.4 (hors lib/)

action/acceder_document.php

@@ -23,6 +23,7 @@ include_spip('inc/headers');
 
 // https://code.spip.net/@action_acceder_document_dist
 function action_acceder_document_dist() {
+	$doc = [];
 	include_spip('inc/documents');
 
 	// $file exige pour eviter le scan id_document par id_document

action/ajouter_documents.php

@@ -100,7 +100,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
 		define('_TITRER_DOCUMENTS', false);
 	}
 
-	$titrer = isset($file['titrer']) ? $file['titrer'] : _TITRER_DOCUMENTS;
+	$titrer = $file['titrer'] ?? _TITRER_DOCUMENTS;
 	$mode = ((isset($file['mode']) and $file['mode']) ? $file['mode'] : $mode);
 
 	include_spip('inc/modifier');

action/changer_mode_document.php

@@ -33,7 +33,7 @@ function action_changer_mode_document_dist($id_document = null, $mode = null) {
 			spip_log("action_changer_mode_document $arg pas compris");
 		} else {
 			array_shift($r);
-			list($id_document, $mode) = $r;
+			[$id_document, $mode] = $r;
 		}
 	}
 

action/desordonner_liens_documents.php

@@ -40,7 +40,7 @@ function action_desordonner_liens_documents_dist($arg = null) {
 	}
 
 	$arg = explode('-', $arg);
-	list($id_objet, $objet, $document) = $arg;
+	[$id_objet, $objet, $document] = $arg;
 
 	if (
 		$id_objet = intval($id_objet)
@@ -71,7 +71,7 @@ function desordonner_liens_documents($document, $objet, $id_objet) {
 	if ($id_document = intval($document)) {
 		desordonner_liens_document($id_document, $objet, $id_objet);
 	} else {
-		list($image, $mode) = explode('/', $document);
+		[$image, $mode] = explode('/', $document);
 		$image = ($image == 'I');
 		$typdoc = sql_in('docs.extension', ['gif', 'jpg', 'png'], $image ? '' : 'NOT');
 

action/dissocier_document.php

@@ -42,11 +42,11 @@ function action_dissocier_document_dist($arg = null) {
 	// attention au cas ou id_objet est negatif !
 	if (strncmp($arg, '-', 1) == 0) {
 		$arg = explode('-', substr($arg, 1));
-		list($id_objet, $objet, $document) = $arg;
+		[$id_objet, $objet, $document] = $arg;
 		$id_objet = -$id_objet;
 	} else {
 		$arg = explode('-', $arg);
-		list($id_objet, $objet, $document) = $arg;
+		[$id_objet, $objet, $document] = $arg;
 	}
 
 	$suppr = $check = false;
@@ -164,7 +164,7 @@ function dissocier_document($document, $objet, $id_objet, $supprime = false, $ch
 	if ($id_document = intval($document)) {
 		supprimer_lien_document($id_document, $objet, $id_objet, $supprime, $check);
 	} else {
-		list($image, $mode) = explode('/', $document);
+		[$image, $mode] = explode('/', $document);
 		$image = ($image == 'I');
 		$typdoc = sql_in('docs.extension', ['gif', 'jpg', 'png'], $image ? '' : 'NOT');
 

action/editer_document.php

@@ -179,8 +179,8 @@ function document_modifier($id_document, $set = null) {
  */
 function document_instituer($id_document, $champs = []) {
 
-	$statut = isset($champs['statut']) ? $champs['statut'] : null;
-	$date_publication = isset($champs['date_publication']) ? $champs['date_publication'] : null;
+	$statut = $champs['statut'] ?? null;
+	$date_publication = $champs['date_publication'] ?? null;
 	if (isset($champs['parents'])) {
 		medias_revision_document_parents($id_document, $champs['parents']);
 	}
@@ -233,7 +233,7 @@ function document_instituer($id_document, $champs = []) {
 		]
 	);
 
-	if (!count($champs)) {
+	if (!(is_countable($champs) ? count($champs) : 0)) {
 		return false;
 	}
 
@@ -244,7 +244,7 @@ function document_instituer($id_document, $champs = []) {
 			'spip_documents_liens',
 			"objet='rubrique' AND id_document=" . intval($id_document)
 		);
-		if (count($publier_rubriques)) {
+		if (is_countable($publier_rubriques) ? count($publier_rubriques) : 0) {
 			include_spip('inc/rubriques');
 			foreach ($publier_rubriques as $r) {
 				calculer_rubriques_if($r['id_objet'], ['statut' => $champs['statut']], $statut_ancien, false);

action/ordonner_liens_documents.php

@@ -42,7 +42,7 @@ function action_ordonner_liens_dist() {
 		return envoyer_json_erreur(_T('medias:erreur_autorisation') . ' ' . _T('medias:erreur_deplacement_impossible'));
 	}
 
-	list($_id_objet, $table_liens) = objet_associable($objet);
+	[$_id_objet, $table_liens] = objet_associable($objet);
 
 	$success = $errors = [];
 
@@ -93,7 +93,7 @@ function action_ordonner_liens_dist() {
 
 function envoyer_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 envoyer_json_erreur($msg) {

action/supprimer_tous_orphelins.php

@@ -22,7 +22,7 @@ function action_supprimer_tous_orphelins() {
 	$arg = $securiser_action();
 
 	//on recupere le contexte pour ne supprimer les orphelins que de ce dernier
-	list($media, $distant, $statut, $sanstitre) = explode('/', $arg);
+	[$media, $distant, $statut, $sanstitre] = explode('/', $arg);
 
 	$where = [];
 	//critere sur le media

action/tourner.php

@@ -36,7 +36,7 @@ function action_tourner_dist($id_document = null, $angle = null) {
 			spip_log("action_tourner_dist $arg pas compris");
 		} else {
 			array_shift($r);
-			list($id_document, $angle) = $r;
+			[$id_document, $angle] = $r;
 		}
 	}
 	if ($id_document and autoriser('modifier', 'document', $id_document)) {
@@ -90,7 +90,7 @@ function action_tourner_post($id_document, $angle) {
 			$res = filtrer('image_format', $res, $row['extension']);
 		}
 
-		list($hauteur, $largeur) = taille_image($res);
+		[$hauteur, $largeur] = taille_image($res);
 		$res = extraire_attribut($res, 'src');
 
 		include_spip('inc/getdocument');

formulaires/configurer_documents.php

@@ -34,7 +34,7 @@ function formulaires_configurer_documents_charger_dist() {
 			'documents_date',
 		] as $m
 	) {
-		$valeurs[$m] = isset($GLOBALS['meta'][$m]) ? $GLOBALS['meta'][$m] : '';
+		$valeurs[$m] = $GLOBALS['meta'][$m] ?? '';
 	}
 	$valeurs['documents_objets'] = explode(',', $valeurs['documents_objets']);
 

formulaires/editer_document.php

@@ -163,6 +163,7 @@ function formulaires_editer_document_traiter_dist(
 	$row = [],
 	$hidden = ''
 ) {
+	$rename = null;
 	if (is_null(_request('parents'))) {
 		set_request('parents', []);
 	}

formulaires/joindre_document.php

@@ -221,7 +221,7 @@ function formulaires_joindre_document_verifier_dist(
 					and !_request('joindre_zip')
 					and $contenu_zip = joindre_verifier_zip($files)
 				) {
-					list($fichiers, $erreurs, $tmp_zip) = $contenu_zip;
+					[$fichiers, $erreurs, $tmp_zip] = str_split($contenu_zip);
 					if ($fichiers) {
 						// on passe le md5 du fichier uniquement, on le retrouvera dans zip_to_clean de la session
 						$token_zip = md5($tmp_zip);

inc/documents.php

@@ -230,7 +230,7 @@ function affiche_raccourci_doc($doc, $id, $align = '', $short = false) {
 	$model = "<$doc$id$pipe>";
 	$text = $model;
 	if ($short) {
-		$text = $align ? $align : $model;
+		$text = $align ?: $model;
 	}
 
 	$classes = 'btn btn_link btn_mini';

inc/joindre_document.php

@@ -28,7 +28,7 @@ function joindre_trouver_fichier_envoye() {
 	// on est appele deux fois dans un hit, resservir ce qu'on a trouve a la verif
 	// lorsqu'on est appelle au traitement
 
-	if (count($files)) {
+	if (is_countable($files) ? count($files) : 0) {
 		return $files;
 	}
 
@@ -151,7 +151,7 @@ function joindre_trouver_fichier_envoye() {
  *   string en cas d'erreur
  */
 function joindre_trouver_http_post_files($name = null) {
-	$post = isset($_FILES) ? $_FILES : $GLOBALS['HTTP_POST_FILES'];
+	$post = $_FILES ?? $GLOBALS['HTTP_POST_FILES'];
 	$files = [];
 	if (is_array($post)) {
 		foreach ($post as $input_name => $file) {
@@ -159,7 +159,7 @@ function joindre_trouver_http_post_files($name = null) {
 				continue;
 			}
 			if (is_array($file['name'])) {
-				while (count($file['name'])) {
+				while (is_countable($file['name']) ? count($file['name']) : 0) {
 					$test = [
 						'input_name' => $input_name,
 						'error' => array_shift($file['error']),
@@ -200,6 +200,7 @@ function joindre_trouver_http_post_files($name = null) {
 // pour les autres erreurs renvoie le message d'erreur
 function joindre_upload_error($error) {
 
+	$msg = null;
 	if (!$error) {
 		return false;
 	}

inc/verifier_taille_document_acceptable.php

@@ -93,13 +93,13 @@ function medias_verifier_largeur_hauteur_image(&$infos, $max_width = null, $max_
 		// on met directement a la taille maxi a la volee
 		if (isset($GLOBALS['meta']['creer_preview']) and $GLOBALS['meta']['creer_preview'] == 'oui') {
 			include_spip('inc/filtres');
-			$img = filtrer('image_reduire', $infos['fichier'], $max_width ? $max_width : '*', $max_height ? $max_height : '*');
+			$img = filtrer('image_reduire', $infos['fichier'], $max_width ?: '*', $max_height ?: '*');
 			$img = extraire_attribut($img, 'src');
 			$img = supprimer_timestamp($img);
 			if (@file_exists($img) and $img !== $infos['fichier']) {
 				spip_unlink($infos['fichier']);
 				@rename($img, $infos['fichier']);
-				list($h, $w) = taille_image($infos['fichier'], true);
+				[$h, $w] = taille_image($infos['fichier'], true);
 				$infos['largeur'] = $w;
 				$infos['hauteur'] = $h;
 				$infos['taille'] = @filesize($infos['fichier']);

medias_fonctions.php

@@ -330,8 +330,8 @@ function medias_lister_methodes_upload($env) {
 	$methodes['distant'] = ['label_lien' => _T('medias:bouton_download_sur_le_web'),'label_bouton' => _T('bouton_choisir')];
 
 	// pipeline pour les méthodes d'upload
-	$objet = isset($env['objet']) ? $env['objet'] : '';
-	$id_objet = isset($env['id_objet']) ? $env['id_objet'] : '';
+	$objet = $env['objet'] ?? '';
+	$id_objet = $env['id_objet'] ?? '';
 
 	$methodes = pipeline(
 		'medias_methodes_upload',

medias_pipelines.php

@@ -65,7 +65,7 @@ function medias_post_insertion($flux) {
 
 	$objet = objet_type($flux['args']['table']);
 	$id_objet = $flux['args']['id_objet'];
-	$id_auteur = isset($GLOBALS['visiteur_session']['id_auteur']) ? $GLOBALS['visiteur_session']['id_auteur'] : 0;
+	$id_auteur = $GLOBALS['visiteur_session']['id_auteur'] ?? 0;
 
 	include_spip('inc/autoriser');
 
@@ -132,7 +132,7 @@ function medias_configurer_liste_metas($config) {
  **/
 function medias_post_edition($flux) {
 	// le serveur n'est pas toujours la
-	$serveur = (isset($flux['args']['serveur']) ? $flux['args']['serveur'] : '');
+	$serveur = ($flux['args']['serveur'] ?? '');
 	// si on ajoute un document, mettre son statut a jour
 	if (isset($flux['args']['action']) and $flux['args']['action'] == 'ajouter_document') {
 		include_spip('action/editer_document');
@@ -140,12 +140,11 @@ function medias_post_edition($flux) {
 		document_instituer($flux['args']['id_objet']);
 	} // si on institue un objet, mettre ses documents lies a jour
 	elseif (isset($flux['args']['table']) and $flux['args']['table'] !== 'spip_documents') {
-		$type = isset($flux['args']['type']) ? $flux['args']['type'] : objet_type($flux['args']['table']);
+		$type = $flux['args']['type'] ?? objet_type($flux['args']['table']);
 		// verifier d'abord les doublons !
 		include_spip('inc/autoriser');
 		if (autoriser('autoassocierdocument', $type, $flux['args']['id_objet'])) {
-			$table_objet = isset($flux['args']['table_objet']) ?
-				$flux['args']['table_objet'] : table_objet($flux['args']['table'], $serveur);
+			$table_objet = $flux['args']['table_objet'] ?? table_objet($flux['args']['table'], $serveur);
 			$marquer_doublons_doc = charger_fonction('marquer_doublons_doc', 'inc');
 			$marquer_doublons_doc(
 				$flux['data'],

metadata/video.php

@@ -25,6 +25,7 @@ if (!defined('_ECRIRE_INC_VERSION')) {
  *    Le tableau comprenant les différentes metas à mettre en base
  */
 function metadata_video($file) {
+	$id3 = [];
 	$meta = [];
 
 	include_spip('lib/getid3/getid3');