Discussion :

[licardentaistor]

Bonjour,

Nous avons un très vieux intranet avec utilisation de PHP 5.2 et de la librairie mssql http://php.adamharvey.name/manual/fr/intro.mssql.php obsolète bien évidemment.

la grande question c'est qu'il y a une centaine de pages PHP comme ça à migrer vers sqlsrv.

il ne suffit pas de changer mssql_query en sqlsrv_query car l'ordre des paramètres change.

--------------------------------------------------------------------------------------
MSSQL PHP extension PHP Driver for SQL Server
--------------------------------------------------------------------------------------
mssql_fetch_assoc($stmt) sqlsrv_fetch_array($stmt, SQLSRV_FETCH_ASSOC)
mssql_fetch_row($stmt) sqlsrv_fetch_array($stmt, SQLSRV_FETCH_NUMERIC)
mssql_fetch_array($stmt, MSSQL_ASSOC) sqlsrv_fetch_array($stmt, SQLSRV_FETCH_ASSOC)
mssql_fetch_array($stmt, MSSQL_NUM) sqlsrv_fetch_array($stmt, SQLSRV_FETCH_NUMERIC)
mssql_fetch_array($stmt, MSSQL_BOTH) sqlsrv_fetch_array($stmt, SQLSRV_FETCH_BOTH)
mssql_next_result($stmt) sqlsrv_next_result($stmt)

votre avis?

[CosmoKnacki]

PHP 5.2 ? Franchement à ta place je recommencerais tout de A à Z. Au-delà du changement d'extension, le PHP et sa pratique ont quand même pas mal évolué depuis quasiment 20 ans, donc pour mettre ton code à niveau sur la version actuelle, il faudra bien plus que de modifier quelques fonctions.

[Séb.]

Je serais moins radical que CosmoKnacki, car si les bonnes pratiques ont été respectées il ne devrait pas non plus y avoir besoin de tout refaire, mais un bon lifting oui sûrement

Concernant mssql et sqlsrv, ne recommence pas la même erreur, et passe par PDO qui se charge d'encapsuler les libs sous-jacentes de manière transparente => https://www.php.net/manual/fr/ref.pdo-sqlsrv.php

Ici https://www.php.net/eol.php tu peux retrouver des guides de migration d'une version de PHP à l'autre, regarde notamment les "Backward Incompatible Changes".

[licardentaistor]

OK merci, je pensais faire un script python ou autre pour faire la conversion en automatique... en relisant le code PHP mais c'est pas si simple car l'ordre des paramètre est different

[Séb.]

Tu peux peut-être ré/écrire les fonctions mssql qui exécuteraient PDO.

Avec $stmt qui serait une instance PDOStatement :

function mssql_fetch_assoc($stmt)
{
    return $stmt->fetch(PDO::FETCH_ASSOC);
}

[licardentaistor]

oui aussi ! j'avais pas pensé à ça.

merci

A+