Valider 5f13efb4 rédigé par marcimat's avatar marcimat
Parcourir les fichiers

PHP 8.1 #4986 : Typer `string` les paramètres $connect dans les fonctions,...

PHP 8.1 #4986 : Typer `string` les paramètres $connect dans les fonctions, puisque c’est ce qui est attendu.

Seuls certains (comme typo()) sont ignorés car ils semblent avoir des spécificités ($connect = null).

De même, typage de quelques $serveur, mais il faudrait généraliser.
parent aba5ad38
Chargement en cours
Chargement en cours
Chargement en cours
Chargement en cours
+7 −5
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -190,10 +190,11 @@ function auth_spip_formulaire_login($flux) {
 * @return bool
 *   toujours true pour un auteur cree dans SPIP
 */
function auth_spip_autoriser_modifier_login($serveur = '') {
function auth_spip_autoriser_modifier_login(string $serveur = '') : bool {
	// les fonctions d'ecriture sur base distante sont encore incompletes
	if (strlen($serveur)) {
		return false;
	} // les fonctions d'ecriture sur base distante sont encore incompletes
	} 
	return true;
}

@@ -353,10 +354,11 @@ function auth_spip_informer_login($infos, $row, $serveur = '') {
 * @return bool
 *  toujours true pour un auteur cree dans SPIP
 */
function auth_spip_autoriser_modifier_pass($serveur = '') {
function auth_spip_autoriser_modifier_pass(string $serveur = '') : bool {
	// les fonctions d'ecriture sur base distante sont encore incompletes
	if (strlen($serveur)) {
		return false;
	} // les fonctions d'ecriture sur base distante sont encore incompletes
	} 
	return true;
}

@@ -437,7 +439,7 @@ function auth_spip_modifier_pass($login, $new_pass, $id_auteur, $serveur = '') {
 * @param string $serveur
 * @return void
 */
function auth_spip_synchroniser_distant($id_auteur, $champs, $options = [], $serveur = '') {
function auth_spip_synchroniser_distant($id_auteur, $champs, $options = [], string $serveur = '') : void {
	// ne rien faire pour une base distante : on ne sait pas regenerer les htaccess
	if (strlen($serveur)) {
		return;
+3 −3
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -4794,7 +4794,7 @@ function generer_info_entite($id_objet, $type_objet, $info, $etoile = '', $param
 *     Nom du connecteur à la base de données
 * @return string
 */
function generer_introduction_entite($id_objet, $type_objet, $ligne_sql, $introduction_longueur = null, $longueur_ou_suite = null, $suite = null, $connect = '') {
function generer_introduction_entite($id_objet, $type_objet, $ligne_sql, $introduction_longueur = null, $longueur_ou_suite = null, $suite = null, string $connect = '') {

	$descriptif = $ligne_sql['descriptif'] ?? '';
	$texte = $ligne_sql['texte'] ?? '';
@@ -4838,7 +4838,7 @@ function generer_introduction_entite($id_objet, $type_objet, $ligne_sql, $introd
 * @param string $connect
 * @return string
 */
function appliquer_traitement_champ($texte, $champ, $table_objet = '', $env = [], $connect = '') {
function appliquer_traitement_champ($texte, $champ, $table_objet = '', $env = [], string $connect = '') {
	if (!$champ) {
		return $texte;
	}
@@ -5110,7 +5110,7 @@ function insert_head_css_conditionnel($flux) {
 * @param string $connect
 * @return string
 */
function produire_fond_statique($fond, $contexte = [], $options = [], $connect = '') {
function produire_fond_statique($fond, $contexte = [], $options = [], string $connect = '') {
	if (isset($contexte['format'])) {
		$extension = $contexte['format'];
		unset($contexte['format']);
+8 −8
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -29,7 +29,7 @@ function inc_lien_dist(
	$title = '',
	$hlang = '',
	$rel = '',
	$connect = '',
	string $connect = '',
	$env = []
) {
	return $lien;
@@ -42,7 +42,7 @@ function inc_lien_dist(
define('_RACCOURCI_LIEN', '/\[([^][]*?([[]\w*[]][^][]*)*)->(>?)([^]]*)\]/msS');

// https://code.spip.net/@expanser_liens
function expanser_liens($t, $connect = '', $env = []) {
function expanser_liens($t, string $connect = '', $env = []) {

	$t = pipeline('pre_liens', $t);

@@ -56,7 +56,7 @@ function expanser_liens($t, $connect = '', $env = []) {
// Meme analyse mais pour eliminer les liens
// et ne laisser que leur titre, a expliciter si ce n'est fait
// https://code.spip.net/@nettoyer_raccourcis_typo
function nettoyer_raccourcis_typo($texte, $connect = '') {
function nettoyer_raccourcis_typo($texte, string $connect = '') {
	return $texte;
}

@@ -96,7 +96,7 @@ function virtuel_redirige($virtuel, $url = false) {
// 'url':   seulement U  (i.e. generer_url_RACCOURCI)

// https://code.spip.net/@calculer_url
function calculer_url($ref, $texte = '', $pour = 'url', $connect = '', $echappe_typo = true) {
function calculer_url($ref, $texte = '', $pour = 'url', string $connect = '', $echappe_typo = true) {
	$r = traiter_lien_implicite($ref, $texte, $pour, $connect, $echappe_typo);

	return $r ? $r : traiter_lien_explicite($ref, $texte, $pour, $connect, $echappe_typo);
@@ -105,7 +105,7 @@ function calculer_url($ref, $texte = '', $pour = 'url', $connect = '', $echappe_
define('_EXTRAIRE_LIEN', ',^\s*(?:' . _PROTOCOLES_STD . '):?/?/?\s*$,iS');

// https://code.spip.net/@traiter_lien_explicite
function traiter_lien_explicite($ref, $texte = '', $pour = 'url', $connect = '', $echappe_typo = true) {
function traiter_lien_explicite($ref, $texte = '', $pour = 'url', string $connect = '', $echappe_typo = true) {
	if (preg_match(_EXTRAIRE_LIEN, $ref)) {
		return ($pour != 'tout') ? '' : ['', '', '', ''];
	}
@@ -146,7 +146,7 @@ function traiter_lien_explicite($ref, $texte = '', $pour = 'url', $connect = '',
	return ['url' => $lien, 'titre' => $texte];
}

function liens_implicite_glose_dist($texte, $id, $type, $args, $ancre, $connect = '') {
function liens_implicite_glose_dist($texte, $id, $type, $args, $ancre, string $connect = '') {
	if (function_exists($f = 'glossaire_' . $ancre)) {
		$url = $f($texte, $id);
	} else {
@@ -157,7 +157,7 @@ function liens_implicite_glose_dist($texte, $id, $type, $args, $ancre, $connect
}

// https://code.spip.net/@traiter_lien_implicite
function traiter_lien_implicite($ref, $texte = '', $pour = 'url', $connect = '') {
function traiter_lien_implicite($ref, $texte = '', $pour = 'url', string $connect = '') {
	if (!($match = typer_raccourci($ref))) {
		return false;
	}
@@ -311,7 +311,7 @@ define(
define('_RACCOURCI_MODELE_DEBUT', '@^' . _RACCOURCI_MODELE . '@isS');

// https://code.spip.net/@traiter_modeles
function traiter_modeles($texte, $doublons = false, $echap = '', $connect = '', $liens = null, $env = []) {
function traiter_modeles($texte, $doublons = false, $echap = '', string $connect = '', $liens = null, $env = []) {
	// preserver la compatibilite : true = recherche des documents
	if ($doublons === true) {
		$doublons = ['documents' => ['doc', 'emb', 'img']];
+1 −1
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -248,7 +248,7 @@ function nettoyer_url_page($url, $contexte = []) {
 * @return string
 *
 */
function generer_url_ecrire_objet($objet, $id, $args = '', $ancre = '', $public = null, $connect = '') {
function generer_url_ecrire_objet($objet, $id, $args = '', $ancre = '', $public = null, string $connect = '') {
	static $furls = [];
	if (!isset($furls[$objet])) {
		if (
+1 −1
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -3286,7 +3286,7 @@ function erreur_squelette($message = '', $lieu = '') {
 *     - Contenu du squelette calculé
 *     - ou tableau d'information sur le squelette.
 */
function recuperer_fond($fond, $contexte = [], $options = [], $connect = '') {
function recuperer_fond($fond, $contexte = [], $options = [], string $connect = '') {
	if (!function_exists('evaluer_fond')) {
		include_spip('public/assembler');
	}
Chargement en cours