Browse Source

Coding standard

pull/5191/head
Matthieu Marcillaud 2 months ago
parent
commit
5d4b0badda
  1. 2
      ecrire/action/editer_logo.php
  2. 2
      ecrire/exec/admin_plugin.php
  3. 12
      ecrire/inc/config.php
  4. 16
      ecrire/inc/distant.php
  5. 2
      ecrire/inc/flock.php
  6. 4
      ecrire/inc/log.php
  7. 4
      ecrire/inc/utils.php
  8. 2
      ecrire/install/etape_3.php
  9. 30
      ecrire/install/etape_ldap2.php
  10. 4
      ecrire/public/debusquer.php
  11. 4
      prive/objets/liste/auteurs_fonctions.php

2
ecrire/action/editer_logo.php

@ -40,7 +40,7 @@ function logo_supprimer($objet, $id_objet, $etat) {
if ($logo) {
# TODO : deprecated, a supprimer -> anciens logos IMG/artonxx.png pas en base
if ((is_countable($logo) ? count($logo) : 0) < 6) {
spip_log("Supprimer ancien logo " . json_encode($logo, JSON_THROW_ON_ERROR), 'logo');
spip_log('Supprimer ancien logo ' . json_encode($logo, JSON_THROW_ON_ERROR), 'logo');
spip_unlink($logo[0]);
}
elseif (

2
ecrire/exec/admin_plugin.php

@ -120,7 +120,7 @@ function admin_plug_args($quoi, $erreur, $format) {
// message d'erreur au retour d'une operation
if ($erreur) {
echo "<div class='error'>".spip_htmlspecialchars($erreur)."</div>";
echo "<div class='error'>" . spip_htmlspecialchars($erreur) . '</div>';
}
if ($erreur_activation) {
echo "<div class='error'>$erreur_activation</div>";

12
ecrire/inc/config.php

@ -256,14 +256,12 @@ function ecrire_config($cfg, $store) {
}
} // dans tous les autres cas, on ecrase
else {
if (
defined('_MYSQL_NOPLANES')
defined('_MYSQL_NOPLANES')
and _MYSQL_NOPLANES
and !empty($GLOBALS['meta']['charset_sql_connexion'])
and $GLOBALS['meta']['charset_sql_connexion'] == 'utf8'
) {
// detecter si la valeur qu'on veut ecrire a des planes
// @see utf8_noplanes
$serialized_store = (is_string($store) ? $store : serialize($store));
@ -305,9 +303,13 @@ function ecrire_config($cfg, $store) {
// et dans ce cas il faut verifier que l'ecriture en base a bien eu lieu a l'identique si il y a des planes dans la chaine
// car sinon ca casse le serialize PHP - par exemple si on est en mysql utf8 (non mb4)
if ($has_planes) {
$check_store = sql_getfetsel('valeur', 'spip_'.$table, 'nom='.sql_quote($casier));
$check_store = sql_getfetsel('valeur', 'spip_' . $table, 'nom=' . sql_quote($casier));
if ($check_store !== $serialized_store) {
array_walk_recursive($store, function (&$value, $key) {if (is_string($value)) {$value = utf8_noplanes($value);}});
array_walk_recursive($store, function (&$value, $key) {
if (is_string($value)) {
$value = utf8_noplanes($value);
}
});
$serialized_store = serialize($store);
ecrire_meta($casier, $serialized_store, null, $table);
}

16
ecrire/inc/distant.php

@ -131,10 +131,12 @@ function copie_locale($source, $mode = 'auto', $local = null, $taille_max = null
}
// si option valider url, verifions que l'URL finale est acceptable
if ($callback_valider_url
and is_callable($callback_valider_url)
and !$callback_valider_url($res['url'])) {
spip_log("copie_locale : url finale ".$res['url']." non valide, on refuse le fichier $localrac_tmp", 'distant' . _LOG_INFO_IMPORTANTE);
if (
$callback_valider_url
and is_callable($callback_valider_url)
and !$callback_valider_url($res['url'])
) {
spip_log('copie_locale : url finale ' . $res['url'] . " non valide, on refuse le fichier $localrac_tmp", 'distant' . _LOG_INFO_IMPORTANTE);
@unlink($localrac_tmp);
return $t ? $local : false;
}
@ -994,9 +996,11 @@ function recuperer_infos_distantes($source, $options = []) {
// de maniere a attrapper le maximum d'infos (titre, taille, etc). Si
// ca echoue l'utilisateur devra les entrer...
$reponse = recuperer_url($source, ['taille_max' => $taille_max, 'refuser_gz' => true]);
if ($callback_valider_url
if (
$callback_valider_url
and is_callable($callback_valider_url)
and !$callback_valider_url($reponse['url'])) {
and !$callback_valider_url($reponse['url'])
) {
return false;
}
$headers = $reponse['headers'] ?? '';

2
ecrire/inc/flock.php

@ -693,7 +693,7 @@ function sous_repertoire($base, $subdir = '', $nobase = false, $tantpis = false)
function preg_files($dir, $pattern = -1 /* AUTO */, $maxfiles = 10000, $recurs = []) {
$nbfiles = 0;
if ($pattern == -1) {
$pattern = "";
$pattern = '';
}
$fichiers = [];
// revenir au repertoire racine si on a recu dossier/truc

4
ecrire/inc/log.php

@ -40,8 +40,8 @@ function inc_log_dist($message, $logname = null, $logdir = null, $logsuf = null)
// si spip_log() est appelé dans mes_options, toutes les constantes n'ont pas été définies
$logfile =
($logdir ?? (defined('_DIR_LOG') ? _DIR_LOG : _DIR_RACINE . _NOM_TEMPORAIRES_INACCESSIBLES))
.$logname
.($logsuf ?? (defined ('_FILE_LOG_SUFFIX') ? _FILE_LOG_SUFFIX : '.log'));
. $logname
. ($logsuf ?? (defined('_FILE_LOG_SUFFIX') ? _FILE_LOG_SUFFIX : '.log'));
if (!isset($test_repertoire[$d = dirname($logfile)])) {
$test_repertoire[$d] = false; // eviter une recursivite en cas d'erreur de sous_repertoire

4
ecrire/inc/utils.php

@ -3319,8 +3319,8 @@ function exec_info_dist() {
foreach ($cookies_masques as $k) {
if (!empty($_COOKIE[$k])) {
$cookies_backup[$k] = $_COOKIE[$k];
$_SERVER['HTTP_COOKIE'] = str_replace("$k=".$_COOKIE[$k], "$k=$mask", $_SERVER['HTTP_COOKIE'] ?? []);
$_ENV['HTTP_COOKIE'] = str_replace("$k=".$_COOKIE[$k], "$k=$mask", $_ENV['HTTP_COOKIE'] ?? []);
$_SERVER['HTTP_COOKIE'] = str_replace("$k=" . $_COOKIE[$k], "$k=$mask", $_SERVER['HTTP_COOKIE'] ?? []);
$_ENV['HTTP_COOKIE'] = str_replace("$k=" . $_COOKIE[$k], "$k=$mask", $_ENV['HTTP_COOKIE'] ?? []);
$_COOKIE[$k] = $mask;
}
}

2
ecrire/install/etape_3.php

@ -366,7 +366,7 @@ function install_etape_3_dist() {
$hidden = predef_ou_cache($adresse_db, $login_db, $pass_db, $server_db)
. (defined('_INSTALL_NAME_DB') ? ''
: "\n<input type='hidden' name='sel_db' value=\"".spip_htmlspecialchars($sel_db)."\" />");
: "\n<input type='hidden' name='sel_db' value=\"" . spip_htmlspecialchars($sel_db) . '" />');
$auteur_obligatoire = ($ldap_present ? 0 : !sql_countsel('spip_auteurs', '', '', '', $server_db));

30
ecrire/install/etape_ldap2.php

@ -48,7 +48,7 @@ function install_etape_ldap2_dist() {
$adresse_ldap = filter_var($adresse_ldap, FILTER_SANITIZE_URL) ?: '';
$ldap_link = ldap_connect($adresse_ldap, $port_ldap);
$erreur = "ldap_connect(".spip_htmlspecialchars($adresse_ldap).", ".spip_htmlspecialchars($port_ldap).")";
$erreur = 'ldap_connect(' . spip_htmlspecialchars($adresse_ldap) . ', ' . spip_htmlspecialchars($port_ldap) . ')';
if ($ldap_link) {
if (!ldap_set_option($ldap_link, LDAP_OPT_PROTOCOL_VERSION, $protocole_ldap)) {
@ -57,19 +57,19 @@ function install_etape_ldap2_dist() {
}
if ($tls === true) {
if (!ldap_start_tls($ldap_link)) {
$erreur = "ldap_start_tls(".spip_htmlspecialchars($ldap_link)
." ".spip_htmlspecialchars($adresse_ldap)
.", ".spip_htmlspecialchars($port_ldap).")";
$erreur = 'ldap_start_tls(' . spip_htmlspecialchars($ldap_link)
. ' ' . spip_htmlspecialchars($adresse_ldap)
. ', ' . spip_htmlspecialchars($port_ldap) . ')';
$ldap_link = false;
}
}
if ($ldap_link) {
$ldap_link = ldap_bind($ldap_link, $login_ldap, $pass_ldap);
$erreur = "ldap_bind('".spip_htmlspecialchars($ldap_link)
."', '".spip_htmlspecialchars($login_ldap)
."', '".spip_htmlspecialchars($pass_ldap)
."'): ".spip_htmlspecialchars($adresse_ldap)
.", ".spip_htmlspecialchars($port_ldap);
$erreur = "ldap_bind('" . spip_htmlspecialchars($ldap_link)
. "', '" . spip_htmlspecialchars($login_ldap)
. "', '" . spip_htmlspecialchars($pass_ldap)
. "'): " . spip_htmlspecialchars($adresse_ldap)
. ', ' . spip_htmlspecialchars($port_ldap);
}
}
@ -80,12 +80,12 @@ function install_etape_ldap2_dist() {
), _T('info_connexion_ldap_ok');
echo generer_form_ecrire('install', (
"\n<input type='hidden' name='etape' value='ldap3' />"
. "\n<input type='hidden' name='adresse_ldap' value=\"".spip_htmlspecialchars($adresse_ldap)."\" />"
. "\n<input type='hidden' name='port_ldap' value=\"".spip_htmlspecialchars($port_ldap)."\" />"
. "\n<input type='hidden' name='login_ldap' value=\"".spip_htmlspecialchars($login_ldap)."\" />"
. "\n<input type='hidden' name='pass_ldap' value=\"".spip_htmlspecialchars($pass_ldap)."\" />"
. "\n<input type='hidden' name='protocole_ldap' value=\"".spip_htmlspecialchars($protocole_ldap)."\" />"
. "\n<input type='hidden' name='tls_ldap' value=\"".spip_htmlspecialchars($tls_ldap)."\" />"
. "\n<input type='hidden' name='adresse_ldap' value=\"" . spip_htmlspecialchars($adresse_ldap) . '" />'
. "\n<input type='hidden' name='port_ldap' value=\"" . spip_htmlspecialchars($port_ldap) . '" />'
. "\n<input type='hidden' name='login_ldap' value=\"" . spip_htmlspecialchars($login_ldap) . '" />'
. "\n<input type='hidden' name='pass_ldap' value=\"" . spip_htmlspecialchars($pass_ldap) . '" />'
. "\n<input type='hidden' name='protocole_ldap' value=\"" . spip_htmlspecialchars($protocole_ldap) . '" />'
. "\n<input type='hidden' name='tls_ldap' value=\"" . spip_htmlspecialchars($tls_ldap) . '" />'
. bouton_suivant()));
} else {
echo info_etape(_T('titre_connexion_ldap')), info_progression_etape(1, 'etape_ldap', 'install/', true),

4
ecrire/public/debusquer.php

@ -113,8 +113,8 @@ function public_debusquer_dist($message = '', $lieu = '', $opt = []) {
include_spip('inc/filtres');
lang_select($GLOBALS['visiteur_session']['lang']);
$fonc = preg_replace(",\W,", "_", _request('var_mode_objet') ?? '');
$mode = preg_replace(",\W,", "_", _request('var_mode_affiche') ?? '');
$fonc = preg_replace(',\W,', '_', _request('var_mode_objet') ?? '');
$mode = preg_replace(',\W,', '_', _request('var_mode_affiche') ?? '');
$self = str_replace("\\'", '&#39;', self());
$self = parametre_url($self, 'var_mode', 'debug');

4
prive/objets/liste/auteurs_fonctions.php

@ -103,7 +103,7 @@ function afficher_initiale($url, $initiale, $compteur, $debut, $pas) {
and (
$newcompt > $debut or ($newcompt == $debut and $newcompt == $memo['compteur'])
));
$res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . "</li>";
$res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . '</li>';
}
if ($initiale) {
$memo = [
@ -116,7 +116,7 @@ function afficher_initiale($url, $initiale, $compteur, $debut, $pas) {
}
if (!$initiale and !$url) {
if ((is_countable($res) ? count($res) : 0) > 1) {
$out = "<ul class='pagination-items'>" . implode(' ', $res) . "</ul>";
$out = "<ul class='pagination-items'>" . implode(' ', $res) . '</ul>';
}
$memo = null;
$res = [];

Loading…
Cancel
Save