Skip to content
Extraits de code Groupes Projets
Bifurcation depuis spip / spip
9017 validations de retard le dépôt en amont.
  • cerdic's avatar
    b8bebc4b
    Centralisation du code complexe d'appels de fonctions pour decoder une url. · b8bebc4b
    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.
    b8bebc4b
    Historique
    Centralisation du code complexe d'appels de fonctions pour decoder une url.
    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