Problem/Motivation

I used the mandrill module on a project and needed to add some functionality. (Module works great, thanks!) I created a custom module for my project that allows site administrators to add a user's email address to Mandrill's unsubscribe/reject list. I created my own copy of mandrill_get_api_object() so that I could create an object from a custom class that inherits the Mandrill class. This works but it now requires duplicate maintenance of that function moving forward.

Proposed resolution

Add parameter to mandrill_get_api_object() to specify a class name. This will allow modules to create classes that inherit DrupalMandrill and call mandrill_get_api_object(FALSE, 'DrupalMandrillFooBar');.

Remaining tasks

  1. review patch

Comments

daven’s picture

StatusFileSize
new739 bytes
daven’s picture

Also, I hope to share my "mandrill_user" module. I could create a new drupal project for it or submit it as a submodule in the main mandrill project. Do the mandrill module maintainers have a preference for this?

thanks,
Dave

levelos’s picture

Status: Needs review » Fixed

@daven, patch applied. I'm not sure about the mandrill_user module, at least for now. Thanks.

  • Commit 3d03988 on 7.x-1.x by levelos:
    #2220663 by daven: Add parameter to mandrill_get_api_object for module...

  • Commit 3d03988 on 7.x-1.x, 7.x-2.x by levelos:
    #2220663 by daven: Add parameter to mandrill_get_api_object for module...

Status: Fixed » Closed (fixed)

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