used 7.x-1.0-alpha1 and the latest dev of simplenews
when
1 term is chosen with or without Allow individual selection of newsletters checked
no checkbox appears and no email is saved in database upon submission (also look at this issue #1364832: single opt_inout newsletter sends confirmations emails )
2 term is chosen with Allow individual selection of newsletters checked
checkbox appears but also
Notice: Undefined index: 216 in _webform_render_newsletter_email() (line 141 of /home/domain/public_html/sites/all/modules/webform_simplenews/webform_simplenews.inc).
the emails are saved correctly
when 2 term is chosen without Allow individual selection of newsletters checked
no checkboxes appear and no notice but no emails are saved in simplenews
issue seems to arise from the fact that simplenews no longer uses any variable_get('simplenews_ as per this #1364850: single opt_inout newsletter block sends confirmations emails
Comments
Comment #1
bmunslow commentedHi,
I come accross the same issue, and after some research, this is what I've come up with:
- Problems arises when you only have 1 newsletter category. The submit hook overrides the value of the selected category:
The problem is easily solved by simply deleting the 'else' structure
As you also well mention, there's another problem with 'variable_get' which is no longer used in simplenews.
This only affects the confirmation email, which is never send, even if the user is not subscribed.
This workaround works fine for me:
Always confirm unless it's a registered users. (I personally think double opt-in is the way to go... regardless of simplenews...).
BTW, pcambra, thank you for a great module (I can't believe this was not done before!). And congratulations for your excellent performance in the Barcelona Drupal Day 2011 ;-)
Comment #2
r0bm1lls commentedI'm struggling to understand this issue but seem to have the same problem. I set up a webform and add the weform_simplenews component. All that appears is a field to input and email address, no options to select which newsletter category (I have three currently). When the form is submitted the email address does not appear in any subscriber list. No confirmation email seems to go out either.
It also seems the alpha version has disappeared since the issue was posted. The functionality mentioned does not seem to exist any more.
Rob
Comment #3
bmunslow commentedHi Rob,
Have you tried the latest dev version with latest stable version of simplenews?
It works fine for me. Multiple newsletter categories + "Allow individual selection of newsletter" checkbox.
Aforementioned issues have already been solved in the dev version.
Comment #4
r0bm1lls commentedHi,
At last I have unraveled this mystery. The form builder does not accommodate any detail about newsletter categories etc. When I used the raw webformform editor I got the details about category etc. I also now see, having done this that (of course) the subscription confirmation email goes off and so the email address will not appear in the subscriptions list until the subscription has been confirmed.
I want my newsletter subscriptions to work with a confirmation email to verify email addresses but I would like the webform component to add the email address without a confirmation email. Is that possible?
Rob
Comment #5
chirhotec commentedNotice: Undefined index: 1 in _webform_render_newsletter_email() (line 142 of /home/domain/public_html/sites/all/modules/webform_simplenews/webform_simplenews.inc).
I'm getting a similar error using the latest dev version of this module, and the latest Webform 3.x release (7.x-3.18).
I have multiple newsletters enable, and the option to select them individually
EDIT: I just noticed that if a registered user submits the post using his/her information, its showing up in People > Newsletter Subscriptions, however, if an anonymous user submits information it doesn't appear in that list. It does appear in a custom table View of subscriber information. Users are shown as being activated (I have it set to Single Opt-in/out)
Comment #6
stijn vanhandsaeme commentedI can confirm #1 works for me.
Comment #7
brandy.brown commented#1 did not work for me. I'm using the latest dev.