Valider c1abd409 rédigé par cerdic's avatar cerdic Validation de Gitea
Parcourir les fichiers

Une fonction charger_fonction_url() qui permet de trouver la fonction pour... 

Une fonction charger_fonction_url() qui permet de trouver la fonction pour generer l'url d'un objet, d'une page ou de decoder une url selon le type d'url active
- remplace l'usage peu evident de generer_objet_url('', '', '', '', true) pour recuperer la fonction de decodage
- permet l'implementation d'une fonction d'url propre pour les pages SPIP, soit dans un module d'url propre soit dans une fonction custom utilisateur (fix #962)
- permet d'eviter un point d'entree unique dans les fonctions d'url propre, ce qui permettra de decoder des urls numerique et de mieux typer les arguments (ouvre la porte pour traiter #4981)
Au passage, les fonction d'urls d'objet de type urls_generer_url_document() sont maintenant attentues sous la forme urls_generer_document_url() par coherence avec les autres nommages, mais l'ancien nommage continue a etre supporter pour la transition du code - pas de rupture de compat donc
parent 334d1378

ecrire/inc/urls.php

+1 −11
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -109,10 +109,7 @@ function urls_decoder_url($url, $fond = '', $contexte = [], $assembler = false) 
	}



	$url_redirect = '';

	$renommer = generer_objet_url('', '', '', '', true);

	if (!$renommer and !function_exists('recuperer_parametres_url')) {

		$renommer = charger_fonction('page', 'urls');

	} // fallback pour decoder l'url

	$renommer = charger_fonction_url('decoder');

	if ($renommer) {

		$a = $renommer($url, $fond, $contexte);

		if (is_array($a)) {
@@ -146,13 +143,6 @@ function urls_decoder_url($url, $fond = '', $contexte = [], $assembler = false) 
				$contexte['type-page'] = $type;

			}

		}

	} // compatibilite <= 1.9.2

	elseif (function_exists('recuperer_parametres_url')) {

		$GLOBALS['fond'] = $fond;

		$GLOBALS['contexte'] = $contexte;

		recuperer_parametres_url($fond, nettoyer_uri());

		// fond est en principe modifiee directement

		$contexte = $GLOBALS['contexte'];

	}



	// retablir les globales

ecrire/inc/utils.php

+81 −28
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -1765,6 +1765,60 @@ function autoriser_sans_cookie($nom, $strict = false) { 
	return false;

}



/**

 * Charger la fonction de gestion des urls si elle existe

 * @param $quoi : 'page' 'objet' 'decoder' ou objet spip pour lequel on cherche la fonction url par defaut (si type==='defaut')

 * @param $type : type des urls (par defaut la meta type_urls) ou 'defaut' pour trouver la fonction par defaut d'un type d'objet

 * @return string

 */

function charger_fonction_url(string $quoi, string $type = '') {

	if ($type === 'defaut') {

		$objet = objet_type($quoi);

		if ($f = charger_fonction('generer_' . $objet . '_url', 'urls', true)

			// deprecated

			or $f = charger_fonction('generer_url_' . $objet, 'urls', true)

		) {

			return $f;

		}

		return '';

	}



	if (!$type) {

		$type = $GLOBALS['type_urls'] ?? $GLOBALS['meta']['type_urls'] ?? 'page'; // sinon type "page" par défaut

	}

	$decoder = charger_fonction($type, 'urls', true);

	// se rabattre sur les urls page si les urls perso non dispo

	if (!$decoder) {

		$decoder = charger_fonction('page', 'urls', true);

		$type = 'page';

	}



	switch ($quoi) {

		case 'page':

			if (

				 function_exists($f = "urls_{$type}_generer_url_page")

			  or function_exists($f .= "_dist")

			  // ou une fonction custom utilisateur independante du type d'url

			  or function_exists($f = "generer_url_page")

			  or function_exists($f .= "_dist")

			) {

				return $f;

			}

			return '';

		case 'objet':

			if (function_exists($f = "urls_{$type}_generer_url_objet")

			  or function_exists($f .= "_dist")) {

				return $f;

			}

			// pas de break : si la fonction urls_{$type}_generer_url_objet n'existe pas c'est un ancien module URL

			// il faut passer par la fonction decoder qui fait tout en un (esperons)

		case 'decoder':

		default:

			return $decoder;

	}

}





/**

 * Fonction codant et décodant les URLs des objets SQL mis en page par SPIP

 *
@@ -1802,18 +1856,10 @@ function generer_objet_url($id = '', $entite = '', $args = '', $ancre = '', $pub 
		}

		$res = generer_objet_url_ecrire($entite, $id, $args, $ancre, false);

	} else {

		if ($type === null) {

			$type = $GLOBALS['type_urls'] ?? $GLOBALS['meta']['type_urls'] ?? 'page'; // sinon type "page" par défaut

		}

		$f = charger_fonction_url('objet', $type ?? '');



		$f = charger_fonction($type, 'urls', true);

		// se rabattre sur les urls page si les urls perso non dispo

		if (!$f) {

			$f = charger_fonction('page', 'urls', true);

		}



		// si $entite='', on veut la fonction de passage URL ==> id

		// sinon on veut effectuer le passage id ==> URL

		// @deprecated si $entite='', on veut la fonction de passage URL ==> id

		// @see charger_fonction_url

		if (!$entite) {

			return $f;

		}
@@ -2209,30 +2255,37 @@ function generer_url_public($script = '', $args = '', $no_entities = false, $rel 
	// si le script est une action (spip_pass, spip_inscription),

	// standardiser vers la nouvelle API



	if (is_array($args)) {

		$args = http_build_query($args);

	}



	$url = '';

	if ($f = charger_fonction_url('page')) {

		$url = $f($script, $args);

		if ($url and !$rel) {

			include_spip('inc/filtres_mini');

			$url = url_absolue($url);

		}

	}

	if (!$url) {

		if (!$action) {

			$action = get_spip_script();

		}

		if ($script) {

			$action = parametre_url($action, _SPIP_PAGE, $script, '&');

		}



		if ($args) {

		if (is_array($args)) {

			$r = '';

			foreach ($args as $k => $v) {

				$r .= '&' . $k . '=' . $v;

			$action .= (strpos($action, '?') !== false ? '&' : '?') . $args;

		}

			$args = substr($r, 1);

		}

		$action .=

			(strpos($action, '?') !== false ? '&' : '?') . $args;

		// ne pas generer une url avec /./?page= en cas d'url absolue et de _SPIP_SCRIPT vide

		$url = ($rel ? _DIR_RACINE . $action : rtrim(url_de_base(), '/') . preg_replace(',^/[.]/,', '/', "/$action"));

	}



	if (!$no_entities) {

		$action = quote_amp($action);

		$url = quote_amp($url);

	}



	// ne pas generer une url avec /./?page= en cas d'url absolue et de _SPIP_SCRIPT vide

	return ($rel ? _DIR_RACINE . $action : rtrim(url_de_base(), '/') . preg_replace(',^/[.]/,', '/', "/$action"));

	return $url;

}



function generer_url_prive($script, $args = '', $no_entities = false) {

Forge communautaire SPIP | Charte d'utilisation | Signaler un problème sur ce site