Closed (fixed)
Project:
SunMailer Newsletter
Version:
6.x-2.0-beta2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Oct 2011 at 20:08 UTC
Updated:
16 Jan 2012 at 02:40 UTC
I'm excited about the 2-x branch of your module as it appears to do most of the things I need for our web site. However, when I change the Mailing method to "SMTP" and click "Submit," the radio button isn't remembered. The information I filled in remains, but the radio button reverts to mail.
Comments
Comment #1
jonlibrary commentedI've made some modifications to get it working, but they aren't ideal.
1) In sunmailer.admin.inc, I changed '#default_value' => variable_get('sunmailer_method', 'mail'), to
'#default_value' => variable_get('sunmailer_method', 'smtp'), .
2) But then SunMailernewsletter.class.php still tried to process it as if 'mail' was selected. (See the switch at line 34.) So I copied the code for case 'smtp' to case 'mail'.
And now it works! But, as I said, this isn't ideal, so I'm leaving the issue open. In (1) my guess is there's something wrong with the form. Not sure in the other case. This is the best I could do...for now!
Comment #2
Mike Wacker commentedhttp://drupalcode.org/project/sunmailer.git/commit/9bf15d6
The value you selected was being saved in the database, and SMTP would be used in the mail. It was a UI bug, the mail option was being displayed on the form regardless of what option was currently enabled and saved in the database.