Request option to NOT send emails [#2365807]

Great module - thank you!

Requesting that on the Node Ownership Settings page, under Email Settings that for each email there is an option (checkbox?) like "Do not send these emails".

I actually only need for the first one, Admin email. With thousands of nodes being claimed on my site, I would like to cut down on all of these emails coming back to me that I don't need.

Even though I would only want to "turn it off" for one of the 3 emails, it makes sense to give this option to not send the emails for each of the 3 emails types, since others might want to disable all of the emails.

Thank you!

Comment	File	Size	Author
#6	interdiff-23650807-4-6.txt	3.96 KB	Vinay15
#6	2365807-6.patch	4.47 KB	Vinay15
#4	disable_emails_on_settingform-23650807-#3.patch	4.65 KB	Pradnya Pingat
#3	disable_emails_on_settingform-23650807-#2.patch	4.67 KB	Pradnya Pingat

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

swarad07

English

Navi Mumbai, India

CreditAttribution: swarad07 commented 30 October 2014 at 05:30

Acknowledged, we will add this in our plan.

Comment #2

Pradnya Pingat CreditAttribution: Pradnya Pingat commented 9 May 2016 at 10:31

Assigned:

Unassigned

» Pradnya Pingat

Comment #3

Pradnya Pingat CreditAttribution: Pradnya Pingat commented 10 May 2016 at 06:26

Status:

Active

» Needs review

File	Size
disable_emails_on_settingform-23650807-#2.patch	4.67 KB

Adding checkbox for all three email options to disable mails .

Comment #4

Pradnya Pingat CreditAttribution: Pradnya Pingat commented 10 May 2016 at 06:42

File	Size
disable_emails_on_settingform-23650807-#3.patch	4.65 KB

1 file was hidden/shown/deleted

File	Size
disable_emails_on_settingform-23650807-#2.patch	4.67 KB

Adding checkbox for all three email options to disable mails .updating coding standard issues.

Comment #5

swarad07

English

Navi Mumbai, India

CreditAttribution: swarad07 at Blisstering Solutions commented 14 June 2016 at 12:52

Status:

Needs review

» Needs work

+++ b/nodeownership.module
@@ -157,8 +157,16 @@ function nodeownership_claim_insert() {
+  ¶
...
+    $mail = drupal_mail('nodeownership', 'user_claim', $recipient['mail'], $language);
+    drupal_set_message(t('Your claim has been recorded and an email has been sent to the site administrators.'));
...
+  else {
+    drupal_set_message(t('Your claim has been recorded.'));   ¶
+  }

Remove Empty spaces. There are many in your patch.