Browse Source

style: Coding Standard (phpcbf)

pull/5260/head
Matthieu Marcillaud 8 months ago
parent
commit
31d75f8a58
  1. 5
      ecrire/auth/spip.php
  2. 2
      ecrire/inc/drapeau_edition.php
  3. 6
      ecrire/inc/filtres_images_lib_mini.php
  4. 4
      ecrire/inc/modifier.php
  5. 2
      ecrire/public/composer.php
  6. 2
      ecrire/src/Chiffrer/SpipCles.php
  7. 13
      prive/formulaires/editer_auteur.php

5
ecrire/auth/spip.php

@ -74,7 +74,6 @@ function auth_spip_dist($login, $pass, $serveur = '', $phpauth = false) {
$hash = null;
switch (strlen($row['pass'])) {
// legacy = md5 ou sha256
case 32:
// tres anciens mots de passe encodes en md5(alea.pass)
@ -88,7 +87,7 @@ function auth_spip_dist($login, $pass, $serveur = '', $phpauth = false) {
$methode = 'sha256';
}
if ($row['pass'] === $hash) {
spip_log("validation du mot de passe pour l'auteur #".$row['id_auteur']." $login via $methode", 'auth' . _LOG_DEBUG);
spip_log("validation du mot de passe pour l'auteur #" . $row['id_auteur'] . " $login via $methode", 'auth' . _LOG_DEBUG);
// ce n'est pas cense arriver, mais si jamais c'est un backup inutilisable, il faut le nettoyer pour ne pas bloquer la creation d'une nouvelle cle d'auth
if (!empty($row['backup_cles'])) {
sql_updateq('spip_auteurs', ['backup_cles' => ''], 'id_auteur=' . intval($row['id_auteur']));
@ -129,7 +128,7 @@ function auth_spip_dist($login, $pass, $serveur = '', $phpauth = false) {
unset($row);
}
else {
spip_log("validation du mot de passe pour l'auteur #".$row['id_auteur']." $login via Password::verifier", 'auth' . _LOG_DEBUG);
spip_log("validation du mot de passe pour l'auteur #" . $row['id_auteur'] . " $login via Password::verifier", 'auth' . _LOG_DEBUG);
}
break;
}

2
ecrire/inc/drapeau_edition.php

@ -124,7 +124,7 @@ function signale_edition($id, $auteur, $type = 'article') {
}
$edition = lire_tableau_edition();
$nom = $auteur['nom'] ?? $GLOBALS['ip'];
$id_a = $auteur['id_auteur'] ?? $GLOBALS['ip'];

6
ecrire/inc/filtres_images_lib_mini.php

@ -1039,9 +1039,8 @@ function _image_gd_output($img, $valeurs, $qualite = _IMG_GD_QUALITE, $fonction
#un flag pour reperer les images gravees
$lock = (
!statut_effacer_images_temporaires('get') // si la fonction n'a pas ete activee, on grave tout
or
(@file_exists($valeurs['fichier_dest']) and !@file_exists($valeurs['fichier_dest'] . '.src'))
);
or (@file_exists($valeurs['fichier_dest']) and !@file_exists($valeurs['fichier_dest'] . '.src'))
);
if (
function_exists($fonction)
&& ($ret = $fonction($img, $valeurs['fichier_dest'], $qualite)) # on a reussi a creer l'image
@ -1429,7 +1428,6 @@ function _image_creer_vignette($valeurs, $maxWidth, $maxHeight, $process = 'AUTO
// php5 imagemagick
elseif ($process == 'imagick') {
if (!class_exists(\Imagick::class)) {
spip_log('Classe Imagick absente !', _LOG_ERREUR);

4
ecrire/inc/modifier.php

@ -140,7 +140,9 @@ function objet_modifier_champs($objet, $id_objet, $options, $c = null, $serveur
// N'accepter que les champs qui existent dans la table
$champs = array_intersect_key($c, $desc['field']);
// et dont la valeur n'est pas null
$champs = array_filter($champs, static function($var){return $var !== null;} );
$champs = array_filter($champs, static function ($var) {
return $var !== null;
});
// TODO: ici aussi on peut valider les contenus
// en fonction du type

2
ecrire/public/composer.php

@ -218,7 +218,7 @@ function analyse_resultat_skel($nom, $cache, $corps, $source = '') {
if ($j == 'X-Spip-Filtre' and isset($headers[$j])) {
$headers[$j] .= '|' . $r[3];
} else {
$headers[$j] = str_replace(['\\\\',"\\'",'\\"'], ['\\',"'",'"'],$r[3]);
$headers[$j] = str_replace(['\\\\',"\\'",'\\"'], ['\\',"'",'"'], $r[3]);
}
}
}

2
ecrire/src/Chiffrer/SpipCles.php

@ -145,7 +145,7 @@ final class SpipCles {
return $this->cles->get($name);
}
// sinon loger et annule la cle generee car il ne faut pas l'utiliser
spip_log("Echec ecriture du fichier cle ".$this->file." ; impossible de generer une cle $name", 'chiffrer' . _LOG_ERREUR);
spip_log('Echec ecriture du fichier cle ' . $this->file . " ; impossible de generer une cle $name", 'chiffrer' . _LOG_ERREUR);
$this->cles->delete($name);
}
return null;

13
prive/formulaires/editer_auteur.php

@ -334,14 +334,13 @@ function formulaires_editer_auteur_traiter_dist(
// le formulaire ne peut être traité depuis une XMLHttpRequest
$prev = formulaires_editer_objet_charger('auteur', $id_auteur, 0, 0, $retour, $config_fonc, $row, $hidden);
if (
_request('new_pass') // nouveau mot de passe
or empty($prev['statut']) // creation auteur
or (_request('email') and $prev['email'] !== _request('email')) // modification email
or (_request('statut') === '0minirezo' and $prev['statut'] !== '0minirezo') // promotion 0minirezo
or (_request('statut') and intval(_request('statut')) < intval($prev['statut'])) // promotion de statut
or (_request('webmestre') and _request('webmestre') !== 'non' and $prev['webmestre'] !== 'oui') // promotion webmestre
_request('new_pass') // nouveau mot de passe
or empty($prev['statut']) // creation auteur
or (_request('email') and $prev['email'] !== _request('email')) // modification email
or (_request('statut') === '0minirezo' and $prev['statut'] !== '0minirezo') // promotion 0minirezo
or (_request('statut') and intval(_request('statut')) < intval($prev['statut'])) // promotion de statut
or (_request('webmestre') and _request('webmestre') !== 'non' and $prev['webmestre'] !== 'oui') // promotion webmestre
) {
refuser_traiter_formulaire_ajax();
// si on arrive là encore en ajax c'est pas OK, on genere une erreur
if (_AJAX or !empty($_SERVER['HTTP_X_REQUESTED_WITH'])) {

Loading…
Cancel
Save