On the configuration page (admin/config/services/simplenews), the list of available newsletters is called using simplenews_newsletter_get_visible(). This is fine except if you create a newsletter with the 'Opt-in/out method' set to hidden, you get a drupal_set_message saying you have created a newsletter but can't see it in your list of newsletters. On top of this you don't know the id of that newsletter and need to do a database query just to find it in order to go to is edit page and change that setting.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | simplenews-simplenews_configuration_fails_hidden_newsletters-1891160-1.patch | 684 bytes | yanniboi |
Comments
Comment #1
yanniboi commentedI am still not that familiar with the 7.x-2.x api so I wasn't sure if a helper function already exists for 'all' newsletters so I used an entity_load() to fix this.
Comment #2
berdirYes, that is correct for now. The idea is that the whole page is going to be replaced with a view but until then, this is a correct fix.
Comment #3
yanniboi commentedCool, thanks for the quick feedback!
Comment #4
berdirThanks, commited.