Modifier l'écriture de l'interpolation des variables pour éviter une future dépréciation en PHP8.2

master
Eric Lupinacci 11 months ago
parent 5b5cec63a8
commit a517573acf

@ -60,7 +60,7 @@ function formulaires_peupler_territoires_charger() {
// -- indiquer les territoires déjà chargés
foreach ($valeurs['_pays'] as $_code => $_pays) {
$pays = extraire_multi($_pays);
$valeurs['_pays'][$_code] = "${_code} - ${pays}";
$valeurs['_pays'][$_code] = "{$_code} - {$pays}";
if (territoire_est_peuple('subdivision', $_code)) {
$valeurs['_pays'][$_code] .= ' - <em>[' . _T('territoires:info_territoire_peuple') . ']</em>';
$un_territoire_charge = true;
@ -78,7 +78,7 @@ function formulaires_peupler_territoires_charger() {
foreach ($infrasubdivisions as $_code => $_pays) {
if (territoire_est_peuple('subdivision', $_code)) {
$pays = extraire_multi($_pays);
$valeurs['_pays_infra'][$_code] = "${_code} - ${pays}";
$valeurs['_pays_infra'][$_code] = "{$_code} - {$pays}";
$un_territoire_charge = true;
if (territoire_est_peuple('infrasubdivision', $_code)) {
$valeurs['_pays_infra'][$_code] .= ' - <em>[' . _T('territoires:info_territoire_peuple') . ']</em>';
@ -167,14 +167,14 @@ function formulaires_peupler_territoires_traiter() {
$options['force'] = true;
}
foreach (array('code', 'info') as $_type_extra) {
if (_request("extra_${_type_extra}")) {
if (_request("extra_{$_type_extra}")) {
$options['extras'][] = $_type_extra;
}
}
// On peuple chaque type (ou type,pays pour les subdivisions).
// (La fonction de chargement lance un vidage préalable si le pays demandé est déjà chargée)
$actionner = "territoire_${action}";
$actionner = "territoire_{$action}";
$statut = array();
foreach ($types as $_type) {
// Traitement du type en prenant en compte le cas particulier des subdivisions et infra subdivisions.
@ -268,7 +268,7 @@ function formulaires_peupler_territoires_notifier($type, $messages, $action, $st
// Traitement des succès
if ($statut_global['ok']) {
$messages['message_ok'] .= _T("territoires:msg_${action}_${type}_succes", array('pays' => implode(', ', $variables['ok'])));
$messages['message_ok'] .= _T("territoires:msg_{$action}_{$type}_succes", array('pays' => implode(', ', $variables['ok'])));
if (!$statut_global['sync']) {
$messages['message_ok'] .= '<br />' . _T("territoires:msg_succes_asynchrone");
}
@ -276,11 +276,11 @@ function formulaires_peupler_territoires_notifier($type, $messages, $action, $st
// Traitement des erreurs
if ($statut_global['nok']) {
$messages['message_erreur'] .= _T("territoires:msg_${action}_${type}_erreur", array('pays' => implode(', ', $variables['nok'])));
$messages['message_erreur'] .= _T("territoires:msg_{$action}_{$type}_erreur", array('pays' => implode(', ', $variables['nok'])));
}
if ($statut_global['sha']) {
$messages['message_erreur'] .= $messages['message_erreur'] ? '<br />' : '';
$messages['message_erreur'] .= _T("territoires:msg_${action}_${type}_notice", array('pays' => implode(', ', $variables['sha'])));
$messages['message_erreur'] .= _T("territoires:msg_{$action}_{$type}_notice", array('pays' => implode(', ', $variables['sha'])));
}
return $messages;

@ -41,10 +41,10 @@ function inc_requeter_isocode_dist($url_base, $collection, $filtres, $taille_max
// Calcul de l'url complète
// -- initialisation
$url = "${url_base}/${collection}";
$url = "{$url_base}/{$collection}";
// -- ajout des filtres
foreach ($filtres as $_critere => $_valeur) {
$url .= "&${_critere}=${_valeur}";
$url .= "&{$_critere}={$_valeur}";
}
// Options de la fonction de récupération du JSON
@ -72,7 +72,7 @@ function inc_requeter_isocode_dist($url_base, $collection, $filtres, $taille_max
'extra' => ['url' => $url]
];
// Trace de log
spip_log("API indiponible : ${url}", 'territoires' . _LOG_ERREUR);
spip_log("API indiponible : {$url}", 'territoires' . _LOG_ERREUR);
} else {
try {
// Transformation de la chaîne json reçue en tableau associatif
@ -90,7 +90,7 @@ function inc_requeter_isocode_dist($url_base, $collection, $filtres, $taille_max
]
];
// Trace de log
spip_log("Erreur d'analyse JSON pour l'URL `${url}` : " . $reponse['erreur']['extra']['erreur_json'], 'territoires' . _LOG_ERREUR);
spip_log("Erreur d'analyse JSON pour l'URL `{$url}` : " . $reponse['erreur']['extra']['erreur_json'], 'territoires' . _LOG_ERREUR);
}
}

@ -197,7 +197,7 @@ function territoire_peupler($type, $pays, $options = array()) {
if (type_pays_est_valide($type, $pays)) {
// Lecture de la configuration statique du type de territoire
include_spip('inc/config');
$configuration = lire_config("territoires/${type}");
$configuration = lire_config("territoires/{$type}");
// Identification de la meta et de la variable de consigne
$meta = 'territoires_peuplement';
@ -466,14 +466,14 @@ function territoire_peupler($type, $pays, $options = array()) {
'lic' => isset($collection['credits']) ? $collection['credits'] : array(),
'ext' => $meta_extras,
);
ecrire_config("${meta}/${consigne}", $contenu);
ecrire_config("{$meta}/{$consigne}", $contenu);
$timestamp['fin'] = microtime(true);
$erreur = false;
}
// Log du traitement pour debug
$duree = $timestamp['fin'] - $timestamp['debut'];
spip_log("Les territoires (Type '${type}' - Pays '${pays}') ont été chargées en ${duree} s", 'territoires' . _LOG_INFO_IMPORTANTE);
spip_log("Les territoires (Type '{$type}' - Pays '{$pays}') ont été chargées en {$duree} s", 'territoires' . _LOG_INFO_IMPORTANTE);
if (
defined('_LOG_FILTRE_GRAVITE')
and (_LOG_FILTRE_GRAVITE >= _LOG_DEBUG)
@ -484,7 +484,7 @@ function territoire_peupler($type, $pays, $options = array()) {
$timestamp_fin = $_timestamp;
$duree = ($timestamp_fin - $timestamp_debut) * 1000;
$timestamp_debut = $timestamp_fin;
spip_log("Période ${_periode}: ${duree} ms", 'territoires' . _LOG_DEBUG);
spip_log("Période {$_periode}: {$duree} ms", 'territoires' . _LOG_DEBUG);
}
}
}
@ -510,11 +510,11 @@ function territoire_peupler($type, $pays, $options = array()) {
and $premiere_iteration
) {
$retour['sha'] = true;
spip_log("Les territoires de (Type '${type}' - Pays '${pays}') sont inchangés", 'territoires' . _LOG_AVERTISSEMENT);
spip_log("Les territoires de (Type '{$type}' - Pays '{$pays}') sont inchangés", 'territoires' . _LOG_AVERTISSEMENT);
} else {
$retour['ok'] = false;
if (!$territoires) {
spip_log("Aucun territoire pour (Type '${type}' - Pays '${pays}') retourné par Nomenclatures", 'territoires' . _LOG_ERREUR);
spip_log("Aucun territoire pour (Type '{$type}' - Pays '{$pays}') retourné par Nomenclatures", 'territoires' . _LOG_ERREUR);
}
}
} else {
@ -522,7 +522,7 @@ function territoire_peupler($type, $pays, $options = array()) {
}
} else {
$retour['arg'] = true;
spip_log("Le couple (Type '${type}' - Pays '${pays}') est invalide", 'territoires' . _LOG_ERREUR);
spip_log("Le couple (Type '{$type}' - Pays '{$pays}') est invalide", 'territoires' . _LOG_ERREUR);
}
return $retour;
@ -559,7 +559,7 @@ function territoire_peupler_asynchrone($type, $pays, $options = array()) {
if (type_pays_est_valide($type, $pays)) {
// Construction de la chaine des arguments passés à l'action.
// -- type et pays sont toujours fournis même si le pays est la chaine vide.
$arguments = "${type}:${pays}";
$arguments = "{$type}:{$pays}";
// -- ajout des éventuelles options : on passe les options par leur identifiant
if (!empty($options['force'])) {
$arguments .= ':force';
@ -579,7 +579,7 @@ function territoire_peupler_asynchrone($type, $pays, $options = array()) {
$retour['ok'] = queue_lancer_url_http_async($url);
} else {
$retour['arg'] = true;
spip_log("Le couple (Type '${type}' - Pays '${pays}') est invalide", 'territoires' . _LOG_ERREUR);
spip_log("Le couple (Type '{$type}' - Pays '{$pays}') est invalide", 'territoires' . _LOG_ERREUR);
}
return $retour;
@ -671,28 +671,28 @@ function territoire_depeupler($type, $pays = '', $options = array()) {
// Supprimer la meta propre au pays même si le vidage des liens est en erreur.
include_spip('inc/config');
effacer_config("territoires_peuplement/${consigne}");
effacer_config("territoires_peuplement/{$consigne}");
}
// Enregistrer le succès ou pas du déchargement de la table
if ($sql_ok !== false) {
// Succès complet
$retour['ok'] = true;
spip_log("Les territoires (Type '${type}' - Pays '${pays}') ont été vidés avec succès ainsi que les extras", 'territoires' . _LOG_DEBUG);
spip_log("Les territoires (Type '{$type}' - Pays '{$pays}') ont été vidés avec succès ainsi que les extras", 'territoires' . _LOG_DEBUG);
} else {
spip_log("Les territoires (Type '${type}' - Pays '${pays}') ont été vidés avec succès mais erreur lors du vidage des extras", 'territoires' . _LOG_ERREUR);
spip_log("Les territoires (Type '{$type}' - Pays '{$pays}') ont été vidés avec succès mais erreur lors du vidage des extras", 'territoires' . _LOG_ERREUR);
}
} else {
spip_log("Erreur de vidage des territoires (Type '${type}' - Pays '${pays}')", 'territoires' . _LOG_ERREUR);
spip_log("Erreur de vidage des territoires (Type '{$type}' - Pays '{$pays}')", 'territoires' . _LOG_ERREUR);
}
}
} else {
$retour['sha'] = true;
spip_log("Aucun territoire (Type '${type}' - Pays '${pays}') à vider", 'territoires' . _LOG_AVERTISSEMENT);
spip_log("Aucun territoire (Type '{$type}' - Pays '{$pays}') à vider", 'territoires' . _LOG_AVERTISSEMENT);
}
} else {
$retour['arg'] = true;
spip_log("Le couple (Type '${type}' - Pays '${pays}') est invalide", 'territoires' . _LOG_ERREUR);
spip_log("Le couple (Type '{$type}' - Pays '{$pays}') est invalide", 'territoires' . _LOG_ERREUR);
}
return $retour;

@ -57,7 +57,7 @@ function sha_est_identique($sha, $meta, $consigne) {
// On récupère le sha de la table dans les metas si il existe (ie. la table a été chargée)
include_spip('inc/config');
$sha_stocke = lire_config("${meta}/${consigne}/sha", false);
$sha_stocke = lire_config("{$meta}/{$consigne}/sha", false);
if ($sha_stocke and ($sha === $sha_stocke)) {
$sha_identique = true;

@ -46,7 +46,7 @@ function territoires_acquerir($type, $pays = '', $options = array()) {
include_spip('inc/territoires_utils')
and type_est_subdivision($type)
) {
$collection = "${type}s";
$collection = "{$type}s";
if ($pays) {
$filtres = array('pays' => $pays);
}
@ -284,7 +284,7 @@ function enregistrement_initialiser($territoire, $type, $pays = '') {
// Traduire les champs de Isocode en champs pour Territoires
include_spip('inc/config');
$enregistrement = array();
$champs = lire_config("territoires/${type}/champs/base");
$champs = lire_config("territoires/{$type}/champs/base");
foreach ($champs as $_champ_api => $_champ_territoire) {
$enregistrement[$_champ_territoire] = $territoire[$_champ_api];
}
@ -429,6 +429,6 @@ function type_est_subdivision($type) {
function consigne_identifier($type, $pays = '') {
return (type_est_subdivision($type))
? "${type}/${pays}"
? "{$type}/{$pays}"
: $type;
}

Loading…
Cancel
Save