Closed Skryz974 closed 2 years ago
Hi,
Thanks for those modifications. I have two issues though:
Your changes are quite significant. 1.0 seems a big step for such a small, un maintained hack, but 0.3.0 definitely makes sense.
I have integrated most of your modifications into my code. I will reject the PR as it is not needed anymore. I am not using GLPI anymore so I was not able to test anything. I hope it all works.
Thanks for your work anyway.
Bonjour,
Désolé j'ai un anglais assez médiocre donc je préfère m'exprimer en français.
A part changer les fonctions qui était obsolète pour GLPI 9.5 et remplacer les fonctions "odbc" par "sqlsrv", je n'ai pas toucher au cœur même du plugins. donc il ne devrait pas avoir de soucis avec le fonctionnement même si je n'ai pas tout retester. Avec du recul je me dis que je n'aurais pas du enlever odbc mais juste modifier le code pour utiliser les deux modules en fonction du choix de l'utilisateur. Je referais des modifications dans ce sens pour une v0.3.1, pour ne pas faire perdre la compatibilité aux utilisateurs qui feront juste une monté de version. Je n'ai pas beaucoup de temps pour bosser sur ce projet, mais il y a encore pas mal de choses que je voudrais ajouter afin de couvrir toutes les fonctionnalités de MDT. Il faut que je prenne le temps de bien lire tout le code avant pour voir tout ce qui est déjà proposé. Peut-être que les futurs modifications pourront faire l'objet d'une V1.0.0 👍🏽. Même si vous n'utilisez plus GLPI ça sert toujours à la communauté. En tout cas merci pour l'intégration 😊
Add compatibility with GLPI 9.5. Replace function ODBC by SQLSRV Change the url to check the version