Closed (won't fix)
Project:
Amazon SES
Version:
7.x-1.0-beta2
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
6 Jun 2011 at 20:25 UTC
Updated:
12 Mar 2015 at 06:11 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Ryan Palmer commentedI agree!
Comment #2
mikeker commentedThe attached patch switches the dependency of this module from Creeper to AWS SDK for the D7 branch. But this might be a bit surprising for the 25 or so installs if they simply "drush up" to pull in the latest changes.
To allow EITHER Creeper or AWS SDK, we could remove the dependencies[] line in the .info file, replace amazon_ses_enable() with
and to wrap all calls the libraries_load('awssdk') with a check that awssdk is enabled. That would make the upgrade for existing users easier, but would lose the automatic checks that dependent modules are enabled before this module is enabled.
My personal preference would be to rip off the band-aid, namely to switch the dependency and require existing users to install AWS SDK...
NOTE: The attached patch also does a few housecleaning items mostly in amazon_ses.admin.inc. Those are unrelated to this issue and I'm happy to remove them if you want a cleaner patch. But they do solve issues with undefined vars on the settings page.
Comment #3
dgwave commentedThis is the patch that I used to enable AWS SDK, with some error checking.
Comment #4
jcanfield55 commentedThanks for this patch -- it worked great. I was newly installing Amazon_SES and I was unable to get Creeper working after about a day of trying. Being able to switch to awssdk module was a lifesaver.
Comment #5
tkuldeep17 commentedThis version is not supported now.