Just found following:

/**
 * Implements hook_install().
 */
function subscriptions_mail_install() {
  if (module_exists('mail_edit')) {
    // Add column to {mail_edit} table.
    _subscriptions_mail_extend_mail_edit();
  }
}

/**
 * Database update function 7000: Extend the {mail_edit} table.
 */
function subscriptions_mail_update_7000() {
  if (db_table_exists('mail_edit') && !db_field_exists('mail_edit', 'subscriptions_comment_body')) {
    // Add column to {mail_edit} table.
    _subscriptions_mail_extend_mail_edit();
  }
}

/**
 * Adds a column to the {mail_edit} table.
 */
function _subscriptions_mail_extend_mail_edit() {
  if (!function_exists('subscriptions_mail_schema_alter')) {
    $t = get_t();
    throw new DrupalUpdateException($t('@Subscriptions_Mail must be enabled before it can be updated', array(
      '@Subscriptions_Mail' => 'Subscriptions Mail',
    )) . ' - ');
  }
  $new_schema = array();
  subscriptions_mail_schema_alter($new_schema);
  foreach ($new_schema['mail_edit']['fields'] as $name => $spec) {
    db_add_field('mail_edit', $name, $spec);
  }
}

I don't think what there is no way to integrate it without this hack.
Can you please get rid of it?

Comments

salvis’s picture

Priority: Critical » Normal
Status: Active » Closed (works as designed)

We're not hacking ME's schema and this is neither a bug nor critical.

hook_schema_alter() is a legitimate core features.