From ee0757b7e78f5a6c47cad17709422829ee009445 Mon Sep 17 00:00:00 2001
From: Matthieu Marcillaud <marcimat@rezo.net>
Date: Fri, 14 Jan 2022 21:03:43 +0100
Subject: [PATCH] Rector sur Medias avec config PHP 7.4 (hors lib/)
---
action/acceder_document.php | 1 +
action/ajouter_documents.php | 2 +-
action/changer_mode_document.php | 2 +-
action/desordonner_liens_documents.php | 4 ++--
action/dissocier_document.php | 6 +++---
action/editer_document.php | 8 ++++----
action/ordonner_liens_documents.php | 4 ++--
action/supprimer_tous_orphelins.php | 2 +-
action/tourner.php | 4 ++--
formulaires/configurer_documents.php | 2 +-
formulaires/editer_document.php | 1 +
formulaires/joindre_document.php | 2 +-
inc/documents.php | 2 +-
inc/joindre_document.php | 7 ++++---
inc/verifier_taille_document_acceptable.php | 4 ++--
medias_fonctions.php | 4 ++--
medias_pipelines.php | 9 ++++-----
metadata/video.php | 1 +
18 files changed, 34 insertions(+), 31 deletions(-)
diff --git a/action/acceder_document.php b/action/acceder_document.php
index bc22bd78..7e6242c3 100644
--- a/action/acceder_document.php
+++ b/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
diff --git a/action/ajouter_documents.php b/action/ajouter_documents.php
index e3767e73..dfcd925b 100644
--- a/action/ajouter_documents.php
+++ b/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');
diff --git a/action/changer_mode_document.php b/action/changer_mode_document.php
index 3b31918c..b428b352 100644
--- a/action/changer_mode_document.php
+++ b/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;
}
}
diff --git a/action/desordonner_liens_documents.php b/action/desordonner_liens_documents.php
index a77a6192..965bc37d 100644
--- a/action/desordonner_liens_documents.php
+++ b/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');
diff --git a/action/dissocier_document.php b/action/dissocier_document.php
index ea7527a9..d3f4537a 100644
--- a/action/dissocier_document.php
+++ b/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');
diff --git a/action/editer_document.php b/action/editer_document.php
index eca564ed..eefc24a7 100644
--- a/action/editer_document.php
+++ b/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);
diff --git a/action/ordonner_liens_documents.php b/action/ordonner_liens_documents.php
index ca1a3ff3..8aaff03a 100644
--- a/action/ordonner_liens_documents.php
+++ b/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) {
diff --git a/action/supprimer_tous_orphelins.php b/action/supprimer_tous_orphelins.php
index 6e3230b0..d1058a16 100644
--- a/action/supprimer_tous_orphelins.php
+++ b/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
diff --git a/action/tourner.php b/action/tourner.php
index aaeb395b..0dd5784b 100644
--- a/action/tourner.php
+++ b/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');
diff --git a/formulaires/configurer_documents.php b/formulaires/configurer_documents.php
index 82dccdd4..3eba3a46 100644
--- a/formulaires/configurer_documents.php
+++ b/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']);
diff --git a/formulaires/editer_document.php b/formulaires/editer_document.php
index 0758cb17..f9fb065c 100644
--- a/formulaires/editer_document.php
+++ b/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', []);
}
diff --git a/formulaires/joindre_document.php b/formulaires/joindre_document.php
index fdc90153..193217f7 100644
--- a/formulaires/joindre_document.php
+++ b/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);
diff --git a/inc/documents.php b/inc/documents.php
index 7f2a0b38..ff147e55 100644
--- a/inc/documents.php
+++ b/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';
diff --git a/inc/joindre_document.php b/inc/joindre_document.php
index 72ac41ce..d11e4f31 100644
--- a/inc/joindre_document.php
+++ b/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;
}
diff --git a/inc/verifier_taille_document_acceptable.php b/inc/verifier_taille_document_acceptable.php
index 70f80f19..072ab836 100644
--- a/inc/verifier_taille_document_acceptable.php
+++ b/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']);
diff --git a/medias_fonctions.php b/medias_fonctions.php
index 7a41d70d..2f9ba73a 100644
--- a/medias_fonctions.php
+++ b/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',
diff --git a/medias_pipelines.php b/medias_pipelines.php
index a706202e..c58050cf 100644
--- a/medias_pipelines.php
+++ b/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'],
diff --git a/metadata/video.php b/metadata/video.php
index 0f7339a0..a4437b7a 100644
--- a/metadata/video.php
+++ b/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');
--
GitLab