drupal_write_record() silently converts an array to an object in case of an exception

If db_write_record() is passed an array and the insertion fails for any reason, it throws an exception. However, it does not handle the exception and turn the passed-in object back to an array.

To reproduce the bug, I have attached a simple demo module. Essentially, any kind of insert which will throw will do the job. Use something like this:

  $entry = array(.... whatever);
  try {
    $result = drupal_write_record('db_write_record_bug', $entry);
  }
  catch (Exception $e) {
    drupal_set_message(t('Insertion: drupal_write_record failed. typeof($entry)=@type,  Message = %message, query= %query',
      array('@type' => gettype($entry), '%message' => $e->getMessage(), '%query' => $e->query_string)), 'error');
  }

What I expect to happen: An exception is thrown and $object (which is an array in this case) is unchanged... or at least it's still an array.

What happens: $entry is converted silently to an object, meaning that the caller will likely get fatal errors trying to deal with the error.

The attached patch is a way to handle this without interface changes. I think it might be better not to pass $object by reference - that way arrays would be unaffected by this function.

The patch is tiny (just a try/catch). It shows up as a few lines because of the indentation of the affected block.