Chamilo LCMS Connect » Core » Common

Currently the Database works with direct calls to mdb2 wich is quite obsolete, spawn errors, is absolutely not OO, is not very performant, is not very extensible, ....

since PHP 5.1 PDO has been a standard for abstracting the access to a database (it does not abstract the SQL, it abstract the driver).

Stephane from the clarolien team is currently trying to start our an application for Chamilo and try to test any piece of functionality he's writing. Unfortunately MDB2 goes into it's way and he tried to hack the Database class for using PDO instead of MDB2. Currently he is obtaining good results but cannot know the impact for other application (even if he doesn't change the methods signature, the behaviour might change).

What would you think of changing the calls to mdb2 by calls to pdo, or change the design of Database class to allow both possibilities based on config options (maybe keep mdb2 n prod and pdo in dev)

How can we share his work ?

PS for those who speak french, here is his message :

"
Salut,

Je t'écris pour faire un peu le point sur la situation et pour te proposer un sujet de réflexion pour notre prochaine réunion.

La semaine dernière, j'ai poursuivi les tests en isolation et en intégration sur la couche modèle de l'application. L'écriture de ces tests
ne posait pas de problème particulier : pris isolément, ils fonctionnaient parfaitement. En revanche, l'ajout d'un nouveau test entraînait presque
systématiquement des problèmes dans les tests déjà écrits. On a vu ensemble que cette situation était liée à l'utilisation de globales par MDB2. J'ai
donc essayé, comme nous l'avions fait, d'utiliser l'option 'backupGlobals disabled' ou de la désactiver là où elle semblait poser problème. Mais rien n'y faisait, et je perdais au moins une heure par jour à tenter toutes sortes de combinaisons pour faire fonctionner l'ensemble.

Ce week-end, j'ai exploré une voie beaucoup plus radicale : remplacer MDB2 par PDO. Je sais que ce n'était pas au programme, je sais que ce genre de remplacement concerne tous les développeurs de chamilo et ne peut se faire à la légère. Mais j'ai quand même essayé, à titre expérimental en quelque sorte. J'ai réimplémenté la classe Database de chamilo en remplaçant tous les appels à MDB2 par des appels à PDO (sans, bien sûr, modifier quoi que ce soit dans les attributs, les noms et les signatures de méthodes); j'ai également retouché quelque classes apparentées, en particulier la classe ObjectResultSet.

Ceci n'est encore qu'un brouillon mais le résultat est tout de même très significatif. Sans plus aucun appel à MDB2, mon installation de chamilo fonctionne parfaitement et mes tests tournent maintenant sans aucun problème. Les performances me semblent également meilleures (bien que sans test dédié, cette impression
reste assez subjective) ainsi que les répercussions possibles sur l'écriture du code : full-orienté objet, clair, propre et ouvert à une gestion efficace des erreurs.

Je ne vais pas te faire un éloge de PDO, dont tu connaîs sûrement mieux que moi les avantages. Je crois que la vraie question concernant chamilo n'est pas : "faut-il remplacer MDB2 par PDO ?" mais bien : "combien de temps chamilo pourra-t-il tenir sur une couche modèle notoirement périmée ?". Et concernant notre propre travail autour de ce projet, je pense que la question est encore plus critique. Bâtir une application sur une base aussi hasardeuse finira, tôt ou tard,
par devenir très problèmatique. Ne vaudrait-il pas mieux de suivre directement une voie plus sûre ?

Cette option entraînerait bien sûr pas mal de changements du côté de chamilo, et je ne pense pas que la transition puisse se faire absolument sans douleur. Mais c'est possible et souhaitable, pour peu que les développeurs de chamilo soient près à tenter le coup d'une manière ou d'une autre.

Bon, je te laisse réfléchir à tout ça (désolé pour la longueur), et je te souhaite une bonne fin de journée.

À jeudi,

Stéphane
"