Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Add default notifiers to the main module, and be able to edit it through the UI module.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | default-notifiers-1825882-7.patch | 3.04 KB | Moo64c |
| #4 | default-notifiers-1825882-4.patch | 2.54 KB | Moo64c |
| #1 | default-notifiers-1825882-1.patch | 2.36 KB | Moo64c |
Comments
Comment #1
Moo64c CreditAttribution: Moo64c commentedNeeds the patch for message notify to work:
http://drupal.org/node/1825962#comment-6662668
Comment #2
Moo64c CreditAttribution: Moo64c commentedComment #4
Moo64c CreditAttribution: Moo64c commentedReroll.
Comment #5
Moo64c CreditAttribution: Moo64c commentedComment #7
Moo64c CreditAttribution: Moo64c commentedRerolled and fixed tests.
Comment #8
Moo64c CreditAttribution: Moo64c commentedComment #9
amitaibuCommitted, thanks.
Comment #11
japerryUnfortunately this patch, with an email set as default, makes it impossible to override the default notifications setup.
In commons, we have message_subscribe_email do the emailing if people elect to have an email sent to them. In the current setup, as soon as you follow someone it'll start emailing you, regardless of if you have the email flag set.
See: #2001702: Message Subscribe sends emails regardless of flag checkmarks in notification-settings for more information.
For now we're setting the array to null, but the UI should have a way to uncheck this option. If message_subscribe_email is enabled, we may want to set this setting differently.
Marking needs work.
Comment #13
joelpittet