IFAEDI / S.I

Mise en place d'un S.I avancé pour l'AEDI
3 stars 2 forks source link

Depreciation de mysql_escape_string #19

Closed blackheaven closed 11 years ago

blackheaven commented 11 years ago

Rapidement, on est passé à PHP 5.4.10, mysql_escape_string est déprécié du coups il y a presque tout qui plante. Voilà comment le remplacer : Dans commun/php/bd.inc.php :

public static function Protect($s)
{
    return self::GetConnection()->quote($s);
}

Ensuite il suffit de faire un s&r de mysql_escape_string par DB:::Protect, et de faire un grep pour remplacer les DB:::Protect autour des ID par un (int) $_GET['id'], et le tour est joué. Deux autres petits conseils :

PS : j'ai commencé à hotfixer en prod' mais c'est à jeter, je suis loin de chez moi et j'ai très peu de temps, désolé.

blackheaven commented 11 years ago

Typiquement, au point de vue de la sécurité c'est simple :

benjaminplanche commented 11 years ago

J'aime le PHP autant que je suis doué pour ... Je préfère laisser ça à ceux qui gèrent, sauf si vraiment y a urgence.

blackheaven commented 11 years ago

pas de soucis, je ferais une PR jeudi