Select MySQL as default database driver, and fix a E_ALL warning during install

No panda shall remain sad.

current(array_keys($drivers)) would be a bit more clear as just key($drivers).

Not really. key($drivers) is the key of the current position in $drivers. current(array_keys($drivers)) is the first key of $drivers. There is no real guarantee that both will be equal.

+ $database = isset($databases['default']['default']) ? $databases['default']['default'] : array(); is weird. $database['default']['default']? I realize this isn't specific to this patch, but why a nested array like that? Can we add some documentation to that line?

The syntax of the $databases array is described in settings.php:

* For each database, you may optionally specify multiple "target" databases.
* A target database allows Drupal to try to send certain queries to a
* different database if it can but fall back to the default connection if not.
* That is useful for master/slave replication, as Drupal may try to connect
* to a slave server when appropriate and if one is not available will simply
* fall back to the single master server.
*
* The general format for the $databases array is as follows:
*
* $databases['default']['default'] = $info_array;
* $databases['default']['slave'][] = $info_array;
* $databases['default']['slave'][] = $info_array;

Unfortunately, this panda will have to wait for a re-roll. Patch no longer applies. The end() trick might be better than the current trick.

The patch in #7 is good to go, thanks justin.

Committed to CVS HEAD. Thanks!