Compatibilité de l'application non assurée en PHP5.5

[Baraban35]

Problème des fonctions mysql_ dépréciées à partir de PHP5.5, abandonnées en PHP7.

[GDucourneau]

Ce qui a été fait pour le moment :

• création de quelques tests qui vérifient la fonctionnalité des méthodes de classe mysql (avec l'extension Mysql)
• modification des méthodes de la classe mysql (utilisation des classes PDO et PDOStatement au lieu de l'extension Mysql)
• création de quelques tests qui vérifient la fonctionnalité des méthodes de classe mysql (avec les classes PDO et PDOStatement)

Ce qu'il reste à faire :

• correction de diverses bugs afin que tous les tests fonctionnent (plus ou moins une demi-journée)
• création de documentations pour les méthodes de la classe mysql (plus ou moins une demi-journée)

[Baraban35]

Bonjour Gwenaël, Bien pour le commentaire. Quelques remarques : 1) Ce qui a été fait pour le moment : La classe de test MySqlTest doit être strictement identique pour tester les méthodes de la classe MySql, que celle-ci s'appuie sur la bibliothèque de fonctions MySql ou les classes PDO et PDOStatement. Est-ce bien le cas ?

2) Ce qui reste à faire : correction de bugs : ok pour la demi-journée

• 1h : écriture et exécution de tests unitaires supplémentaires : méthode testExecSQL à écrire avec un cas de test pour chaque requête action utilisée : insert, update et delete, et en allant vérifier le résultat retourné (cas 1 ou plusieurs lignes affectées), ce que ne fait pas la méthode de test testInsertId car ce n'est pas son rôle premier de tester execSQL.
• 1h : voir quelle(s) solution(s) simple(s) pourraient convenir pour éviter de modifier directement la classe MySql à 2 lignes distinctes (include) lorsqu'elle est utilisée par PHPUnit ou par l'application web. Je regarde aussi de mon côté.
• 1h : vérification de la bonne exécution de l'application web avec cette nouvelle classe MySQL. Il faut tester quelques fonctionnalités des trois rôles (admin, enseignant, étudiant) pour voir s'il n'y a pas eu régression.
• 1h : réécrire le fichier enregparam.php en s'appuyant sur les classes PDO et PDOStatement. Faire un test à partir de instal.php, ce dernier contenant le formulaire pour saisir les données d'installation, puis appelant sur soumission le script enregparam.php. Ne pas passer de temps à faire des tests variés : l'installateur n'est pas recommandé, mais il vaut mieux le migrer sous PDO en attendant une refonte plus complète.
• faire un commit des scripts modifiés en référençant l'issue, puis le publier sur la branche master de la plateforme github.

Les propositions de temps sont données à titre indicatif. Ce sera intéressant de comparer ce qui est prévu à ce qui est effectif. Bonne journée. PS : Je teste l'envoi de la réponse à Baraban35/SuiviSIO. On va voir si cette réponse va être visible directement sur la plateforme github. A. Baraban

2015-06-10 20:14 GMT+02:00 GDucourneau notifications@github.com:

Ce qui a été fait pour le moment :

• création de quelques tests qui vérifient la fonctionnalité des méthodes de classe mysql (avec l'extension Mysql)
• modification des méthodes de la classe mysql (utilisation des classes PDO et PDOStatement au lieu de l'extension Mysql)
• création de quelques tests qui vérifient la fonctionnalité des méthodes de classe mysql (avec les classes PDO et PDOStatement)

Ce qu'il reste à faire :

• correction de diverses bugs afin que tous les tests fonctionnent (plus ou moins une demi-journée)
• création de documentations pour les méthodes de la classe mysql (plus ou moins une demi-journée)

— Reply to this email directly or view it on GitHub https://github.com/Baraban35/SuiviSIO/issues/2#issuecomment-110861106.

[GDucourneau]

Bonjour Madame, Oui, en effet, pour les tests, j'ai repris le même fichier que j'ai modifié afin qu'il corresponde aux classes PDO et PDOStatement

Gwenaël DUCOURNEAU

Le 11 juin 2015 07:52, Baraban35 notifications@github.com a écrit :

Bonjour Gwenaël, Bien pour le commentaire. Quelques remarques : 1) Ce qui a été fait pour le moment : La classe de test MySqlTest doit être strictement identique pour tester les méthodes de la classe MySql, que celle-ci s'appuie sur la bibliothèque de fonctions MySql ou les classes PDO et PDOStatement. Est-ce bien le cas ?

2) Ce qui reste à faire : correction de bugs : ok pour la demi-journée

• 1h : écriture et exécution de tests unitaires supplémentaires : méthode testExecSQL à écrire avec un cas de test pour chaque requête action utilisée : insert, update et delete, et en allant vérifier le résultat retourné (cas 1 ou plusieurs lignes affectées), ce que ne fait pas la méthode de test testInsertId car ce n'est pas son rôle premier de tester execSQL.
• 1h : voir quelle(s) solution(s) simple(s) pourraient convenir pour éviter de modifier directement la classe MySql à 2 lignes distinctes (include) lorsqu'elle est utilisée par PHPUnit ou par l'application web. Je regarde aussi de mon côté.
• 1h : vérification de la bonne exécution de l'application web avec cette nouvelle classe MySQL. Il faut tester quelques fonctionnalités des trois rôles (admin, enseignant, étudiant) pour voir s'il n'y a pas eu régression.
• 1h : réécrire le fichier enregparam.php en s'appuyant sur les classes PDO et PDOStatement. Faire un test à partir de instal.php, ce dernier contenant le formulaire pour saisir les données d'installation, puis appelant sur soumission le script enregparam.php. Ne pas passer de temps à faire des tests variés : l'installateur n'est pas recommandé, mais il vaut mieux le migrer sous PDO en attendant une refonte plus complète.
• faire un commit des scripts modifiés en référençant l'issue, puis le publier sur la branche master de la plateforme github.

Les propositions de temps sont données à titre indicatif. Ce sera intéressant de comparer ce qui est prévu à ce qui est effectif. Bonne journée. PS : Je teste l'envoi de la réponse à Baraban35/SuiviSIO. On va voir si cette réponse va être visible directement sur la plateforme github. A. Baraban

2015-06-10 20:14 GMT+02:00 GDucourneau notifications@github.com:

Ce qui a été fait pour le moment :

• création de quelques tests qui vérifient la fonctionnalité des méthodes de classe mysql (avec l'extension Mysql)
• modification des méthodes de la classe mysql (utilisation des classes PDO et PDOStatement au lieu de l'extension Mysql)
• création de quelques tests qui vérifient la fonctionnalité des méthodes de classe mysql (avec les classes PDO et PDOStatement)

Ce qu'il reste à faire :

• correction de diverses bugs afin que tous les tests fonctionnent (plus ou moins une demi-journée)
• création de documentations pour les méthodes de la classe mysql (plus ou moins une demi-journée)

— Reply to this email directly or view it on GitHub https://github.com/Baraban35/SuiviSIO/issues/2#issuecomment-110861106.

— Reply to this email directly or view it on GitHub https://github.com/Baraban35/SuiviSIO/issues/2#issuecomment-111004030.

[GDucourneau]

Modification des fonctions de la classe mysql_ (sous répertoire_de_project/class/mysql.class.php) :

• création de quelques tests qui vérifient la fonctionnalité des méthodes de classe mysql (avec l'extension Mysql)
• modification des méthodes de la classe mysql (utilisation des classes PDO et PDOStatement au lieu de l'extension Mysql)
• création de quelques tests qui vérifient la fonctionnalité des méthodes de classe mysql (avec les classes PDO et PDOStatement)
• modification du fichier enreg.param.php (sous répertoire_de_project/dirrw/init_A_Ne_Pas_Utiliser), cependant l'installateur n'est pas fonctionnel à 100% /!\
• correction de diverses anomalies

Remarque : les anciens fichiers (utilisants l'extension mysql) sont conservés sous les noms Old_nom_du_fichier.

[GDucourneau]

• Suppression des fichiers Old_'nom_du_fichier'.
• Ajout des documentations des méthodes de la classe Mysql. (sous repertoire_de_projet/class/mysql.class.php)
• Modification du type de donnée en sortie de la méthode InsertId. (sous repertoire_de_projet/class/mysql.class.php)
• Modification de la méthode __construct de la classe Mysql.(sous repertoire_de_projet/class/mysql.class.php)
• Tests fonctionnels de l'application web après ces changements.

[GDucourneau]

• Modification de la méthode sauveTable de la classe Mysql.(sous repertoire_de_projet/class/mysql.class.php)
• Création d'un document qui renseigne les tests fonctionnels effectués sur l'application web.