Merge pull request #1765 from lynndanzig/master

[civicrm-core.git] / CRM / Mailing / BAO / MailingJob.php

diff --git a/CRM/Mailing/BAO/MailingJob.php b/CRM/Mailing/BAO/MailingJob.php
index d9919f106f440649513c8a0995d34e9eba346606..fab73ea23a236859f5204fc9b32e0eb604dcfe48 100644 (file)
--- a/CRM/Mailing/BAO/MailingJob.php
+++ b/CRM/Mailing/BAO/MailingJob.php
@@ -518,11 +518,7 @@ VALUES (%1, %2, %3, %4, %5, %6, %7)
     $fields = array();
 
     if (!empty($testParams)) {
-      $mailing->from_name = ts(
-        'CiviCRM Test Mailer (%1)',
-        array(1 => $mailing->from_name)
-      );
-      $mailing->subject = ts('Test Mailing:') . ' ' . $mailing->subject;
+      $mailing->subject = ts('[CiviMail Draft]') . ' ' . $mailing->subject;
     }
 
     CRM_Mailing_BAO_Mailing::tokenReplace($mailing);
@@ -935,9 +931,21 @@ AND    civicrm_activity.source_record_id = %2
 
         // CRM-9519
         if (CRM_Core_BAO_Email::isMultipleBulkMail()) {
+          static $targetRecordID = NULL;
+          if (!$targetRecordID) {
+            $activityContacts = CRM_Core_OptionGroup::values('activity_contacts', FALSE, FALSE, FALSE, NULL, 'name');
+            $targetRecordID = CRM_Utils_Array::key('Activity Targets', $activityContacts);
+          }
+
           // make sure we don't attempt to duplicate the target activity
           foreach ($activity['target_contact_id'] as $key => $targetID) {
-            $sql = "SELECT id FROM civicrm_activity_target WHERE activity_id = $activityID AND target_contact_id = $targetID;";
+            $sql = "
+SELECT id
+FROM   civicrm_activity_contact
+WHERE  activity_id = $activityID
+AND    contact_id = $targetID
+AND    record_type_id = $targetRecordID
+";
             if (CRM_Core_DAO::singleValueQuery($sql)) {
               unset($activity['target_contact_id'][$key]);
             }