[Deprecated] DMZ 1.6.2 (DataMapper OverZealous Edition)

[eluser]OverZealous[/eluser]
My current idea to solve the above issue has two parts.

The first is to modify the default case of connecting to only one DB, the default one. ($db_params = '') In this case, DMZ now clones $CI->db (instead of creating a new one), which should ensure that all DMZ db objects use the database connection. This works well, and has already helped the situation. (This might also have a tiny performance improvement, as cloning can be a little faster than instantiating.)

The second is to register each $db object that is created with a shutdown hook. This not only fixes the current problem (where creating more than one DB doesn't shut them down), it also fixes the case where you use redirect() or exit() to exit the application, by ensuring the DB is closed if the application exits. (The DB won't get closed more than once, because CI checks for that already.)

I've already submitted a bug report to CodeIgniter suggestion that they use register_shudown_function() for all database connections created.

Now, if you connect to multiple databases, the first fix won't provide much performance benefit, but the second one should ensure that those connections are also closed cleanly.

This fix will be rolled into the 1.7.0 release, as long as I don't find any other issues with it.