Si il s'agit de plumeCMS, je me permet quelques conseils apres un RAPIDE coup d'oeil :
- les magicstrip & co sont des pis allés ou si tu preferes un minimum non suffisant. Il est préférable de filtrer en fonction du type de données attendus par formulaire/action/page.
- tu concatenes directement des variables dans la construction de tes requetes SQL, encore une fois, il est préférable de s'assurer que tout correspond bien aux données attendus et de quoter quand même
- ton code HTML est lui aussi produit par concaténation et tu ne quotes pas, donc il y a aussi un risque de produire de la merde par là
- tu ne sembles pas gérer les histoires de charset, hors cela permet d'injecter des betises
- tu utilises des variables pour ta conf au lieu de constantes, les variables sont RW les constantes sont RO
- tu sembles mélanger différents styles de programmation
[^] # Re: idée toute simple
Posté par Mouns . En réponse au journal Comment tuer un développeur de LL*. Évalué à 10.
- les magicstrip & co sont des pis allés ou si tu preferes un minimum non suffisant. Il est préférable de filtrer en fonction du type de données attendus par formulaire/action/page.
- tu concatenes directement des variables dans la construction de tes requetes SQL, encore une fois, il est préférable de s'assurer que tout correspond bien aux données attendus et de quoter quand même
- ton code HTML est lui aussi produit par concaténation et tu ne quotes pas, donc il y a aussi un risque de produire de la merde par là
- tu ne sembles pas gérer les histoires de charset, hors cela permet d'injecter des betises
- tu utilises des variables pour ta conf au lieu de constantes, les variables sont RW les constantes sont RO
- tu sembles mélanger différents styles de programmation