From 5f2789595d5dcdec5897c8f8efb7865b49e393b9 Mon Sep 17 00:00:00 2001
From: Matthieu Marcillaud <marcimat@rezo.net>
Date: Fri, 4 Mar 2022 13:21:49 +0100
Subject: [PATCH] Coding standard
---
ecrire/action/inscrire_auteur.php | 10 ++++-----
ecrire/inc/filtres.php | 2 +-
ecrire/inc/securiser_action.php | 4 ++--
ecrire/inc/utils.php | 2 +-
ecrire/maj/2021.php | 2 +-
ecrire/public/criteres.php | 6 +++---
ecrire/public/interfaces.php | 10 ++++-----
ecrire/src/Chiffrer/Chiffrement.php | 33 ++++++++++++++---------------
ecrire/src/Chiffrer/SpipCles.php | 2 +-
9 files changed, 35 insertions(+), 36 deletions(-)
diff --git a/ecrire/action/inscrire_auteur.php b/ecrire/action/inscrire_auteur.php
index 59ab85d3f4..69191f255d 100644
--- a/ecrire/action/inscrire_auteur.php
+++ b/ecrire/action/inscrire_auteur.php
@@ -346,7 +346,7 @@ function confirmer_statut_inscription($auteur) {
/**
* Attribuer un jeton temporaire pour un auteur en assurant l'unicite du jeton.
- *
+ *
* Chaque appel crée un nouveau jeton pour l’auteur
* et invalide donc le précédent
*
@@ -359,8 +359,8 @@ function auteur_attribuer_jeton($id_auteur): string {
include_spip('inc/chiffrer');
// s'assurer de l'unicite du jeton pour le couple (email,cookie)
do {
- // Un morceau du jeton est lisible en bdd pour éviter de devoir déchiffrer
- // tous les jetons connus pour vérifier le jeton d’un auteur.
+ // Un morceau du jeton est lisible en bdd pour éviter de devoir déchiffrer
+ // tous les jetons connus pour vérifier le jeton d’un auteur.
$public = substr(creer_uniqid(), 0, 7) . '.';
$jeton = $public . creer_uniqid();
$jeton_chiffre_prefixe = $public . Chiffrement::chiffrer($jeton, SpipCles::secret_du_site());
@@ -372,10 +372,10 @@ function auteur_attribuer_jeton($id_auteur): string {
/**
* Lire un jeton temporaire d’un auteur (peut le créer au besoin)
- *
+ *
* Cette fonction peut être pratique si plusieurs notifications proches
* dans la durée sont envoyées au même auteur.
- *
+ *
* @param int $id_auteur
* @param bool $autoInit Attribue un jeton à l’auteur s’il n’en a pas déjà .
* @return string|null
diff --git a/ecrire/inc/filtres.php b/ecrire/inc/filtres.php
index 039e685985..549d5959c0 100644
--- a/ecrire/inc/filtres.php
+++ b/ecrire/inc/filtres.php
@@ -4432,7 +4432,7 @@ function prepare_icone_base($type, $lien, $texte, $fond, $fonction = '', $class
$class_bouton .= " s$size";
// Icône
- $icone = http_img_pack($fond, $alt, "width='$size' height='$size'",'', ['alternative' => "objet-generique-$size.png"]);
+ $icone = http_img_pack($fond, $alt, "width='$size' height='$size'", '', ['alternative' => "objet-generique-$size.png"]);
$icone = '<span class="icone-image' . ($fonction ? " icone-fonction icone-fonction-$fonction" : '') . "\">$icone</span>";
// Markup final
diff --git a/ecrire/inc/securiser_action.php b/ecrire/inc/securiser_action.php
index 970c5b1f0b..730fd0977d 100644
--- a/ecrire/inc/securiser_action.php
+++ b/ecrire/inc/securiser_action.php
@@ -16,7 +16,7 @@
* @package SPIP\Core\Actions
**/
- use \Spip\Chiffrer\SpipCles;
+ use Spip\Chiffrer\SpipCles;
if (!defined('_ECRIRE_INC_VERSION')) {
return;
@@ -296,7 +296,7 @@ function verifier_action_auteur($action, $hash) {
/**
* Renvoyer le secret du site (le generer si il n'existe pas encore)
- *
+ *
* @uses SpipCles::secret_du_site()
* @return string
*/
diff --git a/ecrire/inc/utils.php b/ecrire/inc/utils.php
index 357002bdc6..ef55e9f594 100644
--- a/ecrire/inc/utils.php
+++ b/ecrire/inc/utils.php
@@ -1826,7 +1826,7 @@ function charger_fonction_url(string $quoi, string $type = '') {
return $f;
}
// sinon on se rabat sur les urls page si ce n'est pas un type demande explicitement
- if (!$type and $url_type !== 'page'){
+ if (!$type and $url_type !== 'page') {
return charger_fonction_url($quoi, 'page');
}
return '';
diff --git a/ecrire/maj/2021.php b/ecrire/maj/2021.php
index 272074aca9..33568a6996 100644
--- a/ecrire/maj/2021.php
+++ b/ecrire/maj/2021.php
@@ -29,5 +29,5 @@ $GLOBALS['maj'][2021_02_18_00] = [
$GLOBALS['maj'][2022_02_23_02] = [
['sql_alter', "TABLE spip_auteurs ADD backup_cles mediumtext DEFAULT '' NOT NULL" ],
- ['sql_delete', "spip_meta","nom='secret_du_site'" ],
+ ['sql_delete', 'spip_meta', "nom='secret_du_site'" ],
];
diff --git a/ecrire/public/criteres.php b/ecrire/public/criteres.php
index 4d808cb313..fa0418cb85 100644
--- a/ecrire/public/criteres.php
+++ b/ecrire/public/criteres.php
@@ -2618,7 +2618,7 @@ function calculer_critere_infixe_date($idb, &$boucles, $col) {
// Le type de critère à prendre en compte
$col = $regs[1];
-
+
// Si on trouve un nom de champ date précis, on l'utilise, pas besoin de déclaration dans l'API objet
if (isset($regs[3]) and $suite = $regs[3]) {
# Recherche de l'existence du champ date_xxxx,
@@ -2628,7 +2628,7 @@ function calculer_critere_infixe_date($idb, &$boucles, $col) {
} else {
$date_orig = substr($suite, 1);
}
-
+
$pred = $date_orig;
} else { // Sinon il FAUT avoir déclaré le champ date officiel dans l'API objet
// Si aucune déclaration trouvée, on quitte
@@ -2637,7 +2637,7 @@ function calculer_critere_infixe_date($idb, &$boucles, $col) {
}
// Par défaut, on prend le champ date déclaré dans l'API
$pred = $date_orig = $GLOBALS['table_date'][$table['id_table']] ?? $table['date'];
-
+
// Si c'est pour du relatif
if (isset($regs[2]) and $rel = $regs[2]) {
$pred = 'date';
diff --git a/ecrire/public/interfaces.php b/ecrire/public/interfaces.php
index dafc8e83b0..8638f26cc7 100644
--- a/ecrire/public/interfaces.php
+++ b/ecrire/public/interfaces.php
@@ -538,13 +538,13 @@ class Boucle {
public $ligne = 0;
- /**
- * table pour stocker les modificateurs de boucle tels que tout, plat ...,
- * utilisable par les plugins egalement
- *
+ /**
+ * table pour stocker les modificateurs de boucle tels que tout, plat ...,
+ * utilisable par les plugins egalement
+ *
* @var array<string, mixed>
*/
- public $modificateur = [];
+ public $modificateur = [];
/**
* Type d'itérateur utilisé pour cette boucle
diff --git a/ecrire/src/Chiffrer/Chiffrement.php b/ecrire/src/Chiffrer/Chiffrement.php
index 3f2e329f3a..2a80056536 100644
--- a/ecrire/src/Chiffrer/Chiffrement.php
+++ b/ecrire/src/Chiffrer/Chiffrement.php
@@ -12,14 +12,13 @@
namespace Spip\Chiffrer;
-/**
- * Chiffrement / déchiffrement symétrique.
- *
+/**
+ * Chiffrement / déchiffrement symétrique.
+ *
* @link https://fr.wikipedia.org/wiki/Cryptographie_sym%C3%A9trique
* @link https://www.php.net/manual/fr/book.sodium.php
*/
class Chiffrement {
-
/** Chiffre un message en utilisant une clé ou un mot de passe */
public static function chiffrer(
string $message,
@@ -40,7 +39,7 @@ class Chiffrement {
return $encoded;
}
- /** Déchiffre un message en utilisant une clé ou un mot de passe */
+ /** Déchiffre un message en utilisant une clé ou un mot de passe */
public static function dechiffrer(
string $encoded,
#[\SensitiveParameter]
@@ -48,9 +47,9 @@ class Chiffrement {
): ?string {
$decoded = base64_decode($encoded);
$salt = substr($decoded, 0, \SODIUM_CRYPTO_PWHASH_SALTBYTES);
- $nonce = substr($decoded, \SODIUM_CRYPTO_PWHASH_SALTBYTES, \SODIUM_CRYPTO_SECRETBOX_NONCEBYTES);
- $encrypted = substr($decoded, \SODIUM_CRYPTO_PWHASH_SALTBYTES + \SODIUM_CRYPTO_SECRETBOX_NONCEBYTES);
- $key = self::deriveKeyFromPassword($key, $salt);
+ $nonce = substr($decoded, \SODIUM_CRYPTO_PWHASH_SALTBYTES, \SODIUM_CRYPTO_SECRETBOX_NONCEBYTES);
+ $encrypted = substr($decoded, \SODIUM_CRYPTO_PWHASH_SALTBYTES + \SODIUM_CRYPTO_SECRETBOX_NONCEBYTES);
+ $key = self::deriveKeyFromPassword($key, $salt);
$padded_message = sodium_crypto_secretbox_open($encrypted, $nonce, $key);
sodium_memzero($key);
sodium_memzero($nonce);
@@ -83,15 +82,15 @@ class Chiffrement {
if (strlen($password) === \SODIUM_CRYPTO_SECRETBOX_KEYBYTES) {
return $password;
}
- $key = sodium_crypto_pwhash(
- \SODIUM_CRYPTO_SECRETBOX_KEYBYTES,
- $password,
- $salt,
- \SODIUM_CRYPTO_PWHASH_OPSLIMIT_INTERACTIVE,
- \SODIUM_CRYPTO_PWHASH_MEMLIMIT_INTERACTIVE
- );
- sodium_memzero($password);
+ $key = sodium_crypto_pwhash(
+ \SODIUM_CRYPTO_SECRETBOX_KEYBYTES,
+ $password,
+ $salt,
+ \SODIUM_CRYPTO_PWHASH_OPSLIMIT_INTERACTIVE,
+ \SODIUM_CRYPTO_PWHASH_MEMLIMIT_INTERACTIVE
+ );
+ sodium_memzero($password);
- return $key;
+ return $key;
}
}
diff --git a/ecrire/src/Chiffrer/SpipCles.php b/ecrire/src/Chiffrer/SpipCles.php
index 1071e13287..1739484f49 100644
--- a/ecrire/src/Chiffrer/SpipCles.php
+++ b/ecrire/src/Chiffrer/SpipCles.php
@@ -58,7 +58,7 @@ final class SpipCles {
return $key ^ $meta;
}
- /** Renvoyer le secret des authentifications */
+ /** Renvoyer le secret des authentifications */
public function getSecretAuth(bool $autoInit = false): ?string {
return $this->getKey('secret_des_auth', $autoInit);
}
--
GitLab