Bifurcation depuis
spip / spip
9017 validations de retard le dépôt en amont.
-
cerdic a rédigé
La complexite est liee a la compat ascendante, car les vieilles fonctions manipulaient directement des globales. La fonction urls_decoder_url() de inc/urls gere donc tout cela une fois pour toute, en isolant les globales concernees (sauvegarde/restauration) ce qui permet de l'utiliser partout sans precaution. Un flag $assembler par defaut a false permet de distinguer l'appel principal depuis public/assembler, qui accepter une redirection brutale, et necessite de prendre en compte les globales $_SERVER['REDIRECT_url_propre'] et $_ENV['url_propre'] Celles-ci ne sont plus videes par assembler car elles sont gerees proprement par la fonction urls_decoder_url Il serait neanmoins plus prudent d'evacuer aussi ces arguments implicites au profit d'arguments explicites, pour plus de clarete du code. Si les tests sur form_hidden sont exhaustif, on ne casse rien en passant par cette fonction depuis le filtre form_hidden. Si des cas de bugs sont remontes, il faudra completer les tests avant de corriger la fonction ou le filtre.
cerdic a rédigéLa complexite est liee a la compat ascendante, car les vieilles fonctions manipulaient directement des globales. La fonction urls_decoder_url() de inc/urls gere donc tout cela une fois pour toute, en isolant les globales concernees (sauvegarde/restauration) ce qui permet de l'utiliser partout sans precaution. Un flag $assembler par defaut a false permet de distinguer l'appel principal depuis public/assembler, qui accepter une redirection brutale, et necessite de prendre en compte les globales $_SERVER['REDIRECT_url_propre'] et $_ENV['url_propre'] Celles-ci ne sont plus videes par assembler car elles sont gerees proprement par la fonction urls_decoder_url Il serait neanmoins plus prudent d'evacuer aussi ces arguments implicites au profit d'arguments explicites, pour plus de clarete du code. Si les tests sur form_hidden sont exhaustif, on ne casse rien en passant par cette fonction depuis le filtre form_hidden. Si des cas de bugs sont remontes, il faudra completer les tests avant de corriger la fonction ou le filtre.
assembler.php 19,29 Kio