{"action":"created","issue":{"url":"https://api.github.com/repos/Baraban35/SuiviSIO/issues/2","labels_url":"https://api.github.com/repos/Baraban35/SuiviSIO/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/Baraban35/SuiviSIO/issues/2/comments","events_url":"https://api.github.com/repos/Baraban35/SuiviSIO/issues/2/events","html_url":"https://github.com/Baraban35/SuiviSIO/issues/2","id":85071404,"number":2,"title":"Compatibilité de l'application non assurée en PHP5.5","user":{"login":"Baraban35","id":12734170,"avatar_url":"https://avatars.githubusercontent.com/u/12734170?v=3","gravatar_id":"","url":"https://api.github.com/users/Baraban35","html_url":"https://github.com/Baraban35","followers_url":"https://api.github.com/users/Baraban35/followers","following_url":"https://api.github.com/users/Baraban35/following{/other_user}","gists_url":"https://api.github.com/users/Baraban35/gists{/gist_id}","starred_url":"https://api.github.com/users/Baraban35/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Baraban35/subscriptions","organizations_url":"https://api.github.com/users/Baraban35/orgs","repos_url":"https://api.github.com/users/Baraban35/repos","events_url":"https://api.github.com/users/Baraban35/events{/privacy}","received_events_url":"https://api.github.com/users/Baraban35/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"GDucourneau","id":12747483,"avatar_url":"https://avatars.githubusercontent.com/u/12747483?v=3","gravatar_id":"","url":"https://api.github.com/users/GDucourneau","html_url":"https://github.com/GDucourneau","followers_url":"https://api.github.com/users/GDucourneau/followers","following_url":"https://api.github.com/users/GDucourneau/following{/other_user}","gists_url":"https://api.github.com/users/GDucourneau/gists{/gist_id}","starred_url":"https://api.github.com/users/GDucourneau/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GDucourneau/subscriptions","organizations_url":"https://api.github.com/users/GDucourneau/orgs","repos_url":"https://api.github.com/users/GDucourneau/repos","events_url":"https://api.github.com/users/GDucourneau/events{/privacy}","received_events_url":"https://api.github.com/users/GDucourneau/received_events","type":"User","site_admin":false},"milestone":null,"comments":2,"created_at":"2015-06-04T12:19:12Z","updated_at":"2015-06-11T05:52:35Z","closed_at":null,"body":"Problème des fonctions mysql_ dépréciées à partir de PHP5.5, abandonnées en PHP7."},"comment":{"url":"https://api.github.com/repos/Baraban35/SuiviSIO/issues/comments/111004030","html_url":"https://github.com/Baraban35/SuiviSIO/issues/2#issuecomment-111004030","issue_url":"https://api.github.com/repos/Baraban35/SuiviSIO/issues/2","id":111004030,"user":{"login":"Baraban35","id":12734170,"avatar_url":"https://avatars.githubusercontent.com/u/12734170?v=3","gravatar_id":"","url":"https://api.github.com/users/Baraban35","html_url":"https://github.com/Baraban35","followers_url":"https://api.github.com/users/Baraban35/followers","following_url":"https://api.github.com/users/Baraban35/following{/other_user}","gists_url":"https://api.github.com/users/Baraban35/gists{/gist_id}","starred_url":"https://api.github.com/users/Baraban35/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Baraban35/subscriptions","organizations_url":"https://api.github.com/users/Baraban35/orgs","repos_url":"https://api.github.com/users/Baraban35/repos","events_url":"https://api.github.com/users/Baraban35/events{/privacy}","received_events_url":"https://api.github.com/users/Baraban35/received_events","type":"User","site_admin":false},"created_at":"2015-06-11T05:52:35Z","updated_at":"2015-06-11T05:52:35Z","body":"Bonjour Gwenaël,\nBien pour le commentaire. Quelques remarques :\n1) Ce qui a été fait pour le moment :\nLa classe de test MySqlTest doit être strictement identique pour tester les\nméthodes de la classe MySql, que celle-ci s'appuie sur la bibliothèque de\nfonctions MySql ou les classes PDO et PDOStatement. Est-ce bien le cas ?\n\n2) Ce qui reste à faire :\ncorrection de bugs : ok pour la demi-journée\n+ 1h : écriture et exécution de tests unitaires supplémentaires :\nméthode testExecSQL à écrire avec un cas de test pour chaque requête action\nutilisée : insert, update et delete, et en allant vérifier le résultat\nretourné (cas 1 ou plusieurs lignes affectées), ce que ne fait pas la\nméthode de test testInsertId car ce n'est pas son rôle premier de tester\nexecSQL.\n+ 1h :  voir quelle(s) solution(s) simple(s) pourraient convenir pour\néviter de modifier directement la classe MySql à 2 lignes distinctes\n(include) lorsqu'elle est utilisée par PHPUnit ou par l'application web. Je\nregarde aussi de mon côté.\n- 1h : vérification de la bonne exécution de l'application web avec cette\nnouvelle classe MySQL. Il faut tester quelques fonctionnalités des trois\nrôles (admin, enseignant, étudiant) pour voir s'il n'y a pas eu régression.\n- 1h : réécrire le fichier enregparam.php en s'appuyant sur les classes PDO\net PDOStatement. Faire un test à partir de instal.php, ce dernier contenant\nle formulaire pour saisir les données d'installation, puis appelant sur\nsoumission le script enregparam.php. Ne pas passer de temps à faire des\ntests variés : l'installateur n'est pas recommandé, mais il vaut mieux le\nmigrer sous PDO en attendant une refonte plus complète.\n- faire un commit des scripts modifiés en référençant l'issue, puis le\npublier sur la branche master de la plateforme github.\n\nLes propositions de temps sont données à titre indicatif. Ce sera\nintéressant de comparer ce qui est prévu à ce qui est effectif.\nBonne journée.\nPS : Je teste l'envoi de la réponse à Baraban35/SuiviSIO. On va voir si\ncette réponse va être visible directement sur la plateforme github.\nA. Baraban\n\n2015-06-10 20:14 GMT+02:00 GDucourneau <notifications@github.com>:\n\n> Ce qui a été fait pour le moment :\n>\n>    - création de quelques tests qui vérifient la fonctionnalité des\n>    méthodes de classe mysql (avec l'extension Mysql)\n>    - modification des méthodes de la classe mysql (utilisation des\n>    classes PDO et PDOStatement au lieu de l'extension Mysql)\n>    - création de quelques tests qui vérifient la fonctionnalité des\n>    méthodes de classe mysql (avec les classes PDO et PDOStatement)\n>\n> Ce qu'il reste à faire :\n>\n>    - correction de diverses bugs afin que tous les tests fonctionnent\n>    (plus ou moins une demi-journée)\n>    - création de documentations pour les méthodes de la classe mysql\n>    (plus ou moins une demi-journée)\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/Baraban35/SuiviSIO/issues/2#issuecomment-110861106>.\n>\n"}}