Browse Source

Rector sur prive/ avec config PHP 7.4

pull/4991/head
Matthieu Marcillaud 1 week ago
parent
commit
2325f0e17d
  1. 4
      prive/formulaires/configurer_identite.php
  2. 8
      prive/formulaires/configurer_multilinguisme.php
  3. 4
      prive/formulaires/configurer_preferences.php
  4. 4
      prive/formulaires/configurer_reducteur.php
  5. 11
      prive/formulaires/dater.php
  6. 6
      prive/formulaires/declarer_bases.php
  7. 9
      prive/formulaires/editer_auteur.php
  8. 16
      prive/formulaires/editer_liens.php
  9. 2
      prive/formulaires/editer_logo.php
  10. 4
      prive/formulaires/instituer_objet.php
  11. 11
      prive/formulaires/login.php
  12. 2
      prive/formulaires/recherche_ecrire.php
  13. 2
      prive/objets/liste/auteurs_fonctions.php
  14. 2
      prive/themes/spip/vars.css_fonctions.php

4
prive/formulaires/configurer_identite.php

@ -21,7 +21,7 @@ function formulaires_configurer_identite_charger_dist() {
$valeurs = [];
foreach (['nom_site', 'adresse_site', 'slogan_site', 'descriptif_site', 'email_webmaster'] as $k) {
$valeurs[$k] = isset($GLOBALS['meta'][$k]) ? $GLOBALS['meta'][$k] : '';
$valeurs[$k] = $GLOBALS['meta'][$k] ?? '';
}
return $valeurs;
@ -46,7 +46,7 @@ function formulaires_configurer_identite_verifier_dist() {
function formulaires_configurer_identite_traiter_dist() {
include_spip('inc/config');
$adresse_site = isset($GLOBALS['meta']['adresse_site']) ? $GLOBALS['meta']['adresse_site'] : '';
$adresse_site = $GLOBALS['meta']['adresse_site'] ?? '';
if (_request('adresse_site') != $adresse_site) {
refuser_traiter_formulaire_ajax();
}

8
prive/formulaires/configurer_multilinguisme.php

@ -18,17 +18,17 @@ function formulaires_configurer_multilinguisme_charger_dist() {
$valeurs = [];
$valeurs['multi_secteurs'] = $GLOBALS['meta']['multi_secteurs'];
foreach (['multi_objets', 'gerer_trad_objets'] as $m) {
$valeurs[$m] = explode(',', isset($GLOBALS['meta'][$m]) ? $GLOBALS['meta'][$m] : '');
$valeurs[$m] = explode(',', $GLOBALS['meta'][$m] ?? '');
}
if (
count($valeurs['multi_objets'])
is_countable($valeurs['multi_objets']) ? count($valeurs['multi_objets']) : 0
or count(explode(',', $GLOBALS['meta']['langues_utilisees'])) > 1
) {
$selection = (is_null(_request('multi_objets')) ?
explode(',', $GLOBALS['meta']['langues_multilingue']) : _request('langues_auth'));
$valeurs['_langues'] = saisie_langues_utiles('langues_auth', $selection ? $selection : []);
$valeurs['_nb_langues_selection'] = count($selection);
$valeurs['_langues'] = saisie_langues_utiles('langues_auth', $selection ?: []);
$valeurs['_nb_langues_selection'] = is_countable($selection) ? count($selection) : 0;
}
return $valeurs;

4
prive/formulaires/configurer_preferences.php

@ -36,8 +36,8 @@ function formulaires_configurer_preferences_charger_dist() {
lire_metas();
$valeurs = [];
$valeurs['display_navigation'] = isset($GLOBALS['visiteur_session']['prefs']['display_navigation']) ? $GLOBALS['visiteur_session']['prefs']['display_navigation'] : 'navigation_avec_icones';
$valeurs['display_outils'] = isset($GLOBALS['visiteur_session']['prefs']['display_outils']) ? $GLOBALS['visiteur_session']['prefs']['display_outils'] : 'oui';
$valeurs['display_navigation'] = $GLOBALS['visiteur_session']['prefs']['display_navigation'] ?? 'navigation_avec_icones';
$valeurs['display_outils'] = $GLOBALS['visiteur_session']['prefs']['display_outils'] ?? 'oui';
$valeurs['display'] = (isset($GLOBALS['visiteur_session']['prefs']['display']) and $GLOBALS['visiteur_session']['prefs']['display'] > 0) ? $GLOBALS['visiteur_session']['prefs']['display'] : 2;
$valeurs['couleur'] = (isset($GLOBALS['visiteur_session']['prefs']['couleur']) and $GLOBALS['visiteur_session']['prefs']['couleur'] > 0) ? $GLOBALS['visiteur_session']['prefs']['couleur'] : 1;

4
prive/formulaires/configurer_reducteur.php

@ -37,7 +37,7 @@ function formulaires_configurer_reducteur_charger_dist() {
'taille_preview',
] as $m
) {
$valeurs[$m] = isset($GLOBALS['meta'][$m]) ? $GLOBALS['meta'][$m] : null;
$valeurs[$m] = $GLOBALS['meta'][$m] ?? null;
}
$valeurs['taille_preview'] = intval($valeurs['taille_preview']);
@ -136,7 +136,7 @@ function url_vignette_choix($process) {
}
break;
case 'imagick':
if (!method_exists('Imagick', 'readImage')) {
if (!method_exists(\Imagick::class, 'readImage')) {
return '';
}
break;

11
prive/formulaires/dater.php

@ -45,6 +45,11 @@ if (!defined('_ECRIRE_INC_VERSION')) {
**/
function formulaires_dater_charger_dist($objet, $id_objet, $retour = '', $options = []) {
$jour = null;
$mois = null;
$annee = null;
$heure = null;
$minute = null;
$objet = objet_type($objet);
if (!$objet or !intval($id_objet)) {
return false;
@ -63,7 +68,7 @@ function formulaires_dater_charger_dist($objet, $id_objet, $retour = '', $option
return false;
}
$champ_date = $desc['date'] ? $desc['date'] : 'date';
$champ_date = $desc['date'] ?: 'date';
if (isset($options['champ_date']) and $options['champ_date']) {
$champ_date = $options['champ_date'];
}
@ -92,7 +97,7 @@ function formulaires_dater_charger_dist($objet, $id_objet, $retour = '', $option
$row = sql_fetsel($select, $desc['table'], "$_id_objet=" . intval($id_objet));
$statut = isset($row['statut']) ? $row['statut'] : 'publie'; // pas de statut => publie
$statut = $row['statut'] ?? 'publie'; // pas de statut => publie
$valeurs['editable'] = autoriser('dater', $objet, $id_objet, null, ['statut' => $statut]);
@ -289,7 +294,7 @@ function formulaires_dater_traiter_dist($objet, $id_objet, $retour = '', $option
return ['message_erreur' => _L('erreur')];
} #impossible en principe
$champ_date = $desc['date'] ? $desc['date'] : 'date';
$champ_date = $desc['date'] ?: 'date';
if (isset($options['champ_date']) and $options['champ_date']) {
$champ_date = $options['champ_date'];
}

6
prive/formulaires/declarer_bases.php

@ -17,7 +17,7 @@ if (!defined('_ECRIRE_INC_VERSION')) {
include_spip('inc/install');
function formulaires_declarer_bases_charger_dist() {
list($adresse_db, $login_db, $pass_db, $sel, $server_db) = analyse_fichier_connection(_FILE_CONNECT);
[$adresse_db, $login_db, $pass_db, $sel, $server_db] = analyse_fichier_connection(_FILE_CONNECT);
$deja = bases_referencees(_FILE_CONNECT);
// proposer un nom de connect si pas encore saisi
@ -104,7 +104,7 @@ function liste_bases($server_db) {
function formulaires_declarer_bases_verifier_1_dist() {
$erreurs = [];
list($def_adresse_db, $def_login_db, $def_pass_db, $sel_db, $def_serveur_db) = analyse_fichier_connection(_FILE_CONNECT);
[$def_adresse_db, $def_login_db, $def_pass_db, $sel_db, $def_serveur_db] = analyse_fichier_connection(_FILE_CONNECT);
if (!$adresse_db = _request('adresse_db')) {
if (defined('_INSTALL_HOST_DB')) {
@ -205,7 +205,7 @@ function formulaires_declarer_bases_traiter_dist() {
$adresse_db = _DECLARER_ADRESSE_DB;
if (preg_match(',(.*):(.*),', $adresse_db, $r)) {
list(, $adresse_db, $port) = $r;
[, $adresse_db, $port] = $r;
} else {
$port = '';
}

9
prive/formulaires/editer_auteur.php

@ -171,7 +171,7 @@ function formulaires_editer_auteur_verifier_dist(
}
$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
$auth_methode = ($auth_methode ? $auth_methode : 'spip');
$auth_methode = ($auth_methode ?: 'spip');
include_spip('inc/auth');
if (!nom_acceptable(_request('nom'))) {
@ -223,7 +223,7 @@ function formulaires_editer_auteur_verifier_dist(
}
// quand c'est un auteur existant on fait le reset password ici
if (!count($erreurs) and _request('reset_password') and intval($id_auteur)) {
if (!(is_countable($erreurs) ? count($erreurs) : 0) and _request('reset_password') and intval($id_auteur)) {
$erreurs = auteur_reset_password($id_auteur, $erreurs);
return $erreurs;
}
@ -321,8 +321,9 @@ function formulaires_editer_auteur_traiter_dist(
$row = [],
$hidden = ''
) {
$id_objet = null;
if (_request('saisie_webmestre') or _request('webmestre')) {
set_request('webmestre', _request('webmestre') ? _request('webmestre') : 'non');
set_request('webmestre', _request('webmestre') ?: 'non');
}
$retour = parametre_url($retour, 'email_confirm', '');
@ -402,7 +403,7 @@ function formulaires_editer_auteur_traiter_dist(
$objet = 'article';
$id_objet = intval($associer_objet);
} elseif (preg_match(',^\w+\|[0-9]+$,', $associer_objet)) {
list($objet, $id_objet) = explode('|', $associer_objet);
[$objet, $id_objet] = explode('|', $associer_objet);
}
if ($objet and $id_objet and autoriser('modifier', $objet, $id_objet)) {
include_spip('action/editer_auteur');

16
prive/formulaires/editer_liens.php

@ -89,7 +89,7 @@ function formulaires_editer_liens_charger_dist($a, $b, $c, $options = []) {
$editable = $options['editable'];
list($table_source, $objet, $id_objet, $objet_lien) = determine_source_lien_objet($a, $b, $c);
[$table_source, $objet, $id_objet, $objet_lien] = determine_source_lien_objet($a, $b, $c);
if (!$table_source or !$objet or !$objet_lien or !$id_objet) {
return false;
}
@ -220,7 +220,7 @@ function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = []) {
$editable = $options['editable'];
$res = ['editable' => $editable ? true : false];
list($table_source, $objet, $id_objet, $objet_lien) = determine_source_lien_objet($a, $b, $c);
[$table_source, $objet, $id_objet, $objet_lien] = determine_source_lien_objet($a, $b, $c);
if (!$table_source or !$objet or !$objet_lien) {
return $res;
}
@ -296,7 +296,7 @@ function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = []) {
foreach ($supprimer as $k => $v) {
if ($lien = lien_verifier_action($k, $v)) {
$lien = explode('-', $lien);
list($objet_source, $ids, $objet_lie, $idl, $role) = array_pad($lien, 5, null);
[$objet_source, $ids, $objet_lie, $idl, $role] = array_pad($lien, 5, null);
// appliquer une condition sur le rôle si défini ('*' pour tous les roles)
$cond = (!is_null($role) ? ['role' => $role] : []);
if ($objet_lien == $objet_source) {
@ -329,7 +329,7 @@ function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = []) {
foreach ($ajouter as $k => $v) {
if ($lien = lien_verifier_action($k, $v)) {
$ajout_ok = true;
list($objet1, $ids, $objet2, $idl) = explode('-', $lien);
[$objet1, $ids, $objet2, $idl] = explode('-', $lien);
$qualifs = lien_retrouver_qualif($objet_lien, $lien);
if ($objet_lien == $objet1) {
lien_ajouter_liaisons($objet1, $ids, $objet2, $idl, $qualifs);
@ -352,7 +352,7 @@ function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = []) {
include_spip('action/editer_liens');
foreach ($ordonner as $k => $rang_lien) {
if ($lien = lien_verifier_action($k, '')) {
list($objet1, $ids, $objet2, $idl) = explode('-', $lien);
[$objet1, $ids, $objet2, $idl] = explode('-', $lien);
$qualif = ['rang_lien' => $rang_lien];
if ($objet_lien == $objet1) {
@ -426,7 +426,7 @@ function lien_verifier_action($k, $v) {
function lien_retrouver_qualif($objet_lien, $lien) {
// un role est défini dans la liaison
$defs = explode('-', $lien);
list($objet1, , $objet2, , $role) = array_pad($defs, 5, null);
[$objet1, , $objet2, , $role] = array_pad($defs, 5, null);
if ($objet_lien == $objet1) {
$colonne_role = roles_colonne($objet1, $objet2);
} else {
@ -464,12 +464,12 @@ function lien_retrouver_qualif($objet_lien, $lien) {
// $qualif de la forme array(role=>array(...),valeur=>array(...),....)
// on le reforme en array(array(role=>..,valeur=>..,..),array(role=>..,valeur=>..,..),...)
$qualifs = [];
while (count($qualif)) {
while (is_countable($qualif) ? count($qualif) : 0) {
$q = [];
foreach ($qualif as $att => $values) {
if (is_array($values)) {
$q[$att] = array_shift($qualif[$att]);
if (!count($qualif[$att])) {
if (!(is_countable($qualif[$att]) ? count($qualif[$att]) : 0)) {
unset($qualif[$att]);
}
} else {

2
prive/formulaires/editer_logo.php

@ -256,7 +256,7 @@ function formulaires_editer_logo_traiter_dist($objet, $id_objet, $retour = '', $
*/
function formulaire_editer_logo_get_sources() {
if (!$_FILES) {
$_FILES = isset($GLOBALS['HTTP_POST_FILES']) ? $GLOBALS['HTTP_POST_FILES'] : [];
$_FILES = $GLOBALS['HTTP_POST_FILES'] ?? [];
}
if (!is_array($_FILES)) {
return [];

4
prive/formulaires/instituer_objet.php

@ -105,8 +105,8 @@ function formulaires_instituer_objet_charger_dist($objet, $id_objet, $retour = '
'_id_objet' => $id_objet,
'_statuts' => $statuts,
'_publiable' => $publiable,
'_label' => isset($desc['texte_changer_statut']) ? $desc['texte_changer_statut'] : 'texte_article_statut',
'_aide' => isset($desc['aide_changer_statut']) ? $desc['aide_changer_statut'] : '',
'_label' => $desc['texte_changer_statut'] ?? 'texte_article_statut',
'_aide' => $desc['aide_changer_statut'] ?? '',
'_hidden' => "<input type='hidden' name='statut_old' value='" . $v['statut'] . "' />",
];

11
prive/formulaires/login.php

@ -33,7 +33,7 @@ include_spip('base/abstract_sql');
function is_url_prive($cible) {
include_spip('inc/filtres_mini');
$path = parse_url(tester_url_absolue($cible) ? $cible : url_absolue($cible));
$path = (isset($path['path']) ? $path['path'] : '');
$path = ($path['path'] ?? '');
return strncmp(substr($path, -strlen(_DIR_RESTREINT_ABS)), _DIR_RESTREINT_ABS, strlen(_DIR_RESTREINT_ABS)) == 0;
}
@ -112,12 +112,12 @@ function formulaires_login_charger_dist($cible = '', $options = [], $deprecated
$valeurs = [
'var_login' => $login,
'editable' => !$row,
'cnx' => isset($row['cnx']) ? $row['cnx'] : '0',
'cnx' => $row['cnx'] ?? '0',
'auth_http' => login_auth_http(),
'rester_connecte' => ((_RENOUVELLE_ALEA < 12 * 3600) ? '' : ' '),
'_logo' => isset($row['logo']) ? $row['logo'] : '',
'_alea_actuel' => isset($row['alea_actuel']) ? $row['alea_actuel'] : '',
'_alea_futur' => isset($row['alea_futur']) ? $row['alea_futur'] : '',
'_logo' => $row['logo'] ?? '',
'_alea_actuel' => $row['alea_actuel'] ?? '',
'_alea_futur' => $row['alea_futur'] ?? '',
'_pipeline' => 'affiche_formulaire_login', // faire passer le formulaire dans un pipe dedie pour les methodes auth
'_autofocus' => ($options['autofocus'] and $options['autofocus'] !== 'non') ? ' ' : '',
];
@ -218,6 +218,7 @@ function login_auth_http() {
**/
function formulaires_login_verifier_dist($cible = '', $options = [], $deprecated = null) {
$erreurs = [];
if (!is_array($options)) {
$options = [
'login' => $options,

2
prive/formulaires/recherche_ecrire.php

@ -41,7 +41,7 @@ function formulaires_recherche_ecrire_charger_dist($action = '', $class = '') {
return
[
'action' => ($action ? $action : generer_url_ecrire('recherche')),
'action' => ($action ?: generer_url_ecrire('recherche')),
# action specifique, ne passe pas par Verifier, ni Traiter
'recherche' => _request('recherche'),
'lang' => $lang,

2
prive/objets/liste/auteurs_fonctions.php

@ -115,7 +115,7 @@ function afficher_initiale($url, $initiale, $compteur, $debut, $pas) {
}
}
if (!$initiale and !$url) {
if (count($res) > 1) {
if ((is_countable($res) ? count($res) : 0) > 1) {
$out = implode(' ', $res);
}
$memo = null;

2
prive/themes/spip/vars.css_fonctions.php

@ -5,7 +5,7 @@
* @internal
*/
class Spip_Css_Vars_Collection {
private $vars = [];
private array $vars = [];
public function add(string $var, string $value) {
$this->vars[$var] = $value;

Loading…
Cancel
Save