Add grouping options for contribution thank you letters

diff --git a/CRM/Contribute/Form/Task/PDFLetter.php b/CRM/Contribute/Form/Task/PDFLetter.php
index f7e620192bdf799f980ed00dfc7bb6f96ea7a042..8ac20f3616b3b83eaf29eae8eb8fcbae5d4c1b47 100644 (file)
--- a/CRM/Contribute/Form/Task/PDFLetter.php
+++ b/CRM/Contribute/Form/Task/PDFLetter.php
@@ -124,7 +124,8 @@ class CRM_Contribute_Form_Task_PDFLetter extends CRM_Contribute_Form_Task {
     );
     $this->addElement('select', 'group_by', ts('Group contributions by'), $options, array(), "<br/>", FALSE);
     // this was going to be free-text but I opted for radio options in case there was a script injection risk
-    $separatorOptions = array('comma' => 'Comma', 'td' => 'Table Cell');
+        $separatorOptions = array('comma' => 'Comma', 'td' => 'Horizontal Table Cell', 'tr' => 'Vertical Table Cell', 'br' => 'Line Break');
+
     $this->addElement('select', 'group_by_separator', ts('Separator (grouped contributions)'), $separatorOptions);
     $emailOptions = array(
       '' => ts('Generate PDFs for printing (only)'),

diff --git a/CRM/Contribute/Form/Task/PDFLetterCommon.php b/CRM/Contribute/Form/Task/PDFLetterCommon.php
index 4dcba8ef7e1c8ecac33f86c2205a907cb5f07da5..5f2876bc93ed86e467bd598361f042453e7e2fc6 100644 (file)
--- a/CRM/Contribute/Form/Task/PDFLetterCommon.php
+++ b/CRM/Contribute/Form/Task/PDFLetterCommon.php
@@ -40,7 +40,13 @@ class CRM_Contribute_Form_Task_PDFLetterCommon extends CRM_Contact_Form_Task_PDF
       if ($formValues['group_by_separator'] == 'td') {
         $realSeparator = "</td><td>";
       }
-    }
+               elseif ($formValues['group_by_separator'] == 'tr') {
+                 $realSeparator = "</td></tr><tr><td>";
+               }
+               elseif ($formValues['group_by_separator'] == 'br') {
+                 $realSeparator = "<br />";
+               }
+        }
     $separator = '****~~~~';// a placeholder in case the separator is common in the string - e.g ', '
     $validated = FALSE;