Hey Lev,

I just noticed a bug when disabling the mailchimp_sts module. Basically, it appears that when this module is disabled on the modules page (after being enabled), the system is not reset to the default mail system. Thus, it seems like we need a fix similar to http://drupalcode.org/project/mailchimp.git/commit/2c29564 (except that we need it to apply to just disabling the module instead of uninstallation).

As for recreating the error, this first surfaced in my testing with Drupal Commerce. Basically, I had mailchimp_sts working and then I decided to disable it when I was testing some Drupal Commerce functionality. Once I did this, I suddenly received an error message/page when completing an order in Drupal Commerce (presumably because the order confirmation e-mail could no longer fire on order completion). I confirmed that re-enabling mailchimp_sts caused the error to go away and then disabling mailchimp_sts caused the error to reoccur.

Let me know if you need any other information.

Hope all is well,
Ben

Comments

levelos’s picture

levelos commented
Status: Active » Fixed

Thanks Ben. I moved that reset code to hook_disable() from hook_uninstall(), the former having to be called before the latter. Should take care of it.

Status: Fixed » Closed (fixed)

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