When you look at drupal_install_schema(), you see:

function drupal_install_schema($module) {
  $schema = drupal_get_schema_unprocessed($module);
  _drupal_initialize_schema($module, $schema);

  $ret = array();
  foreach ($schema as $name => $table) {
    db_create_table($ret, $name, $table);
  }
}

You'll notice that it collects the return values from creating the database tables, but it doesn't do anything with them. Would it make more sense to return what was collected from creating the tables? return $ret; would make sense. The same can be done with drupal_uninstall_schema()... See attached patch.

Comments

dries’s picture

Status: Needs review » Fixed

Good question. I don't think we should do anything with it, unless there is an actual use case for it. I'm going to mark this "won't fix" but feel free to re-open the issue when you have a use case. Thanks Rob.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

pancho’s picture

Just a notice: a similar patch by bjaspan has been committed in November 2007 (see #190899)