Merge pull request #10630 from tschuettler/CRM-CRM-20841

CRM-20841 - Dedupe - Show on_hold, is_bulkmail or signature merge…
civicrm · Jun 6, 2018 · 1c5bc02 · 1c5bc02
2 parents e09262a + 2a7e1dd
commit 1c5bc02
Show file tree

Hide file tree

Showing 3 changed files with 59 additions and 4 deletions.
diff --git a/CRM/Dedupe/Merger.php b/CRM/Dedupe/Merger.php
@@ -1034,7 +1034,7 @@ public static function getLocationBlockInfo() {
       ),
       'email' => array(
         'label' => 'Email',
-        'displayField' => 'email',
+        'displayField' => 'display',
         'sortString' => 'location_type_id',
         'hasLocation' => TRUE,
         'hasType' => FALSE,
@@ -1231,6 +1231,10 @@ public static function getRowsElementsAndInfo($mainId, $otherId, $checkPermissio
               CRM_Core_BAO_Address::fixAddress($value);
               $locations[$moniker][$blockName][$cnt]['display'] = CRM_Utils_Address::format($value);
             }
+            // Fix email display
+            elseif ($blockName == 'email') {
+              $locations[$moniker][$blockName][$cnt]['display'] = CRM_Utils_Mail::format($value);
+            }
 
             $cnt++;
           }

diff --git a/CRM/Utils/Mail.php b/CRM/Utils/Mail.php
@@ -539,4 +539,42 @@ public static function appendPDF($fileName, $html, $format = NULL) {
     );
   }
 
+  /**
+   * Format an email string from email fields.
+   *
+   * @param array $fields
+   *   The email fields.
+   * @return string
+   *   The formatted email string.
+   */
+  public static function format($fields) {
+    $formattedEmail = '';
+    if (!empty($fields['email'])) {
+      $formattedEmail = $fields['email'];
+    }
+
+    $formattedSuffix = array();
+    if (!empty($fields['is_bulkmail'])) {
+      $formattedSuffix[] = '(' . ts('Bulk') . ')';
+    }
+    if (!empty($fields['on_hold'])) {
+      if ($fields['on_hold'] == 2) {
+        $formattedSuffix[] = '(' . ts('On Hold - Opt Out') . ')';
+      }
+      else {
+        $formattedSuffix[] = '(' . ts('On Hold') . ')';
+      }
+    }
+    if (!empty($fields['signature_html']) || !empty($fields['signature_text'])) {
+      $formattedSuffix[] = '(' . ts('Signature') . ')';
+    }
+
+    // Add suffixes on a new line, if there is any.
+    if (!empty($formattedSuffix)) {
+      $formattedEmail .= "\n" . implode(' ', $formattedSuffix);
+    }
+
+    return $formattedEmail;
+  }
+
 }
diff --git a/templates/CRM/Contact/Form/Merge.tpl b/templates/CRM/Contact/Form/Merge.tpl
@@ -114,8 +114,20 @@
 
           <td>
             {* @TODO check if this is ever an array or a fileName? *}
-            {* This is on one long line for address formatting *}
-            {if $row.title|substr:0:7 == "Address"}<span style="white-space: pre">{else}<span>{/if}{if !is_array($row.other)}{$row.other}{elseif $row.other.fileName}{$row.other.fileName}{else}{', '|implode:$row.other}{/if}</span>
+            {if $row.title|substr:0:5 == "Email"   OR
+                $row.title|substr:0:7 == "Address"}
+              <span style="white-space: pre">
+            {else}
+              <span>
+            {/if}
+            {if !is_array($row.other)}
+              {$row.other}
+            {elseif $row.other.fileName}
+              {$row.other.fileName}
+            {else}
+              {', '|implode:$row.other}
+            {/if}
+            </span>
           </td>
 
           <td style='white-space: nowrap'>
@@ -131,7 +143,8 @@
 
             <td>
               {strip}
-                {if $row.title|substr:0:7 == "Address"}
+                {if $row.title|substr:0:5 == "Email"   OR
+                    $row.title|substr:0:7 == "Address"}
                   <span style="white-space: pre" id="main_{$blockName}_{$blockId}">
                 {else}
                   <span id="main_{$blockName}_{$blockId}">